-
Notifications
You must be signed in to change notification settings - Fork 125
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
Uncaught ReferenceError: $gp_translations_options is not defined #476
Comments
It seems to be working alright on my local install (single site). It shouldn't need to be called via wp_enqueue_script as it's being written before the html headers are complete. It may be related to how we add the scripts in |
For some reasons I get this only when I'm logged out (single site). |
I can replicate @ocean90's behaviour. |
My issue with it while logged in was due to a old version of this plugin being network activated https://wpgeodirectory.com/translate/glotpress/projects/geodirectory/sv/default/ I can see the issue with logged out users too. |
Updated pull request, seems to fix for all, feedback welcome. |
This problem goes deeper than just wrapped in a function, it's caused by GP adding the translations page js file on every page instead of just the translation page. There are a few other js files loaded on pages where they aren't required as well. |
@nomaddevs can you try this code and make sure it resolves the issue you are seeing? |
@toolstack That code did not work for me, it does not display any JS errors but it breaks all the JS functionality of opening translations etc, it removed the localize script for me. |
@nomaddevs Give it a try now. |
@toolstack With what version of GP? |
I updated the PR, so just grab the code from the same spot. |
@nomaddevs try this code instead and see if it works for you. Note: If your using my "GP Single Click Edit" plugin you'll need to disable it to test with. |
@toolstack Yes your latest code seems to resolve all issues. |
Good to hear. |
Resolved with PR #500. |
$gp_translations_options is not being added to the page via wp_localize_script() from translations.php template. I am running multi-site so not sure if that is why its showing or if everyone else is seeing it also.
Basically it needs to be wrapped in a function and called via the wp_enqueue_scripts hook.
I will try submitting a patch if i can get my head around the process.
The text was updated successfully, but these errors were encountered: