Changeset 51030

Timestamp:

05/26/2021 02:23:11 PM (2 years ago)

Author:

SergeyBiryukov

Message:

Media: Replace basename() usage on media upload screen with wp_basename() for better multibyte filenames support.

Includes minor code layout fixes for better readability.

Follow-up to [44785], [51023].

See #51754.

Location:

trunk/src

Files:

• js/_enqueues/vendor/plupload/handlers.js (modified) (1 diff)
• wp-admin/async-upload.php (modified) (1 diff)
• wp-admin/css/media.css (modified) (1 diff)

trunk/src/js/_enqueues/vendor/plupload/handlers.js

@@ -373 +373 @@
 function copyAttachmentUploadURLClipboard() {
     var clipboard = new ClipboardJS( '.copy-attachment-url' ),
-    successTimeout;
+        successTimeout;
+
     clipboard.on( 'success', function( event ) {
         var triggerElement = jQuery( event.trigger ),
             successElement = jQuery( '.success', triggerElement.closest( '.copy-to-clipboard-container' ) );
+
         // Clear the selection and move focus back to the trigger.
         event.clearSelection();

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

@@ -49 +49 @@
     switch ( $_REQUEST['fetch'] ) {
         case 3:
-            echo '<div class="attachment-details">';
-            $thumb_url = wp_get_attachment_image_src( $id, 'thumbnail', true );
-            if ( $thumb_url ) {
-                echo '<img class="pinkynail" src="' . esc_url( $thumb_url[0] ) . '" alt="" />';
-            }
-            // Title shouldn't ever be empty, but use filename just in case.
-            $file    = get_attached_file( $post->ID );
-            $fileurl = wp_get_attachment_url( $post->ID );
-            $title   = $post->post_title ? $post->post_title : wp_basename( $file );
+            ?>
+            <div class="attachment-details">
+                <?php
+                $thumb_url = wp_get_attachment_image_src( $id, 'thumbnail', true );
+                if ( $thumb_url ) {
+                    echo '<img class="pinkynail" src="' . esc_url( $thumb_url[0] ) . '" alt="" />';
+                }
 
-            echo '<div class="filename new"><span class="media-list-title"><strong>' . esc_html( wp_html_excerpt( $title, 60, '&hellip;' ) ) . '</strong></span> <span class="media-list-subtitle">' . basename( $file ) . '</span></div>';
-            echo '</div>';
-            ?>
+                // Title shouldn't ever be empty, but use filename just in case.
+                $file     = get_attached_file( $post->ID );
+                $file_url = wp_get_attachment_url( $post->ID );
+                $title    = $post->post_title ? $post->post_title : wp_basename( $file );
+                ?>
+                <div class="filename new">
+                    <span class="media-list-title"><strong><?php echo esc_html( wp_html_excerpt( $title, 60, '&hellip;' ) ); ?></strong></span>
+                    <span class="media-list-subtitle"><?php echo wp_basename( $file ); ?></span>
+                </div>
+            </div>
             <div class="attachment-tools">
                 <span class="media-item-copy-container copy-to-clipboard-container edit-attachment">
-                    <button type="button" class="button button-small copy-attachment-url" data-clipboard-text="<?php echo $fileurl; ?>"><?php _e( 'Copy URL to clipboard' ); ?></button>
+                    <button type="button" class="button button-small copy-attachment-url" data-clipboard-text="<?php echo $file_url; ?>"><?php _e( 'Copy URL to clipboard' ); ?></button>
                     <span class="success hidden" aria-hidden="true"><?php _e( 'Copied!' ); ?></span>
                 </span>

trunk/src/wp-admin/css/media.css

@@ -139 +139 @@
 
 .media-item button .copy-attachment-url {
-    margin-top:14px;
+    margin-top: 14px;
 }
 
 .media-item .copy-to-clipboard-container {
-    margin-top:7px;
+    margin-top: 7px;
 }