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 support for LineFormatter replace inline linebreaks in indexed arrays data #1818

Merged
merged 1 commit into from
Oct 27, 2023
Merged

Conversation

lddtime
Copy link
Contributor

@lddtime lddtime commented Jul 7, 2023

When I pass in an indexed array or a multi-dimensional array, allowInlineLineBreaks is not working!

I think this revision is useful.

$log->info('Testing new lines in log context.', ["This\nhas\nNew Lines."]);

// before
// local.INFO: Testing new lines in log context. ["This\nhas\nNew Lines."] 

// after
// local.INFO: Testing new lines in log context. ["This
// has
// New Lines."] 

@Seldaek Seldaek added this to the 3.x milestone Oct 27, 2023
@Seldaek Seldaek merged commit 8813064 into Seldaek:main Oct 27, 2023
@Seldaek
Copy link
Owner

Seldaek commented Oct 27, 2023

Makes sense thanks

@lddtime
Copy link
Contributor Author

lddtime commented Oct 31, 2023

Thanks for merge! 😄

It would be nice to Cherry-Pick it to the 2.x

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

Successfully merging this pull request may close these issues.

2 participants