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

Using date_default_timezone_set() and similar isn't allowed. #187

Open
matt-bernhardt opened this issue Jan 23, 2017 · 0 comments
Open

Using date_default_timezone_set() and similar isn't allowed. #187

matt-bernhardt opened this issue Jan 23, 2017 · 0 comments

Comments

@matt-bernhardt
Copy link
Member

matt-bernhardt commented Jan 23, 2017

Recent runs of PHPCS on Travis (including on #170) have returned the following:

FILE: ...ordpress/src/wp-content/themes/MITlibraries-parent/lib/hours.php
----------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
----------------------------------------------------------------------
 27 | ERROR | Using date_default_timezone_set() and similar isn't
    |       | allowed, instead use WP internal timezone support.
    |       | (WordPress.VIP.TimezoneChange.timezone_change)
----------------------------------------------------------------------

Fixing this will involve setting the default timezone at the server level, so it may take a bit of coordination. There may also be a better way of declaring such a timezone as the default - will research.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant