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

Jump to bottom

failed fopen() generates E_WARNING #2449

Closed
robocoder opened this issue May 24, 2011 · 1 comment
Closed

failed fopen() generates E_WARNING #2449

robocoder opened this issue May 24, 2011 · 1 comment
Labels
Task Indicates an issue is neither a feature nor a bug and it's purely a "technical" change.
Milestone
1.5

Comments

@robocoder
Copy link
Contributor

see php.net/fopen

Just go through the codebase and make sure we prefix with @ and check the return code for false.
@mattab
Copy link
Member

mattab commented May 30, 2011

(In [4838]) Fixes #2449 - removed all "user facing" fopen (left the one that could be useful such as during auto update & translation file writing)

@robocoder robocoder added this to the 1.5 - Piwik 1.5 milestone Jul 8, 2014
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Task Indicates an issue is neither a feature nor a bug and it's purely a "technical" change.
Projects
None yet
Milestone
1.5
Development

No branches or pull requests
2 participants
@mattab @robocoder