We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
In the Log component, addWriteToFile method, as in: https://github.com/piwik/piwik/blob/master/core/Log.php#L56
Log
addWriteToFile
the set formatter is $this->screenFormatter.
$this->screenFormatter
Shouldn't that be $this->fileFormatter instead?
$this->fileFormatter
The text was updated successfully, but these errors were encountered:
it probably should indeed. Have you used it or just noticed it? we'll investigate
Sorry, something went wrong.
We've used that during some debug session. After the switch, we saw an output change from:
CACHE_CONTROL: max-age=0\r\nCONNECTION: keep-alive\r\n<br/>
to:
CACHE_CONTROL: max-age=0 CONNECTION: keep-alive
So the <br/> disappeared, but the \r\n were not replaced with actual carriage returns.
<br/>
\r\n
In 05de97e: Fixes #3690 Thanks for the report & patch!
mattab
No branches or pull requests
In the
Log
component,addWriteToFile
method, as in:https://github.com/piwik/piwik/blob/master/core/Log.php#L56
the set formatter is
$this->screenFormatter
.Shouldn't that be
$this->fileFormatter
instead?The text was updated successfully, but these errors were encountered: