Coding Standards: Assignments must be the first block of code on a line

[subrataemfluence]

In wp-admin/admin.php (line 177) the $page_hook was not assigned as the first block.

Rather it is first directly checked against the return value of the function get_plugin_page_hook and the value is not null the function has been called again to assign the value to $page_hook variable.

Currently we have:

<?php
if ( ! $page_hook = get_plugin_page_hook( $plugin_page, $the_parent ) ) {
   $page_hook = get_plugin_page_hook( $plugin_page, $plugin_page );
   ...
}

Which could be avoided easily if we restructure the code like this:

<?php
$page_hook = get_plugin_page_hook( $plugin_page, $the_parent );
if ( null !== $page_hook ) {
 ...
}

Proposed patch added.

[jrf]

Updated patch - essential line had been removed.

[jrf]

@subrataemfluence @netweb Looks like the original patch removed an essential line of code. I've uploaded a renewed patch which fixes that.

Props @johnbillion for spotting it.

[subrataemfluence]

Thank you @johnbillion and @jrf for spotting and correcting the patch. However, I think the new patch has a logical issue.

<?php
$page_hook = get_plugin_page_hook( $plugin_page, $the_parent );
if ( null !== $page_hook ) {
   $page_hook = get_plugin_page_hook( $plugin_page, $plugin_page );
   ...
}

The above will generate error.

We should only retrieve the plugin page hook sending current $plugin_page in both parameters if null is returned using $the_parent. Can this be the correct approach?

<?php
$page_hook = get_plugin_page_hook( $plugin_page, $the_parent );
if ( null === $page_hook ) {
   $page_hook = get_plugin_page_hook( $plugin_page, $plugin_page );
}
...

The entire code block looks like:

<?php
if ( isset( $plugin_page ) ) {
   if ( ! empty( $typenow ) ) {
      $the_parent = $pagenow . '?post_type=' . $typenow;
   } else {
      $the_parent = $pagenow;
   }

   $page_hook = get_plugin_page_hook( $plugin_page, $the_parent );
   if ( null === $page_hook ) {
      $page_hook = get_plugin_page_hook($plugin_page, $plugin_page);
   }

   // Back-compat for plugins using add_management_page().
   if ( empty( $page_hook ) && 'edit.php' == $pagenow && '' != get_plugin_page_hook( $plugin_page, 'tools.php' ) ) {
      // There could be plugin specific params on the URL, so we need the whole query string
      if ( ! empty( $_SERVER['QUERY_STRING'] ) ) {
         $query_string = $_SERVER['QUERY_STRING'];
      } else {
         $query_string = 'page=' . $plugin_page;
      }
      wp_redirect( admin_url( 'tools.php?' . $query_string ) );
      exit;
   }

   unset( $the_parent );
}

[pento]

In 44598:

Coding Standards: Move an assignment out of a condition in wp-admin/admin.php.

Props subrataemfluence, jrf, pento.
Fixes #44363.