WordPress.org

Make WordPress Core

Opened 2 years ago

Last modified 5 months ago

#44250 new enhancement

Error silencer @opendir syntax are inconsistently written in different files

Reported by: subrataemfluence Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version:
Component: General Keywords: has-patch 2nd-opinion
Focuses: coding-standards Cc:

Description

We have error silencers like @opendir and @closedir are used in several places of different admin files.

In some areas they appear like @opendir while at some places we have @ opendir. However, @closedir is written consistently.

I think they should be consistent everywhere.

Attachments (3)

error-silencer-plugin.php.diff (1.5 KB) - added by subrataemfluence 2 years ago.
error-silencer-upgrade.php.diff (990 bytes) - added by subrataemfluence 2 years ago.
44250.diff (739 bytes) - added by sabernhardt 5 months ago.
for consistency, this removes the spaces for the remaining two @ opendir instances in wp-admin/includes/plugin.php

Download all attachments as: .zip

Change History (5)

#1 @johnbillion
2 years ago

  • Focuses coding-standards added
  • Type changed from defect (bug) to enhancement
  • Version trunk deleted

#2 @sabernhardt
5 months ago

Changeset [45611] removed the two spaces from wp-admin/includes/upgrade.php plus two of the four in wp-admin/includes/plugin.php. So two remain.

@sabernhardt
5 months ago

for consistency, this removes the spaces for the remaining two @ opendir instances in wp-admin/includes/plugin.php

Note: See TracTickets for help on using tickets.