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

Set PHP default timezone during processors processing #179

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

agileware-pengyi
Copy link
Contributor

WordPress seems to expect the PHP default timezone to be UTC. This will cause Civi to record the wrong date-time with the CFC processor. Here is a way to fix it. #167 #169

Before

Civi records the wrong date-time

After

Civi records the right date-time

Technical Details

May cause wrong date-time record in non CFC processors if them depends on WP date-time system. I didn't see any of them but just mentioned here. So there may be a better solution than this.

Comments

This is the current solution we are using.

Agileware ref: CFC-61

@agileware-fj
Copy link
Contributor

We found that this was causing double offset issues where the time had been explicitly added to an activity, so have adjusted so that the timezone is set during the CiviCRM API calls themselves.

@masetto
Copy link

masetto commented Feb 3, 2021

Hi, any news about this PR? Probably it solves time zone discrepancy.

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

Successfully merging this pull request may close these issues.

4 participants