Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add possibility to configure wrapping length of Dumper-class #56

Closed
wants to merge 1 commit into from

Conversation

duxthefux
Copy link

@duxthefux duxthefux commented Feb 10, 2020

@duxthefux
Copy link
Author

Please note that static getter and setter was added to force type safety

@dg
Copy link
Member

dg commented Feb 13, 2020

Thanks for PR.

I have this thing on the TODO list. However, the solution is not so simple, because it is not possible to use static variables in Nette (because it follows the principles of dependency injection).

So this will be a little harder to solve. I'll leave it open so far.

@duxthefux
Copy link
Author

hey @dg,

any update on this?
As far as I can see it no dependency injection via nette container is used for the Dumper-class currently. So maybe it's not that tragic to use a static property now, as a lot of refactoring needs to be done anyways to get rid of calls like (new Dumper)->format($code, ...$args).

Happy to hear your opinion.

@dg dg force-pushed the master branch 6 times, most recently from 40d6435 to f4f1e78 Compare May 6, 2020 21:27
@dg dg force-pushed the master branch 2 times, most recently from c4955b7 to ea2c8e8 Compare May 26, 2020 16:35
@dg dg force-pushed the master branch 2 times, most recently from 53b1239 to 7051954 Compare June 19, 2020 14:32
@dg dg force-pushed the master branch 9 times, most recently from 668af8c to fc70bcb Compare October 22, 2020 16:12
@dg
Copy link
Member

dg commented Sep 23, 2021

Printer has new property $wrapLength

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add possibility to configure wrapping length of Dumper-class
2 participants