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

The first day piwik has been installed, date should default to "today" instead of yesterday #5594

Closed
mattab opened this issue Apr 6, 2008 · 6 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

Comments

@mattab
Copy link
Member

mattab commented Apr 6, 2008

… so that the users can see their today stats.

we’ve received many emails from users who didn’t notice that yesterday was selected by default (the UI is quite bad at this level btw)

@anonymous-matomo-user
Copy link

Attachment: Possible Solution Patch
[patch_174.txt](http://issues.piwik.org/attachments/5594/patch_174.txt)

@anonymous-matomo-user
Copy link

I added a section installation and an value installation_date to the config.ini.php file and before redirecting to the index page I added an check if today equal installation date. If this is the case, I override die default_day value.

@mattab
Copy link
Member Author

mattab commented Apr 25, 2008

Christian, it doesn’t work, because if you add a new website it has to do the same for this new website
you must compare the value against Site→getCreationDate()

@anonymous-matomo-user
Copy link

Attachment: Patch V.1.1
[patch_174_1.txt](http://issues.piwik.org/attachments/5594/patch_174_1.txt)

@anonymous-matomo-user
Copy link

Sorry, I wasn’t aware of this aspect. In the current patch I added da function isToday to the Piwik_Date Class. With is function I check if Site→getCreationDate() is today

@mattab
Copy link
Member Author

mattab commented May 29, 2008

(In 496) – fix #5594: The first day piwik has been installed, date should default to “today” instead of yesterday.
Thanks Christian for the patch

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. Major Indicates the severity or impact or benefit of an issue is much higher than normal but not critical.
Projects
None yet
Development

No branches or pull requests

2 participants