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

Warning: fread(): Length parameter must be greater than 0 in plugins/ScheduledReports/API.php on line 519 #6454

Closed
mattab opened this issue Oct 16, 2014 · 4 comments
Closed

Warning: fread(): Length parameter must be greater than 0 in plugins/ScheduledReports/API.php on line 519 #6454

mattab opened this issue Oct 16, 2014 · 4 comments
Labels
Bug For errors / faults / flaws / inconsistencies etc. Major Indicates the severity or impact or benefit of an issue is much higher than normal but not critical.
Milestone
2.8.1

Comments

@mattab
Copy link
Member

mattab commented Oct 16, 2014 •
edited

This bug was triggered
Warning: fread(): Length parameter must be greater than 0 in plugins/ScheduledReports/API.php on line 519

This bug was reproduced on Piwik 2.6.1 and maybe already fixed in 2.8.0 eg. #5300 not sure.
@mattab mattab added Bug For errors / faults / flaws / inconsistencies etc. Major Indicates the severity or impact or benefit of an issue is much higher than normal but not critical. labels Oct 16, 2014
@mattab mattab added this to the Piwik 2.9.0 milestone Oct 16, 2014
@diosmosis
Copy link
Member

@mattab I looked at the code, do you know if there's a reason file_get_contents isn't used?

@mattab
Copy link
Member Author

mattab commented Oct 17, 2014

@diosmosis I don't think there is a reason, file_get_contents should do the trick
cc @julienmoumne

@julienmoumne
Copy link
Member

no reason, but if size is indeed equal to zero, this won't solve the issue

@mattab mattab modified the milestones: Piwik 2.9.0, Piwik 2.8.1 Oct 19, 2014
diosmosis pushed a commit that referenced this issue Oct 19, 2014
Refs #6454, use file_put_contents and file_get_contents for scheduled…
d369e1b 
… reports generation and hopefully get more information if file size = 0 error reoccurs.
@mattab
Copy link
Member Author

mattab commented Oct 20, 2014

I'm marking as closed though it's not fully closed, but next time the error message will change at least, so we shall re-open a new issue.

@mattab mattab closed this as completed Oct 20, 2014
@mattab mattab mentioned this issue Oct 27, 2014
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug For errors / faults / flaws / inconsistencies etc. Major Indicates the severity or impact or benefit of an issue is much higher than normal but not critical.
Projects
None yet
Milestone
2.8.1
Development

No branches or pull requests
3 participants
@diosmosis @mattab @julienmoumne