Changeset 45583 for trunk/src/wp-admin/includes/class-wp-filesystem-base.php

Timestamp:

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

Author:

pento

Message:

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

See #47632.

File:

• trunk/src/wp-admin/includes/class-wp-filesystem-base.php (modified) (3 diffs)

trunk/src/wp-admin/includes/class-wp-filesystem-base.php

@@ -225 +225 @@
             return $folder;
         }
-        if ( $return = $this->search_for_folder( $folder ) ) {
+        $return = $this->search_for_folder( $folder );
+        if ( $return ) {
             $this->cache[ $folder ] = $return;
         }
@@ -285 +286 @@
                 // Only search for the remaining path tokens in the directory, not the full path again.
                 $newfolder = implode( '/', array_slice( $folder_parts, $index + 1 ) );
-                if ( $ret = $this->search_for_folder( $newfolder, $newdir, $loop ) ) {
+                $ret       = $this->search_for_folder( $newfolder, $newdir, $loop );
+                if ( $ret ) {
                     return $ret;
                 }
@@ -399 +401 @@
 
         for ( $i = 0, $c = count( $attarray ); $i < $c; $i++ ) {
-            if ( $key = array_search( $attarray[ $i ], $legal ) ) {
+            $key = array_search( $attarray[ $i ], $legal );
+            if ( $key ) {
                 $realmode .= $legal[ $key ];
             }