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

Invalidating reports may fail because of log delete settings #4568

Closed
mattab opened this issue Jan 23, 2014 · 1 comment
Closed

Invalidating reports may fail because of log delete settings #4568

mattab opened this issue Jan 23, 2014 · 1 comment
Assignees
Labels
Bug For errors / faults / flaws / inconsistencies etc.
Milestone

Comments

@mattab
Copy link
Member

mattab commented Jan 23, 2014

How to reproduce original error:

  • Enable log deleting in config.ini.php with small interval (i.e. 1 day)
  • override those options via UI with bigger interval (i.e. 90 days)
  • try to use invalidateArchivedReports() method with date between 1 and 90 days back from today - it will cause warning and reports for given date will not be invalidated even though they actually could be.

see #206

@mattab
Copy link
Member Author

mattab commented Jan 23, 2014

In 2d41d1e: Merge pull request #206 from clearcode/addedMissingOptionCheckToInvalidateArchivedReports

Switched from reading raw config values to reusing the helper method, fixes #4568

@mattab mattab added this to the 2.1 - Piwik 2.1 milestone Jul 8, 2014
@mattab mattab self-assigned this Jul 8, 2014
sabl0r pushed a commit to sabl0r/piwik that referenced this issue Sep 23, 2014
…eckToInvalidateArchivedReports

Switched from reading raw config values to reusing the helper method, fixes matomo-org#4568
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug For errors / faults / flaws / inconsistencies etc.
Projects
None yet
Development

No branches or pull requests

1 participant