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.

Sign up for GitHub

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

Jump to bottom

cookie vs. wp-cron.php #779

Closed
thabbs opened this issue Jan 27, 2020 · 4 comments
Closed

cookie vs. wp-cron.php #779

thabbs opened this issue Jan 27, 2020 · 4 comments
Labels
core Core functionalities, including the admin section

Comments

@thabbs
Copy link

thabbs commented Jan 27, 2020 •
edited
Loading

It looks like that qtranslate-xt tries to set the language cookie even when invoked by wp-cron.php. If there is no reason to do so it shouldn't try.

This warning is logged every time wp-cron is running:

Cannot modify header information - headers already sent in
[...]/wp-content/plugins/qtranslate-xt/qtranslate_core.php on line 463
@herrvigg
Copy link
Collaborator

herrvigg commented Feb 7, 2020 •
edited
Loading

The qtranxf_can_redirect is checking wp_doing_cron so i don't know, need more investigation. Check in debug mode, i also added some asserts that can help to understand.

(PS: forget my first comment about the line and version, i was checking the gutenberg branch)

@herrvigg herrvigg added the need info The submitter must provide more info label Feb 7, 2020
@herrvigg herrvigg added the core Core functionalities, including the admin section label Feb 15, 2020
@herrvigg
Copy link
Collaborator

@thabbs I just updated Wiki Troubbleshooting. Maybe you could enable the WP_DEBUG_LOG to better see what's going on. I can't find the problem without any information from your side.

@herrvigg
Copy link
Collaborator

herrvigg commented May 3, 2020

@thabbs i can't reproduce this, so i close it for now. Feel free to re-open when you can provide more information.

@herrvigg herrvigg closed this as completed May 3, 2020
@herrvigg herrvigg mentioned this issue Feb 23, 2022
Merged
@herrvigg
Copy link
Collaborator

@thabbs This should be solved by #1114, just merged in master.

@herrvigg herrvigg removed the need info The submitter must provide more info label Feb 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
core Core functionalities, including the admin section
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@thabbs @herrvigg