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

wp_register_script causes a PHP Notice #361

Closed
anekola opened this issue Jun 6, 2020 · 1 comment · Fixed by #368
Closed

wp_register_script causes a PHP Notice #361

anekola opened this issue Jun 6, 2020 · 1 comment · Fixed by #368
Labels
FIDO U2F
Milestone
0.7.0

Comments

@anekola
Copy link

anekola commented Jun 6, 2020

I have no idea why I just started getting this, but I just updated this VIP Go repo in my local environment and I suddenly started getting two PHP notices. Turns out the notice came from these functions:

two-factor/providers/class.two-factor-fido-u2f.php

Line 70 in 529c973

wp_register_script(

Notice: wp_register_script was called incorrectly. Scripts and styles should not be registered or enqueued until the wp_enqueue_scripts, admin_enqueue_scripts, or login_enqueue_scripts hooks. Please see Debugging in WordPress for more information. (This message was added in version 3.3.0.) in /srv/www/ona-journalists/public_html/wp-includes/functions.php on line 5167

@PypWalters
Copy link

Seeing the same issue as well.

@kasparsd kasparsd mentioned this issue Jun 26, 2020
Closed
kasparsd added a commit that referenced this issue Jun 26, 2020
@kasparsd
Register scripts during *_enqueue_scripts hook instead
a97c32b 
Fixes #361
@kasparsd kasparsd mentioned this issue Jun 26, 2020
Merged
kasparsd added a commit that referenced this issue Jun 26, 2020
@kasparsd
Register scripts during *_enqueue_scripts hook instead (#368)
536678d 
Fixes #361
@kasparsd kasparsd mentioned this issue Jul 6, 2020
Closed
@jeffpaul jeffpaul added this to the 0.7.0 milestone Jan 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
FIDO U2F
Projects
None yet
Milestone
0.7.0
Development

Successfully merging a pull request may close this issue.

Register scripts during *_enqueue_scripts hook instead WordPress/two-factor
4 participants
@kasparsd @anekola @jeffpaul @PypWalters