WordPress.org

Make WordPress Core

Opened 8 months ago

Closed 3 weeks ago

Last modified 3 weeks ago

#53359 closed task (blessed) (fixed)

Coding Standards fixes for WP 5.9

Reported by: desrosj Owned by:
Milestone: 5.9 Priority: normal
Severity: normal Version:
Component: General Keywords: has-patch has-unit-tests
Focuses: Cc:

Description

Previously:

Change History (90)

@kapilpaul
7 months ago

Patch for PHPCS fixing in bundled themes.

#1 @kapilpaul
7 months ago

In 53359.diff mostly fixed the strict comparison. here is the GitHub PR

#2 @SergeyBiryukov
7 months ago

In 51285:

Coding Standards: Remove redundant type casting to array in WP_Query::get_posts().

This brings some consistency with other instances of using get_post_stati() in core.

get_post_stati() always returns an array, so the type casting is not needed.

Follow-up to [13172].

See #53359.

#3 @SergeyBiryukov
7 months ago

In 51381:

Coding Standards: Remove wrapping HTML tags from translatable strings.

This fixes a "Strings should not be wrapped in HTML" WPCS warning in help tabs for Export Personal Data and Erase Personal Data screens.

Follow-up to [50147].

See #53359.

#4 @SergeyBiryukov
7 months ago

In 51399:

Coding Standards: Rename the $ID variable to $user_id in wp_insert_user() and wp_update_user().

This fixes a "Variable $ID is not in valid snake_case format" WPCS warning.

Follow-up to [2872].

See #53359.

#5 @SergeyBiryukov
7 months ago

In 51400:

Bundled Themes: Correct comment formatting in inc/block-patterns.php.

Follow-up to [51045], [51103].

Props kapilpaul.
See #53359, #52627.

#6 @SergeyBiryukov
7 months ago

In 51401:

Twenty Fifteen: Use strict comparison in inc/custom-header.php.

Props kapilpaul.
See #53359.

#7 @desrosj
7 months ago

In 51406:

Bundled Themes: Correct comment formatting in inc/block-patterns.php.

Follow-up to [51045], [51103].

Props kapilpaul, SergeyBiryukov.
See #53359, #52627.

This ticket was mentioned in PR #1509 on WordPress/wordpress-develop by jrfnl.


6 months ago

  • Keywords has-patch has-unit-tests added

#9 @jrf
6 months ago

I've just uploaded 9 CS related patches to fix some low hanging fruit. To verify, one PR containing all these patches has been opened on GH and the build is passing: https://github.com/WordPress/wordpress-develop/pull/1509

#10 @prbot
6 months ago

hellofromtonya commented on PR #1509:

@jrfnl Is this PR complete and ready for code review?

#11 @prbot
6 months ago

jrfnl commented on PR #1509:

@hellofromtonya There will always be more fixes to make, but the fixes as pulled here are ready for commit as far as I'm concerned.

#12 @jrf
6 months ago

Added three more patches related to the ignore annotation review done for #53477. As the changes in ignore annotations are not related to the PHPCS update in #53477, I'm adding the patches here.

This ticket was mentioned in PR #1510 on WordPress/wordpress-develop by jrfnl.


6 months ago

Update the PHP_CodeSniffer dependency from version 3.5.5 to version 3.6.0.

I have also verified the ignore annotations, but no changes are needed there which are directly related to the PHPCS update. Changes which could be made have been added to #53359.

Trac ticket: https://core.trac.wordpress.org/ticket/53477

#14 @hellofromTonya
6 months ago

  • Keywords commit added

Marking submitted patches from [https://github.com/WordPress/wordpress-develop/pull/1509 PR 1509) ready for commit.

#15 @prbot
6 months ago

jrfnl commented on PR #1510:

Closing as merged via changeset https://core.trac.wordpress.org/changeset/51470

#16 @SergeyBiryukov
6 months ago

In 51526:

Coding Standards: Fix typo in the JS function name for handling the password reset button.

Follow-up to [50129].

See #53359.

#17 @SergeyBiryukov
6 months ago

In 51534:

Coding Standards: Correct DateTimeZone class name in WP_Customize_Date_Time_Control::get_timezone_info().

Follow-up to [41626].

See #53359.

#18 @peterwilsoncc
6 months ago

In 51548:

Coding Standards: Use strict comparisons in wp-admin/options-discussion.php.

See #53359.

#19 @peterwilsoncc
6 months ago

In 51549:

Coding Standards: Use strict comparisons in wp-admin/upload.php.

See #53359.

#20 @SergeyBiryukov
6 months ago

In 51550:

Coding Standards: Fix incorrect comment indent in safecss_filter_attr().

Props jrf.
See #53359.

#21 @SergeyBiryukov
6 months ago

In 51551:

Coding Standards: Fix incorrect alignment in two comment blocks.

Tabs should only be used for indentation and not for mid-line alignment.

Props jrf.
See #53359.

#22 @SergeyBiryukov
6 months ago

In 51552:

Coding Standards: Remove redundant semicolons after closing curly brackets.

Props jrf.
See #53359.

#23 @SergeyBiryukov
6 months ago

In 51553:

Bundled Themes: Remove redundant semicolons after closing curly brackets.

Includes a few minor indentation fixes.

Props jrf.
See #53359.

#24 @SergeyBiryukov
6 months ago

In 51554:

Twenty Thirteen: Remove wrapping HTML tag from translatable string.

This fixes a "Strings should not be wrapped in HTML" WPCS warning.

Props jrf.
See #53359.

#25 @SergeyBiryukov
6 months ago

In 51556:

Twenty Thirteen: Correct indentation in image.php template.

Props jrf.
See #53359.

#26 @SergeyBiryukov
6 months ago

In 51557:

Coding Standards: Silence a WPCS warning in date_i18n().

This fixes a "Calling current_time() with a $type of timestamp or U is strongly discouraged as it will not return a Unix (UTC) timestamp" warning.

Props jrf.
See #53359.

This ticket was mentioned in PR #1552 on WordPress/wordpress-develop by jrfnl.


6 months ago

If a closure doesn't use $this, it can be made static for improved performance.

Static closures are supported in PHP since PHP 5.4.

Trac ticket: https://core.trac.wordpress.org/ticket/53359

#28 @jrf
6 months ago

I've just opened PR 1552 with two more patches to fix all closures not using $this to be static (performance tweak).

#29 @prbot
6 months ago

jrfnl commented on PR #1509:

I've rebased the remaining commits in this PR to solve the merge conflicts and make the patches commitable again.

#30 @hellofromTonya
5 months ago

PR 1552 is ready for commit.

cc @SergeyBiryukov

#31 @hellofromTonya
5 months ago

PR 1509 is also ready for commit.

cc @SergeyBiryukov

#32 @hellofromTonya
5 months ago

In 51657:

Coding Standards: Use static closures when not using $this.

When a closure does not use $this, it can be made static for improved performance.

Static closures are supported in PHP since PHP 5.4. ​

Props jrf, hellofromTonya, swissspidy, SergeyBiryukov.
See #53359.

#34 @hellofromTonya
5 months ago

In 51658:

Coding Standards: Update the config for WordPress.PHP.NoSilencedErrors in PHPCS ruleset.

Follow-up to [45611].

Props jrf, hellofromTonya.
See #53359.

#35 @hellofromTonya
5 months ago

In 51659:

Coding Standards: Update the config for WordPress.PHP.FileName in PHPCS ruleset.

The WPCS sniff already takes the test classes in the tests/phpunit/includes directory into account.

Only the test cases outside of that directory, i.e. in the tests/phpunit/tests directory need to be listed here.

Includes alphabetizing the list.

Follow-up to [42346], [45607].

Props jrf, hellofromTonya, SergeyBiryukov.
See #53359.

#36 @SergeyBiryukov
5 months ago

Just noting that some of the patches here make changes to the wp-includes/blocks directory. However, these changes can be overwritten later with package updates from Gutenberg, so they would have to be made upstream in the Gutenberg repo first. The latest commits will skip those changes for now.

#37 @hellofromTonya
5 months ago

In 51660:

Coding Standards: Remove redundant ignore annotation in populate_options().

Follow-up to [43627], [43628].

Props jrf, hellofromTonya.
See #53359.

#38 @hellofromTonya
5 months ago

In 51661:

Coding Standards: Make ignore annotation more specific in WP_Site_Health_Auto_Updates::test_vcs_abspath().

Follow-up to [44986].

Props jrf, hellofromTonya.
See #53359.

#39 @hellofromTonya
5 months ago

In 51662:

Coding Standards: Fix indentation and remove ignore annotation in wp_privacy_send_personal_data_export_email().

Follow-up to [51129], [51410].

Props jrf, hellofromTonya.
See #53359.

#40 @desrosj
5 months ago

In 51693:

Coding Standards: Apply some minor alignment fixes.

These are updates caused by running composer format.

Follow up to [51501], [51599], [51618], [51653].
See #53359, #50542, #53238, #53668, #53690.

#41 @prbot
5 months ago

jrfnl commented on PR #1509:

Closing as fully committed via a variety of changesets.

#42 @SergeyBiryukov
4 months ago

In 51822:

Coding Standards: Use strict comparison in wp-inclues/class-wp-http-cookie.php.

Follow-up to [10512].

See #53359.

#43 @SergeyBiryukov
4 months ago

In 51823:

Coding Standards: Rename the $arrURL variable to $parsed_url in WP_Http::request().

This fixes a Variable "$arrURL" is not in valid snake_case format WPCS warning.

Follow-up to [10509], [45667].

See #53359.

#44 @SergeyBiryukov
4 months ago

In 51824:

Coding Standards: Rename the $arrURL variable to $parsed_url in WP_Http_Cookie::__construct().

This fixes a Variable "$arrURL" is not in valid snake_case format WPCS warning.

Follow-up to [10509], [25044], [45667], [51823].

See #53359.

#45 @SergeyBiryukov
4 months ago

In 51825:

Coding Standards: Rename the $arrURL variable to $parsed_url in WP_Http_Streams::request().

This fixes a Variable "$arrURL" is not in valid snake_case format WPCS warning.

Follow-up to [8620], [10509], [25044], [25224], [45667], [51823], [51824].

See #53359.

#46 @SergeyBiryukov
4 months ago

In 51826:

Coding Standards: Rename the $processedHeaders variable to $processed_headers in WP_Http::request().

This fixes a Variable "$processedHeaders" is not in valid snake_case format WPCS warning.

Follow-up to [8620], [51823].

See #53359.

This ticket was mentioned in Slack in #core by sergey. View the logs.


4 months ago

#48 @costdev
4 months ago

Posting related ticket for visibility: #54177

#49 @SergeyBiryukov
4 months ago

In 51877:

Coding Standards: Use strict comparison in wp-includes/class-http.php.

Follow-up to [10625], [10864], [13274], [29661], [33748], [47508], [47808].

See #53359.

#50 @SergeyBiryukov
4 months ago

In 51879:

Coding Standards: Use strict comparison in wp-includes/class-wp-http-ixr-client.php.

Follow-up to [16064], [16871], [17928].

See #53359.

#51 @hellofromTonya
4 months ago

In 51888:

Coding Standards: Use strict type check for in_array() in register_block_script_handle().

Follow-up to [51501].

Props aristath, sergeybiryukov.
Fixes #54206. See #53359.

#52 @johnbillion
3 months ago

  • Keywords commit removed

#53 @SergeyBiryukov
3 months ago

In 51929:

Coding Standards: Rename the $arrHeaders variable to $processed_headers in WP_Http_Streams::request().

This fixes a Variable "$arrHeaders" is not in valid snake_case format WPCS warning.

Follow-up to [8516], [51826].

See #53359.

#54 @SergeyBiryukov
3 months ago

In 51931:

Coding Standards: Rename $theHeaders variable to $processed_headers in WP_Http_Curl::request().

This fixes a Variable "$theHeaders" is not in valid snake_case format WPCS warning.

Follow-up to [8516], [8520], [51826], [51929].

See #53359.

#55 @SergeyBiryukov
3 months ago

In 51940:

Coding Standards: Rename the $process variable to $processed_response for clarity in WP_Http_Streams::request().

Includes minor code layout fixes for better readability.

Follow-up to [8516], [51826], [51929], [51931].

See #53359.

This ticket was mentioned in Slack in #core by sergey. View the logs.


3 months ago

#57 @SergeyBiryukov
3 months ago

In 51944:

Coding Standards: Correct alignment in wp_enqueue_global_styles().

This fixes an Equals sign not aligned with surrounding assignments; expected 5 spaces but found 1 space WPCS warning.

Follow-up to [50973], [51819].

See #53359.

#58 @SergeyBiryukov
3 months ago

In 51951:

Coding Standards: Correct alignment in WP_Site_Health::get_test_update_temp_backup_writable().

This fixes an Equals sign not aligned with surrounding assignments; expected 1 space but found 6 spaces WPCS warning.

Follow-up to [51815].

See #51857, #53359.

This ticket was mentioned in Slack in #core by sergey. View the logs.


3 months ago

#60 @SergeyBiryukov
3 months ago

In 52025:

Coding Standards: Rename the $strResponse argument to $str_response in WP_Http::processResponse().

This fixes a Variable "$strResponse" is not in valid snake_case format WPCS warning.

Additionally, rename a $res variable to $response for clarity.

Follow-up to [8516], [51823], [51826], [51877].

See #53359.

#61 @SergeyBiryukov
3 months ago

In 52026:

Coding Standards: Move wp-includes/class-http.php to wp-includes/class-wp-http.php.

This renames the file containing the WP_Http class to conform to the coding standards.

This commit also includes:

  • A new class-http.php that includes the new file, for anyone that may've been including the file directly.
  • Replaces references to the old filename with the new filename.

Follow-up to [8516], [13274], [33748].

Fixes #54389. See #53359.

#62 @SergeyBiryukov
2 months ago

In 52150:

Coding Standards: Rename the $gzData argument to $gz_data in WP_Http_Encoding::compatible_gzinflate().

This fixes a Variable "$gzData" is not in valid snake_case format WPCS warning.

Follow-up to [11684].

See #53359.

#63 @SergeyBiryukov
2 months ago

In 52228:

Coding Standards: Wrap some long lines in js/_enqueues/admin/post.js per the JS coding standards for better readability.

See #53359.

#64 @SergeyBiryukov
8 weeks ago

In 52278:

I18N: Remove redundant default text domain parameter in some __() calls.

Follow-up to [51160], [52069].

See #53359.

#66 @audrasjb
7 weeks ago

In 52345:

Coding Standards: Correct alignment in blocks.php and class-wp-rest-templates-controller.

See #53359.

#68 @SergeyBiryukov
7 weeks ago

In 52346:

Coding Standards: Use camel case with a lowercase first letter for the blockTheme array key.

This matches the WordPress JS naming conventions and the naming of other keys in the $prepared_themes array.

Follow-up to [52341].

See #54578, #53359.

#69 @SergeyBiryukov
6 weeks ago

In 52348:

Coding Standards: Simplify or wrap some long conditions in wp-includes/theme.php for better readability.

Includes adjusting some DocBlocks per the documentation standards.

Follow-up to [35595], [41937], [52256].

See #53359, #53399.

#70 @SergeyBiryukov
6 weeks ago

In 52350:

Coding Standards: Remove a trailing comma from a compact() call in _wp_customize_include().

This fixes a Trailing comma's are not allowed in function calls in PHP 7.2 or earlier error in the PHP compatibility check.

Follow-up to [52348].

See #53359.

#73 @audrasjb
6 weeks ago

In 52354:

Docs: Docblock adjustments in some 5.9 block related functions.

Adds missing @since mention to wp_enqueue_block_style() and _wp_multiple_block_styles(), and a few other minor changes.

Follow-up to [52069].

See #53359.

#75 @SergeyBiryukov
3 weeks ago

In 52428:

Coding Standards: Use strict comparison in wp-admin/options-reading.php.

Follow-up to [16816].

See #53359.

#76 @hellofromTonya
3 weeks ago

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

With 5.9 RC1 tomorrow and since coding standards updates usually touch code, I'm going to close this out.

If there are any more changes required between now and the final 5.9 release, this can be referenced and reopened if necessary with dev-reviewed.

#54728 is open for continuation during the 6.0 cycle.

#77 @SergeyBiryukov
3 weeks ago

In 52431:

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

Follow-up to [21852], [47874], [51075].

See #53359.

Note: See TracTickets for help on using tickets.