WordPress.org
Get WordPress

Make WordPress Core

Opened 5 years ago

Closed 5 years ago

#53672 closed defect (bug) (fixed)

JS error thrown in Media Library when infinite scrolling re-enabled

Reported by: wildworks Owned by: joedolson
Priority: normal Milestone: 5.8
Component: Media Version: 5.8
Severity: normal Keywords: has-patch commit fixed-major dev-reviewed
Cc: Focuses: javascript

Description

this.updateLoadMoreView(); is called without checking whether infinite scrolling is enabled. Needs to be wrapped in a check for infiniteScrolling.

Reported by @wildworks on #50105; issue broken out for tracking.

Patch forthcoming.

Attachments (1)

53672.diff​ (509 bytes ) - added by joedolson 5 years ago.
Fix JS error.

Download all attachments as: .zip

Change History (6)

@joedolson
5 years ago

Fix JS error.

#1 @joedolson
5 years ago

  • Keywords has-patch commit added; needs-patch removed

#2 @joedolson
5 years ago

  • Owner set to joedolson
  • Resolution → fixed
  • Status new → closed

In 51440:

Media: Fix JS error in Media Library when infinite scroll enabled.

Wrap call to this.updateLoadMoreView() is wrapped in a check that infinite scroll is disabled to prevent JS errors.

Props wildworks.
Fixes #53672.

This ticket was mentioned in ​Slack in #core-media by desrosj. ​View the logs.
5 years ago

#4 @desrosj
5 years ago

  • Keywords fixed-major dev-reviewed added
  • Resolution fixed
  • Status closed → reopened

Reopening to backport to 5.8.

#5 @desrosj
5 years ago

  • Resolution → fixed
  • Status reopened → closed

In 51441:

Media: Fix JS error in Media Library when infinite scroll enabled.

Wrap call to this.updateLoadMoreView() is wrapped in a check that infinite scroll is disabled to prevent JS errors.

Props wildworks, joedolson.
Merges [51440] to the 5.8 branch.
Fixes #53672.

Note: See TracTickets for help on using tickets.