WordPress 4.6.1
[autoinstalls/wordpress.git] / wp-admin / includes / media.php
index fc8ce0fe97a654d81f8495a92a68a4d6c845e989..c6b2dcd1ba4fb25271de5e637dcc9ce793268eb2 100644 (file)
@@ -22,7 +22,7 @@ function media_upload_tabs() {
        );
 
        /**
-        * Filter the available tabs in the legacy (pre-3.5.0) media popup.
+        * Filters the available tabs in the legacy (pre-3.5.0) media popup.
         *
         * @since 2.5.0
         *
@@ -36,7 +36,7 @@ function media_upload_tabs() {
  *
  * @since 2.5.0
  *
- * @global wpdb $wpdb
+ * @global wpdb $wpdb WordPress database abstraction object.
  *
  * @param array $tabs
  * @return array $tabs with gallery if post has image attachment
@@ -65,7 +65,7 @@ function update_gallery_tab($tabs) {
 }
 
 /**
- * {@internal Missing Short Description}}
+ * Outputs the legacy media upload tabs UI.
  *
  * @since 2.5.0
  *
@@ -102,42 +102,52 @@ function the_media_upload_tabs() {
 }
 
 /**
- * {@internal Missing Short Description}}
+ * Retrieves the image HTML to send to the editor.
  *
  * @since 2.5.0
  *
- * @param integer $id image attachment id
- * @param string $caption image caption
- * @param string $alt image alt attribute
- * @param string $title image title attribute
- * @param string $align image css alignment property
- * @param string $url image src url
- * @param string|bool $rel image rel attribute
- * @param string $size image size (thumbnail, medium, large, full or added  with add_image_size() )
- * @return string the html to insert into editor
+ * @param int          $id      Image attachment id.
+ * @param string       $caption Image caption.
+ * @param string       $title   Image title attribute.
+ * @param string       $align   Image CSS alignment property.
+ * @param string       $url     Optional. Image src URL. Default empty.
+ * @param bool|string  $rel     Optional. Value for rel attribute or whether to add a default value. Default false.
+ * @param string|array $size    Optional. Image size. Accepts any valid image size, or an array of width
+ *                              and height values in pixels (in that order). Default 'medium'.
+ * @param string       $alt     Optional. Image alt attribute. Default empty.
+ * @return string The HTML output to insert into the editor.
  */
-function get_image_send_to_editor($id, $caption, $title, $align, $url='', $rel = false, $size='medium', $alt = '') {
+function get_image_send_to_editor( $id, $caption, $title, $align, $url = '', $rel = false, $size = 'medium', $alt = '' ) {
 
-       $html = get_image_tag($id, $alt, '', $align, $size);
+       $html = get_image_tag( $id, $alt, '', $align, $size );
 
-       $rel = $rel ? ' rel="attachment wp-att-' . esc_attr($id).'"' : '';
+       if ( $rel ) {
+               if ( is_string( $rel ) ) {
+                       $rel = ' rel="' . esc_attr( $rel ) . '"';
+               } else {
+                       $rel = ' rel="attachment wp-att-' . intval( $id ) . '"';
+               }
+       } else {
+               $rel = '';
+       }
 
        if ( $url )
-               $html = '<a href="' . esc_attr($url) . "\"$rel>$html</a>";
+               $html = '<a href="' . esc_attr( $url ) . '"' . $rel . '>' . $html . '</a>';
 
        /**
-        * Filter the image HTML markup to send to the editor.
+        * Filters the image HTML markup to send to the editor.
         *
         * @since 2.5.0
         *
-        * @param string $html    The image HTML markup to send.
-        * @param int    $id      The attachment id.
-        * @param string $caption The image caption.
-        * @param string $title   The image title.
-        * @param string $align   The image alignment.
-        * @param string $url     The image source URL.
-        * @param string $size    The image size.
-        * @param string $alt     The image alternative, or alt, text.
+        * @param string       $html    The image HTML markup to send.
+        * @param int          $id      The attachment id.
+        * @param string       $caption The image caption.
+        * @param string       $title   The image title.
+        * @param string       $align   The image alignment.
+        * @param string       $url     The image source URL.
+        * @param string|array $size    Size of image. Image size or array of width and height values
+        *                              (in that order). Default 'medium'.
+        * @param string       $alt     The image alternative, or alt, text.
         */
        $html = apply_filters( 'image_send_to_editor', $html, $id, $caption, $title, $align, $url, $size, $alt );
 
@@ -152,23 +162,23 @@ function get_image_send_to_editor($id, $caption, $title, $align, $url='', $rel =
  * @param string $html
  * @param integer $id
  * @param string $caption image caption
- * @param string $alt image alt attribute
  * @param string $title image title attribute
  * @param string $align image css alignment property
  * @param string $url image src url
  * @param string $size image size (thumbnail, medium, large, full or added with add_image_size() )
+ * @param string $alt image alt attribute
  * @return string
  */
 function image_add_caption( $html, $id, $caption, $title, $align, $url, $size, $alt = '' ) {
 
        /**
-        * Filter the caption text.
+        * Filters the caption text.
         *
         * Note: If the caption text is empty, the caption shortcode will not be appended
         * to the image HTML when inserted into the editor.
         *
         * Passing an empty value also prevents the {@see 'image_add_caption_shortcode'}
-        * filter from being evaluated at the end of {@see image_add_caption()}.
+        * Filters from being evaluated at the end of image_add_caption().
         *
         * @since 4.1.0
         *
@@ -178,7 +188,7 @@ function image_add_caption( $html, $id, $caption, $title, $align, $url, $size, $
        $caption = apply_filters( 'image_add_caption_text', $caption, $id );
 
        /**
-        * Filter whether to disable captions.
+        * Filters whether to disable captions.
         *
         * Prevents image captions from being appended to image HTML when inserted into the editor.
         *
@@ -210,7 +220,7 @@ function image_add_caption( $html, $id, $caption, $title, $align, $url, $size, $
        $shcode = '[caption id="' . $id . '" align="align' . $align     . '" width="' . $width . '"]' . $html . ' ' . $caption . '[/caption]';
 
        /**
-        * Filter the image HTML markup including the caption shortcode.
+        * Filters the image HTML markup including the caption shortcode.
         *
         * @since 2.6.0
         *
@@ -242,7 +252,7 @@ function media_send_to_editor($html) {
 ?>
 <script type="text/javascript">
 var win = window.dialogArguments || opener || parent || top;
-win.send_to_editor('<?php echo addslashes($html); ?>');
+win.send_to_editor( <?php echo wp_json_encode( $html ); ?> );
 </script>
 <?php
        exit;
@@ -253,11 +263,11 @@ win.send_to_editor('<?php echo addslashes($html); ?>');
  *
  * @since 2.5.0
  *
- * @param string $file_id   Index of the {@link $_FILES} array that the file was sent. Required.
+ * @param string $file_id   Index of the `$_FILES` array that the file was sent. Required.
  * @param int    $post_id   The post ID of a post to attach the media item to. Required, but can
  *                          be set to 0, creating a media item that has no relationship to a post.
  * @param array  $post_data Overwrite some of the attachment. Optional.
- * @param array  $overrides Override the {@link wp_handle_upload()} behavior. Optional.
+ * @param array  $overrides Override the wp_handle_upload() behavior. Optional.
  * @return int|WP_Error ID of the attachment or a WP_Error object on failure.
  */
 function media_handle_upload($file_id, $post_id, $post_data = array(), $overrides = array( 'test_form' => false )) {
@@ -280,7 +290,7 @@ function media_handle_upload($file_id, $post_id, $post_data = array(), $override
        $url = $file['url'];
        $type = $file['type'];
        $file = $file['file'];
-       $title = $name;
+       $title = sanitize_title( $name );
        $content = '';
        $excerpt = '';
 
@@ -370,17 +380,17 @@ function media_handle_upload($file_id, $post_id, $post_data = array(), $override
 }
 
 /**
- * This handles a sideloaded file in the same way as an uploaded file is handled by {@link media_handle_upload()}
+ * Handles a side-loaded file in the same way as an uploaded file is handled by media_handle_upload().
  *
  * @since 2.6.0
  *
- * @param array $file_array Array similar to a {@link $_FILES} upload array
- * @param int $post_id The post ID the media is associated with
- * @param string $desc Description of the sideloaded file
- * @param array $post_data allows you to overwrite some of the attachment
- * @return int|object The ID of the attachment or a WP_Error on failure
+ * @param array  $file_array Array similar to a `$_FILES` upload array.
+ * @param int    $post_id    The post ID the media is associated with.
+ * @param string $desc       Optional. Description of the side-loaded file. Default null.
+ * @param array  $post_data  Optional. Post data to override. Default empty array.
+ * @return int|object The ID of the attachment or a WP_Error on failure.
  */
-function media_handle_sideload($file_array, $post_id, $desc = null, $post_data = array()) {
+function media_handle_sideload( $file_array, $post_id, $desc = null, $post_data = array() ) {
        $overrides = array('test_form'=>false);
 
        $time = current_time( 'mysql' );
@@ -449,7 +459,7 @@ wp_enqueue_style( 'colors' );
 // Check callback name for 'media'
 if ( ( is_array( $content_func ) && ! empty( $content_func[1] ) && 0 === strpos( (string) $content_func[1], 'media' ) )
        || ( ! is_array( $content_func ) && 0 === strpos( $content_func, 'media' ) ) )
-       wp_enqueue_style( 'media' );
+       wp_enqueue_style( 'deprecated-media' );
 wp_enqueue_style( 'ie' );
 ?>
 <script type="text/javascript">
@@ -558,10 +568,12 @@ function media_buttons($editor_id = 'content') {
                $img . __( 'Add Media' )
        );
        /**
-        * Filter the legacy (pre-3.5.0) media buttons.
+        * Filters the legacy (pre-3.5.0) media buttons.
+        *
+        * Use {@see 'media_buttons'} action instead.
         *
         * @since 2.5.0
-        * @deprecated 3.5.0 Use 'media_buttons' action instead.
+        * @deprecated 3.5.0 Use {@see 'media_buttons'} action instead.
         *
         * @param string $string Media buttons context. Default empty.
         */
@@ -598,7 +610,7 @@ function get_upload_iframe_src( $type = null, $post_id = null, $tab = null ) {
                $upload_iframe_src = add_query_arg('tab', $tab, $upload_iframe_src);
 
        /**
-        * Filter the upload iframe source URL for a specific media type.
+        * Filters the upload iframe source URL for a specific media type.
         *
         * The dynamic portion of the hook name, `$type`, refers to the type
         * of media uploaded.
@@ -613,7 +625,7 @@ function get_upload_iframe_src( $type = null, $post_id = null, $tab = null ) {
 }
 
 /**
- * {@internal Missing Short Description}}
+ * Handles form submissions for the legacy media uploader.
  *
  * @since 2.5.0
  *
@@ -650,14 +662,14 @@ function media_upload_form_handler() {
                }
 
                /**
-                * Filter the attachment fields to be saved.
+                * Filters the attachment fields to be saved.
                 *
                 * @since 2.5.0
                 *
                 * @see wp_get_attachment_metadata()
                 *
-                * @param WP_Post $post       The WP_Post object.
-                * @param array   $attachment An array of attachment metadata.
+                * @param array $post       An array of post data.
+                * @param array $attachment An array of attachment metadata.
                 */
                $post = apply_filters( 'attachment_fields_to_save', $post, $attachment );
 
@@ -706,7 +718,7 @@ function media_upload_form_handler() {
                }
 
                /**
-                * Filter the HTML markup for a media item sent to the editor.
+                * Filters the HTML markup for a media item sent to the editor.
                 *
                 * @since 2.5.0
                 *
@@ -724,7 +736,7 @@ function media_upload_form_handler() {
 }
 
 /**
- * {@internal Missing Short Description}}
+ * Handles the process of uploading media.
  *
  * @since 2.5.0
  *
@@ -764,7 +776,7 @@ function wp_media_upload_handler() {
                                        $type = $ext_type;
 
                        /**
-                        * Filter the URL sent to the editor for a specific media type.
+                        * Filters the URL sent to the editor for a specific media type.
                         *
                         * The dynamic portion of the hook name, `$type`, refers to the type
                         * of media being sent.
@@ -787,7 +799,7 @@ function wp_media_upload_handler() {
                                $html = "<img src='" . esc_url($src) . "' alt='$alt'$class />";
 
                        /**
-                        * Filter the image URL sent to the editor.
+                        * Filters the image URL sent to the editor.
                         *
                         * @since 2.8.0
                         *
@@ -805,7 +817,9 @@ function wp_media_upload_handler() {
 
        if ( isset( $_POST['save'] ) ) {
                $errors['upload_notice'] = __('Saved.');
-               return media_upload_gallery();
+               wp_enqueue_script( 'admin-gallery' );
+               return wp_iframe( 'media_upload_gallery_form', $errors );
+
        } elseif ( ! empty( $_POST ) ) {
                $return = media_upload_form_handler();
 
@@ -842,6 +856,10 @@ function media_sideload_image( $file, $post_id, $desc = null, $return = 'html' )
 
                // Set variables for storage, fix file filename for query strings.
                preg_match( '/[^\?]+\.(jpe?g|jpe|gif|png)\b/i', $file, $matches );
+               if ( ! $matches ) {
+                       return new WP_Error( 'image_sideload_failed', __( 'Invalid image URL' ) );
+               }
+
                $file_array = array();
                $file_array['name'] = basename( $matches[0] );
 
@@ -880,7 +898,7 @@ function media_sideload_image( $file, $post_id, $desc = null, $return = 'html' )
 }
 
 /**
- * {@internal Missing Short Description}}
+ * Retrieves the legacy media uploader form in an iframe.
  *
  * @since 2.5.0
  *
@@ -903,7 +921,7 @@ function media_upload_gallery() {
 }
 
 /**
- * {@internal Missing Short Description}}
+ * Retrieves the legacy media library form in an iframe.
  *
  * @since 2.5.0
  *
@@ -962,7 +980,7 @@ function image_align_input_fields( $post, $checked = '' ) {
  */
 function image_size_input_fields( $post, $check = '' ) {
        /**
-        * Filter the names and labels of the default image sizes.
+        * Filters the names and labels of the default image sizes.
         *
         * @since 3.3.0
         *
@@ -1072,7 +1090,7 @@ function wp_caption_input_textarea($edit_post) {
 }
 
 /**
- * {@internal Missing Short Description}}
+ * Retrieves the image attachment fields to edit form fields.
  *
  * @since 2.5.0
  *
@@ -1085,7 +1103,7 @@ function image_attachment_fields_to_edit($form_fields, $post) {
 }
 
 /**
- * {@internal Missing Short Description}}
+ * Retrieves the single non-image attachment fields to edit form fields.
  *
  * @since 2.5.0
  *
@@ -1099,7 +1117,7 @@ function media_single_attachment_fields_to_edit( $form_fields, $post ) {
 }
 
 /**
- * {@internal Missing Short Description}}
+ * Retrieves the post non-image attachment fields to edito form fields.
  *
  * @since 2.8.0
  *
@@ -1116,7 +1134,7 @@ function media_post_single_attachment_fields_to_edit( $form_fields, $post ) {
  * Filters input from media_upload_form_handler() and assigns a default
  * post_title from the file name if none supplied.
  *
- * Illustrates the use of the attachment_fields_to_save filter
+ * Illustrates the use of the {@see 'attachment_fields_to_save'} filter
  * which can be used to add default values to any field before saving to DB.
  *
  * @since 2.5.0
@@ -1138,7 +1156,7 @@ function image_attachment_fields_to_save( $post, $attachment ) {
 }
 
 /**
- * {@internal Missing Short Description}}
+ * Retrieves the media element HTML to send to the editor.
  *
  * @since 2.5.0
  *
@@ -1154,7 +1172,7 @@ function image_media_send_to_editor($html, $attachment_id, $attachment) {
                $align = !empty($attachment['align']) ? $attachment['align'] : 'none';
                $size = !empty($attachment['image-size']) ? $attachment['image-size'] : 'medium';
                $alt = !empty($attachment['image_alt']) ? $attachment['image_alt'] : '';
-               $rel = ( $url == get_attachment_link($attachment_id) );
+               $rel = ( strpos( $url, 'attachment_id') || $url === get_attachment_link( $attachment_id ) );
 
                return get_image_send_to_editor($attachment_id, $attachment['post_excerpt'], $attachment['post_title'], $align, $url, $rel, $size, $alt);
        }
@@ -1163,7 +1181,7 @@ function image_media_send_to_editor($html, $attachment_id, $attachment) {
 }
 
 /**
- * {@internal Missing Short Description}}
+ * Retrieves the attachment fields to edit form fields.
  *
  * @since 2.5.0
  *
@@ -1239,7 +1257,7 @@ function get_attachment_fields_to_edit($post, $errors = null) {
        }
 
        // Merge default fields with their errors, so any key passed with the error (e.g. 'error', 'helps', 'value') will replace the default
-       // The recursive merge is easily traversed with array casting: foreach( (array) $things as $thing )
+       // The recursive merge is easily traversed with array casting: foreach ( (array) $things as $thing )
        $form_fields = array_merge_recursive($form_fields, (array) $errors);
 
        // This was formerly in image_attachment_fields_to_edit().
@@ -1269,7 +1287,7 @@ function get_attachment_fields_to_edit($post, $errors = null) {
        }
 
        /**
-        * Filter the attachment fields to edit.
+        * Filters the attachment fields to edit.
         *
         * @since 2.5.0
         *
@@ -1353,7 +1371,7 @@ function get_media_item( $attachment_id, $args = null ) {
        $args = wp_parse_args( $args, $default_args );
 
        /**
-        * Filter the arguments used to retrieve an image for the edit image form.
+        * Filters the arguments used to retrieve an image for the edit image form.
         *
         * @since 3.1.0
         *
@@ -1366,7 +1384,8 @@ function get_media_item( $attachment_id, $args = null ) {
        $toggle_on  = __( 'Show' );
        $toggle_off = __( 'Hide' );
 
-       $filename = esc_html( wp_basename( $post->guid ) );
+       $file = get_attached_file( $post->ID );
+       $filename = esc_html( wp_basename( $file ) );
        $title = esc_attr( $post->post_title );
 
        $post_mime_types = get_post_mime_types();
@@ -1410,7 +1429,7 @@ function get_media_item( $attachment_id, $args = null ) {
                $media_dims .= "<span id='media-dims-$post->ID'>{$meta['width']}&nbsp;&times;&nbsp;{$meta['height']}</span> ";
 
        /**
-        * Filter the media metadata.
+        * Filters the media metadata.
         *
         * @since 2.5.0
         *
@@ -1442,7 +1461,7 @@ function get_media_item( $attachment_id, $args = null ) {
                        <td>
                        <p><strong>" . __('File name:') . "</strong> $filename</p>
                        <p><strong>" . __('File type:') . "</strong> $post->post_mime_type</p>
-                       <p><strong>" . __('Upload date:') . "</strong> " . mysql2date( get_option('date_format'), $post->post_date ). '</p>';
+                       <p><strong>" . __('Upload date:') . "</strong> " . mysql2date( __( 'F j, Y' ), $post->post_date ). '</p>';
                        if ( !empty( $media_dims ) )
                                $item .= "<p><strong>" . __('Dimensions:') . "</strong> $media_dims</p>\n";
 
@@ -1451,8 +1470,9 @@ function get_media_item( $attachment_id, $args = null ) {
        $item .= "
                </thead>
                <tbody>
-               <tr><td colspan='2' class='imgedit-response' id='imgedit-response-$post->ID'></td></tr>
-               <tr><td style='display:none' colspan='2' class='image-editor' id='image-editor-$post->ID'></td></tr>\n";
+               <tr><td colspan='2' class='imgedit-response' id='imgedit-response-$post->ID'></td></tr>\n
+               <tr><td style='display:none' colspan='2' class='image-editor' id='image-editor-$post->ID'></td></tr>\n
+               <tr><td colspan='2'><p class='media-types media-types-required-info'>" . sprintf( __( 'Required fields are marked %s' ), '<span class="required">*</span>' ) . "</p></td></tr>\n";
 
        $defaults = array(
                'input'      => 'text',
@@ -1471,7 +1491,9 @@ function get_media_item( $attachment_id, $args = null ) {
                        $delete = "<a href='" . wp_nonce_url( "post.php?action=delete&amp;post=$attachment_id", 'delete-post_' . $attachment_id ) . "' id='del[$attachment_id]' class='delete-permanently'>" . __( 'Delete Permanently' ) . '</a>';
                } elseif ( !MEDIA_TRASH ) {
                        $delete = "<a href='#' class='del-link' onclick=\"document.getElementById('del_attachment_$attachment_id').style.display='block';return false;\">" . __( 'Delete' ) . "</a>
-                        <div id='del_attachment_$attachment_id' class='del-attachment' style='display:none;'><p>" . sprintf( __( 'You are about to delete <strong>%s</strong>.' ), $filename ) . "</p>
+                        <div id='del_attachment_$attachment_id' class='del-attachment' style='display:none;'>" .
+                        /* translators: %s: file name */
+                       '<p>' . sprintf( __( 'You are about to delete %s.' ), '<strong>' . $filename . '</strong>' ) . "</p>
                         <a href='" . wp_nonce_url( "post.php?action=delete&amp;post=$attachment_id", 'delete-post_' . $attachment_id ) . "' id='del[$attachment_id]' class='button'>" . __( 'Continue' ) . "</a>
                         <a href='#' class='button' onclick=\"this.parentNode.style.display='none';return false;\">" . __( 'Cancel' ) . "</a>
                         </div>";
@@ -1522,12 +1544,13 @@ function get_media_item( $attachment_id, $args = null ) {
                        continue;
                }
 
-               $required      = $field['required'] ? '<span class="alignright"><abbr title="required" class="required">*</abbr></span>' : '';
-               $aria_required = $field['required'] ? " aria-required='true' " : '';
+               $required      = $field['required'] ? '<span class="required">*</span>' : '';
+               $required_attr = $field['required'] ? ' required' : '';
+               $aria_required = $field['required'] ? " aria-required='true'" : '';
                $class  = $id;
                $class .= $field['required'] ? ' form-required' : '';
 
-               $item .= "\t\t<tr class='$class'>\n\t\t\t<th scope='row' class='label'><label for='$name'><span class='alignleft'>{$field['label']}</span>$required<br class='clear' /></label></th>\n\t\t\t<td class='field'>";
+               $item .= "\t\t<tr class='$class'>\n\t\t\t<th scope='row' class='label'><label for='$name'><span class='alignleft'>{$field['label']}{$required}</span><br class='clear' /></label></th>\n\t\t\t<td class='field'>";
                if ( !empty( $field[ $field['input'] ] ) )
                        $item .= $field[ $field['input'] ];
                elseif ( $field['input'] == 'textarea' ) {
@@ -1536,9 +1559,9 @@ function get_media_item( $attachment_id, $args = null ) {
                                $field['value'] = htmlspecialchars( $field['value'], ENT_QUOTES );
                        }
                        // Post_excerpt is already escaped by sanitize_post() in get_attachment_fields_to_edit().
-                       $item .= "<textarea id='$name' name='$name' $aria_required>" . $field['value'] . '</textarea>';
+                       $item .= "<textarea id='$name' name='$name'{$required_attr}{$aria_required}>" . $field['value'] . '</textarea>';
                } else {
-                       $item .= "<input type='text' class='text' id='$name' name='$name' value='" . esc_attr( $field['value'] ) . "' $aria_required />";
+                       $item .= "<input type='text' class='text' id='$name' name='$name' value='" . esc_attr( $field['value'] ) . "'{$required_attr}{$aria_required} />";
                }
                if ( !empty( $field['helps'] ) )
                        $item .= "<p class='help'>" . join( "</p>\n<p class='help'>", array_unique( (array) $field['helps'] ) ) . '</p>';
@@ -1627,7 +1650,7 @@ function get_compat_media_markup( $attachment_id, $args = null ) {
        }
 
        // Merge default fields with their errors, so any key passed with the error (e.g. 'error', 'helps', 'value') will replace the default
-       // The recursive merge is easily traversed with array casting: foreach( (array) $things as $thing )
+       // The recursive merge is easily traversed with array casting: foreach ( (array) $things as $thing )
        $form_fields = array_merge_recursive($form_fields, (array) $args['errors'] );
 
        /** This filter is documented in wp-admin/includes/media.php */
@@ -1675,8 +1698,9 @@ function get_compat_media_markup( $attachment_id, $args = null ) {
                }
 
                $readonly      = ! $user_can_edit && ! empty( $field['taxonomy'] ) ? " readonly='readonly' " : '';
-               $required      = $field['required'] ? '<span class="alignright"><abbr title="required" class="required">*</abbr></span>' : '';
-               $aria_required = $field['required'] ? " aria-required='true' " : '';
+               $required      = $field['required'] ? '<span class="required">*</span>' : '';
+               $required_attr = $field['required'] ? ' required' : '';
+               $aria_required = $field['required'] ? " aria-required='true'" : '';
                $class  = 'compat-field-' . $id;
                $class .= $field['required'] ? ' form-required' : '';
 
@@ -1691,9 +1715,9 @@ function get_compat_media_markup( $attachment_id, $args = null ) {
                                // sanitize_post() skips the post_content when user_can_richedit.
                                $field['value'] = htmlspecialchars( $field['value'], ENT_QUOTES );
                        }
-                       $item .= "<textarea id='$id_attr' name='$name' $aria_required>" . $field['value'] . '</textarea>';
+                       $item .= "<textarea id='$id_attr' name='$name'{$required_attr}{$aria_required}>" . $field['value'] . '</textarea>';
                } else {
-                       $item .= "<input type='text' class='text' id='$id_attr' name='$name' value='" . esc_attr( $field['value'] ) . "' $readonly $aria_required />";
+                       $item .= "<input type='text' class='text' id='$id_attr' name='$name' value='" . esc_attr( $field['value'] ) . "' $readonly{$required_attr}{$aria_required} />";
                }
                if ( !empty( $field['helps'] ) )
                        $item .= "<p class='help'>" . join( "</p>\n<p class='help'>", array_unique( (array) $field['helps'] ) ) . '</p>';
@@ -1717,8 +1741,12 @@ function get_compat_media_markup( $attachment_id, $args = null ) {
 
        if ( !empty( $form_fields['_final'] ) )
                $item .= "\t\t<tr class='final'><td colspan='2'>{$form_fields['_final']}</td></tr>\n";
-       if ( $item )
-               $item = '<table class="compat-attachment-fields">' . $item . '</table>';
+
+       if ( $item ) {
+               $item = '<p class="media-types media-types-required-info">' .
+                       sprintf( __( 'Required fields are marked %s' ), '<span class="required">*</span>' ) . '</p>
+                       <table class="compat-attachment-fields">' . $item . '</table>';
+       }
 
        foreach ( $hidden_fields as $hidden_field => $value ) {
                $item .= '<input type="hidden" name="' . esc_attr( $hidden_field ) . '" value="' . esc_attr( $value ) . '" />' . "\n";
@@ -1734,7 +1762,7 @@ function get_compat_media_markup( $attachment_id, $args = null ) {
 }
 
 /**
- * {@internal Missing Short Description}}
+ * Outputs the legacy media upload header.
  *
  * @since 2.5.0
  */
@@ -1750,7 +1778,7 @@ function media_upload_header() {
 }
 
 /**
- * {@internal Missing Short Description}}
+ * Outputs the legacy media upload form.
  *
  * @since 2.5.0
  *
@@ -1819,7 +1847,7 @@ $post_params = array(
 );
 
 /**
- * Filter the media upload post parameters.
+ * Filters the media upload post parameters.
  *
  * @since 3.1.0 As 'swfupload_post_params'
  * @since 3.3.0
@@ -1852,7 +1880,7 @@ if ( wp_is_mobile() && strpos( $_SERVER['HTTP_USER_AGENT'], 'OS 7_' ) !== false
 }
 
 /**
- * Filter the default Plupload settings.
+ * Filters the default Plupload settings.
  *
  * @since 3.3.0
  *
@@ -1914,7 +1942,7 @@ do_action( 'post-plupload-upload-ui' ); ?>
        <p id="async-upload-wrap">
                <label class="screen-reader-text" for="async-upload"><?php _e('Upload'); ?></label>
                <input type="file" name="async-upload" id="async-upload" />
-               <?php submit_button( __( 'Upload' ), 'button', 'html-upload', false ); ?>
+               <?php submit_button( __( 'Upload' ), 'primary', 'html-upload', false ); ?>
                <a href="#" onclick="try{top.tb_remove();}catch(e){}; return false;"><?php _e('Cancel'); ?></a>
        </p>
        <div class="clear"></div>
@@ -1942,7 +1970,7 @@ do_action( 'post-html-upload-ui' );
 }
 
 /**
- * {@internal Missing Short Description}}
+ * Outputs the legacy media upload form for a given media type.
  *
  * @since 2.5.0
  *
@@ -1959,7 +1987,7 @@ function media_upload_type_form($type = 'file', $errors = null, $id = null) {
        $form_action_url = admin_url("media-upload.php?type=$type&tab=type&post_id=$post_id");
 
        /**
-        * Filter the media upload form action URL.
+        * Filters the media upload form action URL.
         *
         * @since 2.6.0
         *
@@ -2012,7 +2040,7 @@ if ( $id ) {
 }
 
 /**
- * {@internal Missing Short Description}}
+ * Outputs the legacy media upload form for external media.
  *
  * @since 2.7.0
  *
@@ -2096,7 +2124,7 @@ var addExtImage = {
                t.width = t.height = '';
                document.getElementById('go_button').style.color = '#bbb';
                if ( ! document.forms[0].src.value )
-                       document.getElementById('status_img').innerHTML = '*';
+                       document.getElementById('status_img').innerHTML = '';
                else document.getElementById('status_img').innerHTML = '<img src="<?php echo esc_url( admin_url( 'images/no.png' ) ); ?>" alt="" />';
        },
 
@@ -2139,7 +2167,7 @@ jQuery(document).ready( function($) {
 <div class="media-item media-blank">
 <?php
 /**
- * Filter the insert media from URL form HTML.
+ * Filters the insert media from URL form HTML.
  *
  * @since 3.3.0
  *
@@ -2303,7 +2331,7 @@ jQuery(function($){
 }
 
 /**
- * {@internal Missing Short Description}}
+ * Outputs the legacy media upload form for the media library.
  *
  * @since 2.5.0
  *
@@ -2392,7 +2420,7 @@ foreach ( $post_mime_types as $mime_type => $label ) {
        $type_links[] = '<li><a href="' . esc_url(add_query_arg(array('post_mime_type'=>$mime_type, 'paged'=>false))) . '"' . $class . '>' . sprintf( translate_nooped_plural( $label[2], $num_posts[$mime_type] ), '<span id="' . $mime_type . '-counter">' . number_format_i18n( $num_posts[$mime_type] ) . '</span>') . '</a>';
 }
 /**
- * Filter the media upload mime type list items.
+ * Filters the media upload mime type list items.
  *
  * Returned values should begin with an `<li>` tag.
  *
@@ -2526,21 +2554,21 @@ function wp_media_insert_url_form( $default_view = 'image' ) {
 
        return '
        <p class="media-types"><label><input type="radio" name="media_type" value="image" id="image-only"' . checked( 'image-only', $view, false ) . ' /> ' . __( 'Image' ) . '</label> &nbsp; &nbsp; <label><input type="radio" name="media_type" value="generic" id="not-image"' . checked( 'not-image', $view, false ) . ' /> ' . __( 'Audio, Video, or Other File' ) . '</label></p>
+       <p class="media-types media-types-required-info">' . sprintf( __( 'Required fields are marked %s' ), '<span class="required">*</span>' ) . '</p>
        <table class="describe ' . $table_class . '"><tbody>
                <tr>
                        <th scope="row" class="label" style="width:130px;">
-                               <label for="src"><span class="alignleft">' . __('URL') . '</span></label>
-                               <span class="alignright"><abbr id="status_img" title="required" class="required">*</abbr></span>
+                               <label for="src"><span class="alignleft">' . __( 'URL' ) . '</span> <span class="required">*</span></label>
+                               <span class="alignright" id="status_img"></span>
                        </th>
-                       <td class="field"><input id="src" name="src" value="" type="text" aria-required="true" onblur="addExtImage.getImageData()" /></td>
+                       <td class="field"><input id="src" name="src" value="" type="text" required aria-required="true" onblur="addExtImage.getImageData()" /></td>
                </tr>
 
                <tr>
                        <th scope="row" class="label">
-                               <label for="title"><span class="alignleft">' . __('Title') . '</span></label>
-                               <span class="alignright"><abbr title="required" class="required">*</abbr></span>
+                               <label for="title"><span class="alignleft">' . __( 'Title' ) . '</span> <span class="required">*</span></label>
                        </th>
-                       <td class="field"><input id="title" name="title" value="" type="text" aria-required="true" /></td>
+                       <td class="field"><input id="title" name="title" value="" type="text" required aria-required="true" /></td>
                </tr>
 
                <tr class="not-image"><td></td><td><p class="help">' . __('Link text, e.g. &#8220;Ransom Demands (PDF)&#8221;') . '</p></td></tr>
@@ -2673,6 +2701,8 @@ function multisite_over_quota_message() {
  * Displays the image and editor in the post editor
  *
  * @since 3.5.0
+ *
+ * @param WP_Post $post A post object.
  */
 function edit_form_image_editor( $post ) {
        $open = isset( $_GET['image-editor'] );
@@ -2686,7 +2716,7 @@ function edit_form_image_editor( $post ) {
        $alt_text = get_post_meta( $post->ID, '_wp_attachment_image_alt', true );
 
        $att_url = wp_get_attachment_url( $post->ID ); ?>
-       <div class="wp_attachment_holder">
+       <div class="wp_attachment_holder wp-clearfix">
        <?php
        if ( wp_attachment_is_image( $post->ID ) ) :
                $image_edit_button = '';
@@ -2698,7 +2728,7 @@ function edit_form_image_editor( $post ) {
 
                <div class="imgedit-response" id="imgedit-response-<?php echo $attachment_id; ?>"></div>
 
-               <div<?php if ( $open ) echo ' style="display:none"'; ?> class="wp_attachment_image" id="media-head-<?php echo $attachment_id; ?>">
+               <div<?php if ( $open ) echo ' style="display:none"'; ?> class="wp_attachment_image wp-clearfix" id="media-head-<?php echo $attachment_id; ?>">
                        <p id="thumbnail-head-<?php echo $attachment_id; ?>"><img class="thumbnail" src="<?php echo set_url_scheme( $thumb_url[0] ); ?>" style="max-width:100%" alt="" /></p>
                        <p><?php echo $image_edit_button; ?></p>
                </div>
@@ -2736,6 +2766,17 @@ function edit_form_image_editor( $post ) {
 
                echo wp_video_shortcode( $attr );
 
+       else :
+
+               /**
+                * Fires when an attachment type can't be rendered in the edit form.
+                *
+                * @since 4.6.0
+                *
+                * @param WP_Post $post A post object.
+                */
+               do_action( 'wp_edit_form_attachment_display', $post );
+
        endif; ?>
        </div>
        <div class="wp_attachment_details edit-form-section">
@@ -2763,7 +2804,7 @@ function edit_form_image_editor( $post ) {
                );
        ?>
 
-       <label for="content"><strong><?php _e( 'Description' ); ?></strong><?php
+       <label for="attachment_content"><strong><?php _e( 'Description' ); ?></strong><?php
        if ( preg_match( '#^(audio|video)/#', $post->post_mime_type ) ) {
                echo ': ' . __( 'Displayed on attachment pages.' );
        } ?></label>
@@ -2777,14 +2818,15 @@ function edit_form_image_editor( $post ) {
 }
 
 /**
- * Displays non-editable attachment metadata in the publish metabox
+ * Displays non-editable attachment metadata in the publish meta box.
  *
  * @since 3.5.0
  */
 function attachment_submitbox_metadata() {
        $post = get_post();
 
-       $filename = esc_html( wp_basename( $post->guid ) );
+       $file = get_attached_file( $post->ID );
+       $filename = esc_html( wp_basename( $file ) );
 
        $media_dims = '';
        $meta = wp_get_attachment_metadata( $post->ID );
@@ -2796,8 +2838,8 @@ function attachment_submitbox_metadata() {
        $att_url = wp_get_attachment_url( $post->ID );
 ?>
        <div class="misc-pub-section misc-pub-attachment">
-                       <label for="attachment_url"><?php _e( 'File URL:' ); ?></label>
-                       <input type="text" class="widefat urlfield" readonly="readonly" name="attachment_url" value="<?php echo esc_attr($att_url); ?>" />
+               <label for="attachment_url"><?php _e( 'File URL:' ); ?></label>
+               <input type="text" class="widefat urlfield" readonly="readonly" name="attachment_url" id="attachment_url" value="<?php echo esc_attr( $att_url ); ?>" />
        </div>
        <div class="misc-pub-section misc-pub-filename">
                <?php _e( 'File name:' ); ?> <strong><?php echo $filename; ?></strong>
@@ -2819,7 +2861,6 @@ function attachment_submitbox_metadata() {
        </div>
 
        <?php
-               $file  = get_attached_file( $post->ID );
                $file_size = false;
 
                if ( isset( $meta['filesize'] ) )
@@ -2837,7 +2878,7 @@ function attachment_submitbox_metadata() {
        if ( preg_match( '#^(audio|video)/#', $post->post_mime_type ) ) {
 
                /**
-                * Filter the audio and video metadata fields to be shown in the publish meta box.
+                * Filters the audio and video metadata fields to be shown in the publish meta box.
                 *
                 * The key for each item in the array should correspond to an attachment
                 * metadata key, and the value should be the desired label.
@@ -2875,7 +2916,7 @@ function attachment_submitbox_metadata() {
                }
 
                /**
-                * Filter the audio attachment metadata fields to be shown in the publish meta box.
+                * Filters the audio attachment metadata fields to be shown in the publish meta box.
                 *
                 * The key for each item in the array should correspond to an attachment
                 * metadata key, and the value should be the desired label.
@@ -2963,13 +3004,19 @@ function wp_add_id3_tag_data( &$metadata, $data ) {
  * @return array|bool Returns array of metadata, if found.
  */
 function wp_read_video_metadata( $file ) {
-       if ( ! file_exists( $file ) )
+       if ( ! file_exists( $file ) ) {
                return false;
+       }
 
        $metadata = array();
 
-       if ( ! class_exists( 'getID3' ) )
+       if ( ! defined( 'GETID3_TEMP_DIR' ) ) {
+               define( 'GETID3_TEMP_DIR', get_temp_dir() );
+       }
+
+       if ( ! class_exists( 'getID3', false ) ) {
                require( ABSPATH . WPINC . '/ID3/getid3.php' );
+       }
        $id3 = new getID3();
        $data = $id3->analyze( $file );
 
@@ -3019,12 +3066,18 @@ function wp_read_video_metadata( $file ) {
  * @return array|bool Returns array of metadata, if found.
  */
 function wp_read_audio_metadata( $file ) {
-       if ( ! file_exists( $file ) )
+       if ( ! file_exists( $file ) ) {
                return false;
+       }
        $metadata = array();
 
-       if ( ! class_exists( 'getID3' ) )
+       if ( ! defined( 'GETID3_TEMP_DIR' ) ) {
+               define( 'GETID3_TEMP_DIR', get_temp_dir() );
+       }
+
+       if ( ! class_exists( 'getID3', false ) ) {
                require( ABSPATH . WPINC . '/ID3/getid3.php' );
+       }
        $id3 = new getID3();
        $data = $id3->analyze( $file );
 
@@ -3068,7 +3121,7 @@ function wp_media_attach_action( $parent_id, $action = 'attach' ) {
        }
 
        if ( ! current_user_can( 'edit_post', $parent_id ) ) {
-               wp_die( __( 'You are not allowed to edit this post.' ) );
+               wp_die( __( 'Sorry, you are not allowed to edit this post.' ) );
        }
        $ids = array();
        foreach ( (array) $_REQUEST['media'] as $att_id ) {