Make WordPress Core

Opened 12 months ago

Closed 12 months ago

Last modified 12 months ago

#60415 closed defect (bug) (fixed)

Coding Standards: Need to improve below two files comparison operator conditions.

Reported by: upadalavipul's profile upadalavipul Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: 6.5 Priority: normal
Severity: normal Version:
Component: General Keywords:
Focuses: coding-standards Cc:

Description

I have reviewed the code and found the two files in condition check only != operator. but as per the coding standard needs to be checked !== operator. Please check below to provide two file codes.

Files:

  1. wp-admin/plugin-editor.php
  2. wp-admin/theme-editor.php

Attachments (11)

60415.patch (2.4 KB) - added by upadalavipul 12 months ago.
added patch
60415.3.patch (1.6 KB) - added by viralsampat 12 months ago.
60415.2.patch (3.1 KB) - added by viralsampat 12 months ago.
I have checked above mentioned issue and founds few more files. I have applied its patch.
60415.4.patch (1.2 KB) - added by upadalavipul 12 months ago.
Added new patch
60415.5.patch (4.6 KB) - added by upadalavipul 12 months ago.
Added one more patch
60415.6.patch (3.0 KB) - added by upadalavipul 12 months ago.
added new patch
60415.7.patch (944 bytes) - added by upadalavipul 12 months ago.
Added new patch
60415.8.patch (944 bytes) - added by upadalavipul 12 months ago.
Added new patch
60415.9.patch (2.8 KB) - added by viralsampat 12 months ago.
I have added another patch
60415.10.patch (13.0 KB) - added by viralsampat 11 months ago.
I have checked above mentioned issue and founds few files. Here, I have added its patch.
60415.11.patch (11.4 KB) - added by viralsampat 11 months ago.
I have checked above mentioned issue and founds few files. Here, I have added its patch.

Download all attachments as: .zip

Change History (15)

@upadalavipul
12 months ago

added patch

#1 @SergeyBiryukov
12 months ago

  • Milestone changed from Awaiting Review to 6.5

#2 @SergeyBiryukov
12 months ago

In 57522:

Coding Standards: Use strict comparison for functions lookup in plugin/theme editors.

Follow-up to [10607], [44617].

Props upadalavipul.
See #60415.

#3 @SergeyBiryukov
12 months ago

  • Owner set to SergeyBiryukov
  • Resolution set to fixed
  • Status changed from new to closed

In 57529:

Coding Standards: Use strict comparison in wp-admin/update-core.php.

Follow-up to [11273], [25784], [54654].

Props wpfy, mukesh27, azaozz, viralsampat.
Fixes #58061, #60415.

@viralsampat
12 months ago

I have checked above mentioned issue and founds few more files. I have applied its patch.

@upadalavipul
12 months ago

Added new patch

@upadalavipul
12 months ago

Added one more patch

@upadalavipul
12 months ago

added new patch

#4 @SergeyBiryukov
12 months ago

Thanks for the patches! It looks like these are all external libraries though, they don't have to follow the WordPress coding standards, and should not be changed directly in core.

@upadalavipul
12 months ago

Added new patch

@upadalavipul
12 months ago

Added new patch

@viralsampat
12 months ago

I have added another patch

@viralsampat
11 months ago

I have checked above mentioned issue and founds few files. Here, I have added its patch.

@viralsampat
11 months ago

I have checked above mentioned issue and founds few files. Here, I have added its patch.

Note: See TracTickets for help on using tickets.