WordPress.org

Make WordPress Core

Opened 3 months ago

Closed 2 months ago

#48968 closed defect (bug) (reported-upstream)

hoverintent-js.min.js conflicted amp

Reported by: patkhonsiam Owned by: audrasjb
Milestone: Priority: normal
Severity: normal Version: 5.3.1
Component: Toolbar Keywords: has-screenshots
Focuses: Cc:

Description

Hello,

My web used plugin AMP (standard mode), and i have updated wordpress to Version 5.3.1
and found Invalid script

/wp-includes/js/hoverintent-js.min.js

old version of hoverintent-js.min.js no problem

Thank you

Attachments (2)

wp-bug-01.jpg (496.6 KB) - added by patkhonsiam 2 months ago.
amperror.png (394.9 KB) - added by ehtis 2 months ago.
amp-error-local-install

Download all attachments as: .zip

Change History (12)

#1 @SergeyBiryukov
3 months ago

  • Component changed from General to Toolbar
  • Keywords reporter-feedback added
  • Milestone changed from Awaiting Review to 5.3.2
  • Version set to 5.3.1

Hi there, welcome to WordPress Trac! Thanks for the report.

Is there an error message referring to the script? What does it say?

Introduced in [46872] for #47069.

#2 @azaozz
2 months ago

@patkhonsiam could you please provide some more info:

  • The file /wp-includes/js/hoverintent-js.min.js is new in WP 5.3.1, there's no old version of it. Perhaps you mean /wp-includes/js/hoverintent.min.js (no -js before .min) which is the jQuery version of hoverIntent.
  • Where exactly are you seeing the error? Assuming it's on the front-end when you're logged in and using AMP. Do you see the same error in other places or cases?
  • Could you paste the exact error message from the browser console here please (you can replace sensitive information with dots).

#3 @patkhonsiam
2 months ago

Hello,

I have tried to install version 5.3.1 and installed the AMP plug-in. After that, it was activated and used in standard mod. And the plug-in display the following errror message:

Please see image
https://core.trac.wordpress.org/attachment/ticket/48968/wp-bug-01.jpg

Thank you

#4 @audrasjb
2 months ago

Hi @patkhonsiam

Are you using https://wordpress.org/plugins/amp/?
I can't reproduce the issue on my side.

Last edited 2 months ago by audrasjb (previous) (diff)

#5 @Murrejb
2 months ago

Hi,

I have the exact same issue.

I'm using https://wordpress.org/plugins/amp/

I use the Astra Theme 2.1.4 By Brainstorm Force.

Thank you.

This ticket was mentioned in Slack in #core by audrasjb. View the logs.


2 months ago

#7 @audrasjb
2 months ago

  • Keywords has-screenshots needs-patch added; reporter-feedback removed
  • Owner set to audrasjb
  • Status changed from new to reviewing

I'm able to reproduce the issue with AMP plugin Standard option enabled.

@ehtis
2 months ago

amp-error-local-install

#8 @ehtis
2 months ago

I'm able to reproduce this on a local install.

Installed AMP, switched to Standard mode, then visited Validated URLs.

Switched to Transitional mode and it was still showing.

Checked the attached screenshot:
amp-error-local-install

Last edited 2 months ago by ehtis (previous) (diff)

#9 @karl94
2 months ago

This looks like a bug of the plugin. They are already working to fix it:
https://github.com/ampproject/amp-wp/issues/3927

#10 @audrasjb
2 months ago

  • Keywords needs-patch removed
  • Milestone 5.3.2 deleted
  • Resolution set to reported-upstream
  • Status changed from reviewing to closed

Closed as reported upstream.
Thank you all for reporting this ticket :-)

Note: See TracTickets for help on using tickets.