Changeset 13901 for trunk/src/bp-core/classes/class-bp-attachment-cover-image.php

Timestamp:

06/02/2024 07:14:55 PM (20 months ago)

Author:

espellcaste

Message:

WPCS: Part IX: miscellaneous fixes for some of the files of the core component.

Follow-up to [13883], [13886], [13887], [13888], [13891], [13892], [13893] and [13900]

See #9164 and #7228

File:

• trunk/src/bp-core/classes/class-bp-attachment-cover-image.php (modified) (8 diffs)

trunk/src/bp-core/classes/class-bp-attachment-cover-image.php

@@ -30 +30 @@
         $max_upload_file_size = bp_attachments_get_max_upload_file_size( 'cover_image' );
 
-        parent::__construct( array(
-            'action'                => 'bp_cover_image_upload',
-            'file_input'            => 'file',
-            'original_max_filesize' => $max_upload_file_size,
-            'base_dir'              => bp_attachments_uploads_dir_get( 'dir' ),
-            'required_wp_files'     => array( 'file', 'image' ),
-
-            // Specific errors for cover images.
-            'upload_error_strings'  => array(
-                /* translators: %s: Max file size for the cover image */
-                11  => sprintf( _x( 'That image is too big. Please upload one smaller than %s', 'cover image upload error', 'buddypress' ), size_format( $max_upload_file_size ) ),
-
-                /* translators: %s: comma separated list of file types allowed for the cover image */
-                12  => sprintf( _nx( 'Please upload only this file type: %s.', 'Please upload only these file types: %s.', count( $allowed_types ), 'cover image upload error', 'buddypress' ), self::get_cover_image_types( $allowed_types ) ),
-            ),
-        ) );
+        parent::__construct(
+            array(
+                'action'                => 'bp_cover_image_upload',
+                'file_input'            => 'file',
+                'original_max_filesize' => $max_upload_file_size,
+                'base_dir'              => bp_attachments_uploads_dir_get( 'dir' ),
+                'required_wp_files'     => array( 'file', 'image' ),
+
+                // Specific errors for cover images.
+                'upload_error_strings'  => array(
+                    /* translators: %s: Max file size for the cover image */
+                    11 => sprintf( _x( 'That image is too big. Please upload one smaller than %s', 'cover image upload error', 'buddypress' ), size_format( $max_upload_file_size ) ),
+
+                    /* translators: %s: comma separated list of file types allowed for the cover image */
+                    12 => sprintf( _nx( 'Please upload only this file type: %s.', 'Please upload only these file types: %s.', count( $allowed_types ), 'cover image upload error', 'buddypress' ), self::get_cover_image_types( $allowed_types ) ),
+                ),
+            )
+        );
     }
 
@@ -83 +85 @@
             $file['error'] = 11;
 
-        // File is of invalid type.
+            // File is of invalid type.
         } elseif ( isset( $file['tmp_name'] ) && isset( $file['name'] ) && ! bp_attachments_check_filetype( $file['tmp_name'], $file['name'], bp_attachments_get_allowed_mimes( 'cover_image' ) ) ) {
             $file['error'] = 12;
@@ -138 +140 @@
             3 => 180,
             6 => -90,
-            8 =>  90,
+            8 => 90,
         );
 
@@ -149 +151 @@
             return false;
 
-        // Add the file to the edit arguments.
+            // Add the file to the edit arguments.
         } else {
-            $edit_args = array_merge( $edit_args, array( 'file' => $file, 'save' => false ) );
+            $edit_args = array_merge(
+                $edit_args,
+                array(
+                    'file' => $file,
+                    'save' => false,
+                )
+            );
         }
 
@@ -157 +165 @@
         $editor = parent::edit_image( 'cover_image', $edit_args );
 
-        if ( is_wp_error( $editor ) )  {
+        if ( is_wp_error( $editor ) ) {
             return $editor;
         } elseif ( ! is_a( $editor, 'WP_Image_Editor' ) ) {
@@ -173 +181 @@
      *
      * @param string $file The absolute path to the file.
-     * @return false|string $value The absolute path to the new file name.
+     * @return false|string
      */
     public function generate_filename( $file = '' ) {
@@ -208 +216 @@
                 'item_id'         => $item_id,
                 'has_cover_image' => bp_attachments_get_user_has_cover_image( $item_id ),
-                'nonces'  => array(
+                'nonces'          => array(
                     'remove' => wp_create_nonce( 'bp_delete_cover_image' ),
                 ),
@@ -226 +234 @@
                 'item_id'         => bp_get_current_group_id(),
                 'has_cover_image' => bp_attachments_get_group_has_cover_image( $item_id ),
-                'nonces'  => array(
+                'nonces'          => array(
                     'remove' => wp_create_nonce( 'bp_delete_cover_image' ),
                 ),