Let's do some `phpcbf` cleanup...

[whyisjake]

There are a couple of regressions to cleanup that have been flagged by the linter tools.

[whyisjake]

In 46312:

General: Linter cleanup

phpcbf was able to clean up a few files. Tests were breaking as a result of code formatting.

Fixes #48142
Props whyisjake

[whyisjake]

In 46313:

General: Further phpcs cleanup.

In [46312] we attempted to do some phpcs cleanup. This commit cleans up three issues that were introduced in [46309].

Fixes #48142.
Props whyisjake, garrett-eclipse.

[whyisjake]

In 46314:

General: Further phpcs cleanup.

In [46312] and [46313] we attempted to do some phpcs cleanup. This commit cleans up three issues that were introduced in [46309].

Fixes #48142.

Props: whyisjake, garrett-eclipse, desrosj

[SergeyBiryukov]

In 46316:

General: Correct strict comparison in WP_Links_List_Table::column_categories() introduced in [46313].

$cat_id is a string, $category is an integer.

See #48142.

[SergeyBiryukov]

In 46317:

Coding Standards: Remove inline assignment in test_wp_delete_term_should_invalidate_cache().

See #48142.

[garrett-eclipse]

CS improvement for network.php

[garrett-eclipse]

Thanks for correcting the CS in getTerms.php @SergeyBiryukov
I attached CS-network.php.diff to correct the CS on /multisite/network.php
*No need for 27 spaces ;)

[jrf]

@garrett-eclipse pinged me to have a look at the patches here.

Re: [46316] - this may lead to difficult to debug issues unless you are 100% sure both sides of the comparison will always be numeric. Effectively, the change makes no difference to the code and it is still a loose type comparison, as the (int) cast is what PHP would do internally when using !=.

So if a true strict type comparison is intended here, a string-based strict comparison should be used:

<?php
if ( $cat_id !== (string) $category ) {

Also see: ​https://phpcheatsheets.com/compare/not_equal/

Re: CS-network.php.diff: comments should use proper punctuation, i.e. start with a capital letter and end with a punctuation character, so this inline comment would not pass once the docs checks get activated.

<?php
activate_plugin( $path, '', true );  // $network_wide = true

[garrett-eclipse]

Updated network.php patch to address comment convention (Capital first letter, punctuation at end) Thanks @jrf

[garrett-eclipse]

Thanks for the feedback @jrf

Replying to jrf:

@garrett-eclipse pinged me to have a look at the patches here.

Re: [46316] - this may lead to difficult to debug issues unless you are 100% sure both sides of the comparison will always be numeric. Effectively, the change makes no difference to the code and it is still a loose type comparison, as the (int) cast is what PHP would do internally when using !=.

So if a true strict type comparison is intended here, a string-based strict comparison should be used:

<?php
if ( $cat_id !== (string) $category ) {

Also see: ​https://phpcheatsheets.com/compare/not_equal/

I'm going to leave this comparison check to @SergeyBiryukov for review.

Re: CS-network.php.diff: comments should use proper punctuation, i.e. start with a capital letter and end with a punctuation character, so this inline comment would not pass once the docs checks get activated.

<?php
activate_plugin( $path, '', true );  // $network_wide = true

I appreciate the feedback, as I copied that comment from previous in the file I've gone through and addressed all comments in the file in CS-network.php.2.diff ensuring they started with a capital and ended with punctuation.

[SergeyBiryukov]

In 46342:

Docs: Improve comments in tests/multisite/network.php per the documentation standards.

Props garrett-eclipse, jrf.
See #48142.

[SergeyBiryukov]

In 46343:

Coding Standards: Remove inline assignments and extra whitespace in tests/multisite/network.php.

Props garrett-eclipse, jrf.
See #48142.

[SergeyBiryukov]

Replying to jrf:

Re: [46316] - this may lead to difficult to debug issues unless you are 100% sure both sides of the comparison will always be numeric. Effectively, the change makes no difference to the code and it is still a loose type comparison, as the (int) cast is what PHP would do internally when using !=.

So if a true strict type comparison is intended here, a string-based strict comparison should be used:

<?php
if ( $cat_id !== (string) $category ) {

Thanks for the feedback! After some consideration, I'd like to leave that part as is, for a few reasons:

• [46316] was not meant to introduce the strict comparison, only to fix the issue introduced in [46313]. I've decided to double-check the change in [46313] and confirmed my suspicions, as the types were indeed different, hence the fix in [46316]. In my testing, this worked as expected, and did make a difference to the code.
• Link Manager was deprecated in core 7 years ago (#21307), so there should not be any changes to data structures there in the forseeable future, and it's safe to assume both sides of the comparison will always be numeric.
• I don't mind converting this check to (string), however (int) seems more consistent with other similar checks in core:
  • 29 matches for \(int\) [$a-z_]+? (=|!)== or (=|!)== \(int\)
  • 2 matches for \(string\) [$a-z_]+? (=|!)== or (=|!)== \(string\)