Make WordPress Core

Opened 7 years ago

Closed 5 years ago

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

wp-embed.min.js contains extra code

Reported by: szepeviktor's profile szepe.viktor Owned by:
Milestone: Priority: normal
Severity: trivial Version: 4.5
Component: Embeds Keywords: close
Focuses: Cc:



"else" is not in wp-embed.js!

Could it be an Uglify2JS issue?

Change History (5)

#1 @szepe.viktor
7 years ago

Props markoheijnen

#2 @dd32
7 years ago

This isn't a regression in the uglify process in 4.7, although does seem like extra code at first glance.

The else; has been there since which seems to stem from the conditionals: false in r35708 which is required to avoid the && issue in #34698.

#3 @swissspidy
7 years ago

  • Keywords close added
  • Severity changed from normal to trivial
  • Version changed from 4.6.1 to 4.5

This should indeed be reported upstream as it's specific to the conditionals: false setting in UglifyJS2. I get the same errant else in their online demo. Perhaps there's even a valid reason for it that we just don't know about.

#5 @swissspidy
5 years ago

  • Milestone Awaiting Review deleted
  • Resolution set to reported-upstream
  • Status changed from new to closed

This has been fixed upstream in early 2017:

Note: See TracTickets for help on using tickets.