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
Added LogViewer as submodule + format a multi line log message into multiple log messages for file handler #8749
Conversation
FYI: Will try to fix tests tmrw. Not sure why they fail but work in the LogViewer plugin itself |
Since this will close the issue, I will try to fix the build. |
The following changes will make LogTest.php pass:
& remove The LogViewer tests seem to run against the 3.0 branch, so didn't go into those. |
That's not intended. Think I will remove the .travis.yml from LogViewer plugin once it is merged as a submodule as the tests should run with core already anyway. Does that make sense? |
I wasn't referring to travis. I tried to fix the LogViewer tests and noticed the API output for the system tests includes errors for the 3.0 branch. Did you run the tests initially against 3.0? I wasn't sure, so I didn't look further. |
I get what you mean, yes that's expected. I created a sample log file that I configured to be used in tests via DI. The log file is simply a few lines extracted of my own log file. Will have a look at the tests |
1adbe66
to
cb26480
Compare
Tests are fixed. Failing UI tests are expected in http://builds-artifacts.piwik.org/piwik/piwik/7239/15524/ |
Wanted to merge this PR but noticed the
|
Made a change that should fix this one. Will merge as soon as it passes |
Added LogViewer as submodule + format a multi line log message into multiple log messages for file handler
fixes #7239
Re formatting see #7239 (comment)
It's not a good idea to log multi-line messages instead we log one message for each line.