WordPress.org

Make WordPress Core

Opened 3 months ago

Last modified 2 months ago

#53856 new defect (bug)

Check if the 'mirroring' property exists in media-views.js

Reported by: MMDeveloper Owned by:
Milestone: 5.9 Priority: normal
Severity: normal Version:
Component: Media Keywords: has-patch
Focuses: javascript, administration Cc:

Description

/wp-includes/js/media-views.js

<?php
this.collection.mirroring.args.s 

on line 2121 should be

<?php
                if ( 
    this.collection.hasOwnProperty('mirroring') &&
    this.collection.mirroring.args.s
){

Similar issue
/wp-includes/js/media-views.min.js

Change History (7)

#1 @mukesh27
3 months ago

#53857 was marked as a duplicate.

This ticket was mentioned in PR #1537 on WordPress/wordpress-develop by mukeshpanchal27.


3 months ago

  • Keywords has-patch added

#3 @mukesh27
3 months ago

  • Component changed from General to Media
  • Focuses coding-standards removed
  • Version 5.8 deleted

Hi there!

@MMDeveloper thanks for the ticket.

media-views.js and media-views.min.js is automated build files.

PR added as per your suggestion.

#4 @SergeyBiryukov
3 months ago

  • Summary changed from Small Bug to Check if the 'mirroring' property exists in media-views.js

#5 @johnbillion
3 months ago

  • Keywords reporter-feedback added

@mukesh27 @MMDeveloper How can this bug be reproduced please?

#6 @MMDeveloper
3 months ago

I think its to do with legacy software. I'm on WordPress 5.8 but the theme has not been updated in many years. So you may not necessarily care. On WordPress Visual editor, I click edit gallery and when I see WordPress Media Gallery, it produces this error in the Google Developer Tools console.

I think it could be a legacy issue, but considering its just checking if a variable exists or not, I didn't think it was a big deal for you guys to implement in next build.

#7 @SergeyBiryukov
2 months ago

  • Keywords reporter-feedback removed
  • Milestone changed from Awaiting Review to 5.9
Note: See TracTickets for help on using tickets.