Make WordPress Core

Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#51315 closed task (blessed) (fixed)

Update mediaelement.js

Reported by: desrosj's profile desrosj Owned by: desrosj's profile desrosj
Milestone: 5.6 Priority: normal
Severity: normal Version:
Component: External Libraries Keywords: has-patch needs-testing commit
Focuses: Cc:

Description

Attachments (1)

51315.patch (570.4 KB) - added by mukesh27 4 years ago.
Initial patch.

Download all attachments as: .zip

Change History (10)

#1 @desrosj
4 years ago

  • Keywords needs-patch added

@mukesh27
4 years ago

Initial patch.

#2 @mukesh27
4 years ago

  • Keywords has-patch added; needs-patch removed

I have uploaded initial patch. @desrosj how to test if latest version is redacted or not?

#3 follow-up: @desrosj
4 years ago

how to test if latest version is redacted or not?

I'm not sure I understand the question. Can you rephrase?

For future reference:

#4 in reply to: ↑ 3 @SergeyBiryukov
4 years ago

Replying to desrosj:

how to test if latest version is redacted or not?

I'm not sure I understand the question. Can you rephrase?

It's probably about whether the version bundled with WP has any modifications to it, or is just a copy from upstream.

I think we include unmodified copy, but might be worth double-checking.

#5 @Hareesh Pillai
4 years ago

  • Keywords needs-testing added

Previously: #46681

As far as I have seen, it's a direct copy without any modifications. This patch needs testing across browsers.

#6 @desrosj
4 years ago

  • Keywords commit added
  • Owner set to desrosj
  • Status changed from new to assigned

Looking at the recent changes in the release notes, it seems that only minor bug fixes and typo/link URL corrections have been made. I'll give this a final review this week and get it committed so that it can be tested.

#7 @desrosj
4 years ago

I wanted to document the weirdness that I just noticed in case anyone stumbles upon the same thing in the future.

In [46436], MediaElementJS was updated to 4.2.13. However, mejs.version was not bumped prior to tagging the 4.2.13 release, so it's value remained at 4.2.12. When I linked to the comparison between versions above in comment:3, I examined mejs.version when choosing the version to compare 4.2.16 with.

This caused some inconsistencies in the diff comparisons and it took me a while to figure out where those inconsistencies were coming from.

A temporary shim was added in 4.2.13 along with a few other changes to fix some breaking bugs. These temporary fixes were then removed in 4.2.15. Because the shim was not present in 4.2.12 and Core was actually at 4.2.13, the original comparison posted above was missing the these deletions, but comparing trunk after updating to 4.2.16 was showing them.

#8 @desrosj
4 years ago

  • Resolution set to fixed
  • Status changed from assigned to closed

In 49070:

External Libraries: Update MediaElementJS to version 4.2.16.

This change brings a handful of bug fixes made since the previous update in [46436]. For a full list of changes included, see https://github.com/mediaelement/mediaelement/compare/4.2.13...4.2.16.

Props mukesh27, hareesh-pillai, SergeyBiryukov.
Fixes #51315.

#9 @SergeyBiryukov
4 years ago

In 49079:

Script Loader: Update versions for MediaElement.js and related files.

Follow-up to [49070].

See #51315.

Note: See TracTickets for help on using tickets.