Coding Standards: Yoda condition check is not present

[subrataemfluence]

In wp-admin/admin.php between line nos. 358 and 372, Yoda conditions are not met according to the sniffer.

Here is the code we have now:

<?php
if ( $typenow == 'page' ) {
   if ( $pagenow == 'post-new.php' ) {
     do_action( 'load-page-new.php' );
   } elseif ( $pagenow == 'post.php' ) {
     do_action( 'load-page.php' );
   }
} elseif ( $pagenow == 'edit-tags.php' ) {
   if ( $taxnow == 'category' ) {
     do_action( 'load-categories.php' );
   } elseif ( $taxnow == 'link_category' ) {
     do_action( 'load-edit-link-categories.php' );
   }
} elseif ( 'term.php' === $pagenow ) {
   do_action( 'load-edit-tags.php' );
}

One thing to notice at the end of the above code block. We have a Yoda condition checked. I am not sure why it used only at one place. Is this done in purpose?

<?php
...
} elseif ( 'term.php' === $pagenow ) {
   do_action( 'load-edit-tags.php' );
}

Can we re-write the block of code and make it consistent as proposed in my patch?

[abdullahramzan]

Duplicate of #44028.

[subrataemfluence]

Thank you for looking into it.
But as far as I have seen there is no patch for admin.php in #44208.
Should I upload the patch there?

[mukesh27]

#47193 was marked as a duplicate.

[SergeyBiryukov]

In 47218:

Coding Standards: Use Yoda conditions in some wp-admin files.

Props subrataemfluence, marcio-zebedeu, bookdude13, 1naveengiri, alishankhan.
Fixes #44365, #48455.