Changeset 45583 for trunk/src/wp-admin/async-upload.php

Timestamp:

07/01/2019 12:50:14 PM (6 years ago)

Author:

pento

Message:

Coding Standards: Fix the Squiz.PHP.DisallowMultipleAssignments violations in wp-admin.

See #47632.

File:

• trunk/src/wp-admin/async-upload.php (modified) (2 diffs)

trunk/src/wp-admin/async-upload.php

@@ -40 +40 @@
 
 // just fetch the detail form for that attachment
-if ( isset( $_REQUEST['attachment_id'] ) && ( $id = intval( $_REQUEST['attachment_id'] ) ) && $_REQUEST['fetch'] ) {
+if ( isset( $_REQUEST['attachment_id'] ) && intval( $_REQUEST['attachment_id'] ) && $_REQUEST['fetch'] ) {
+    $id   = intval( $_REQUEST['attachment_id'] );
     $post = get_post( $id );
     if ( 'attachment' != $post->post_type ) {
@@ -51 +52 @@
     switch ( $_REQUEST['fetch'] ) {
         case 3:
-            if ( $thumb_url = wp_get_attachment_image_src( $id, 'thumbnail', true ) ) {
+            $thumb_url = wp_get_attachment_image_src( $id, 'thumbnail', true );
+            if ( $thumb_url ) {
                 echo '<img class="pinkynail" src="' . esc_url( $thumb_url[0] ) . '" alt="" />';
             }