Changeset 46088 for trunk/src/wp-admin/includes/class-custom-background.php

Timestamp:

09/10/2019 07:22:07 PM (5 years ago)

Author:

SergeyBiryukov

Message:

Docs: Fix placement of some duplicate hook references.

Hook documentation should be on the line directly above the line containing the do_action() or apply_filters() call. The CS auto-fixing, which changed some inconsistent function calls to multi-line function calls, is part of the reason why this was no longer the case for a select group of duplicate hook references.

Includes minor code layout fixes.

See #47110.

File:

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

trunk/src/wp-admin/includes/class-custom-background.php

@@ -585 +585 @@
             exit;
         }
+
         $attachment_id = absint( $_POST['attachment_id'] );
-        /** This filter is documented in wp-admin/includes/media.php */
+
         $sizes = array_keys(
+            /** This filter is documented in wp-admin/includes/media.php */
             apply_filters(
                 'image_size_names_choose',
@@ -598 +600 @@
             )
         );
-        $size  = 'thumbnail';
+
+        $size = 'thumbnail';
         if ( in_array( $_POST['size'], $sizes ) ) {
             $size = esc_attr( $_POST['size'] );
@@ -604 +607 @@
 
         update_post_meta( $attachment_id, '_wp_attachment_is_custom_background', get_option( 'stylesheet' ) );
+
         $url       = wp_get_attachment_image_src( $attachment_id, $size );
         $thumbnail = wp_get_attachment_image_src( $attachment_id, 'thumbnail' );