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
dev/core#4410: fixed timezone support for standalone #28468
Conversation
🤖 Thank you for contributing to CiviCRM! ❤️ We will need to test and review this PR. 👷 Introduction for new contributors...
Quick links for reviewers...
|
The issue associated with the Pull Request can be viewed at https://lab.civicrm.org/dev/core/-/issues/4410 |
@jaapjansma I can see that the timezone is stored against the user. However I changed my user's timezone and the dates did not change, e.g. Created Date at the bottom of the contact summary screen; or searchkit timestamps on the user table: neither of these change when changing timezones. Is this correct? |
I have this same issue in a Drupal 10 site where I can change my timezone. Looks like a CiviCRM core issue not handling the timezones correctly in displaying dates. |
I think this should be merged; it seems to work as it should, and the fact that core doesn't is not something we can fix in this pr. @mlutfy if you agree can you merge? |
It seems like loadBootstrap is not called in standalone? So then setMySQLTimeZone() doesn't get called. If I put |
Yeah, I think the loadBootstrap is called for cli only, but I don't know if that's correct.
Sounds like we should be calling setMySQLTimeZone() though.
Hmm. I think this is one of those things where once it's fixed things may get worse...! |
Ok let's merge this and figure out where to put the call to setMySQLTimeZone after. Since obviously the form part is working. test fail is the usual ongoing fail. |
Related: #29282 |
Overview
This adds timezone support the user of stand alone CiviCRM.
See https://lab.civicrm.org/dev/core/-/issues/4410
@artfulrobot can you review this?