Ticket #39667: 39667-4.diff

src/wp-includes/functions.php

@@ -274 +274 @@
 }
 
 /**
+ * Function will return duration/ file length in human readable format.
+ * This will be used for making readable string from duration.
+ *
+ * @since 4.8
+ * @param string $filelength duration will be in string format (HH:ii:ss) OR (ii:ss).
+ * @return boolean | string return human readable string, false otherwise.
+ */
+function duration_format( $filelength = '' ) {
+
+        // return false if filelenght is empty or not in format.
+        if ( ( empty( $filelength ) && ! is_string( $filelength ) ) ) {
+
+                return false;
+        }
+
+        // validate filelenght format.
+        if ( ! ( (bool) preg_match( '/^(([0-3]?[0-9])|([2][0-3])):([0-5]?[0-9])(:([0-5]?[0-9]))?$/', $filelength ) ) ) {
+
+                return false;
+        }
+
+        $human_readable_duration = false;
+        // Extract duration.
+        $durations              = array_reverse( explode( ':', $filelength ) );
+        $duration_count = count( $durations );
+
+        // extract hours, minutes and seconds elseif extract minutes and senconds
+        if ( 3 === $duration_count ) {
+
+                list ( $second, $minute, $hour ) = $durations;
+        } elseif ( 2 === $duration_count ) {
+
+                list ( $second, $minute ) = $durations;
+        } else {
+
+                return false;
+        }
+
+        if ( ! empty( $hour ) && is_numeric( $hour ) ) {
+                /* translators: Time duration in hour or hours */
+                $human_readable_duration .= sprintf( _n( '%s hour', '%s hours', $hour ), $hour ) . ' ';
+        }
+
+        if ( ! empty( $minute ) && is_numeric( $minute ) ) {
+                /* translators: Time duration in minute or minutes */
+                $human_readable_duration .= sprintf( _n( '%s minute', '%s minutes', $minute ), $minute ) . ' ';
+        }
+
+        if ( ! empty( $second ) && is_numeric( $second ) ) {
+                /* translators: Time duration in second or seconds */
+                $human_readable_duration .= sprintf( _n( '%s second', '%s seconds', $second ), $second );
+        }
+        // return human readable duration
+        return $human_readable_duration;
+}
+
+/**
  * Get the week start and end from the datetime or date string from MySQL.
  *
  * @since 0.71

src/wp-includes/media-template.php

@@ -344 +344 @@
                                 <div class="file-size"><strong><?php _e( 'File size:' ); ?></strong> {{ data.filesizeHumanReadable }}</div>
                                 <# if ( 'image' === data.type && ! data.uploading ) { #>
                                         <# if ( data.width && data.height ) { #>
-                                                <div class="dimensions"><strong><?php _e( 'Dimensions:' ); ?></strong> {{ data.width }} &times; {{ data.height }}</div>
+                                                <div class="dimensions"><strong><?php _e( 'Dimensions:' ); ?></strong>
+                                                        <?php
+                                                        /* translators: 1: a number of pixels wide, 2: a number of pixels tall */
+                                                        printf( __( '%1$s by %2$s pixels' ),'{{ data.width }}', '{{ data.height }}' );
+                                                        ?>
+                                                </div>
                                         <# } #>
                                 <# } #>
 
-                                <# if ( data.fileLength ) { #>
-                                        <div class="file-length"><strong><?php _e( 'Length:' ); ?></strong> {{ data.fileLength }}</div>
+                                <# if ( data.fileLength && data.fileLengthHumanReadable ) { #>
+                                        <div class="file-length"><strong><?php _e( 'Length:' ); ?></strong>
+                                                <span aria-label="{{ data.fileLengthHumanReadable }}">{{ data.fileLength }}</span>
+                                        </div>
                                 <# } #>
 
                                 <# if ( 'audio' === data.type && data.meta.bitrate ) { #>

src/wp-includes/media.php

@@ -3219 +3219 @@
         }
 
         if ( $meta && ( 'audio' === $type || 'video' === $type ) ) {
-                if ( isset( $meta['length_formatted'] ) )
-                        $response['fileLength'] = $meta['length_formatted'];
+
+                if ( isset( $meta['length_formatted'] ) ) {
+                        $response['fileLength']              = $meta['length_formatted'];
+                        $response['fileLengthHumanReadable'] = duration_format( $meta['length_formatted'] );
+                }
 
                 $response['meta'] = array();
                 foreach ( wp_get_attachment_id3_keys( $attachment, 'js' ) as $key => $label ) {

new file tests/phpunit/tests/functions/durationFormat.php

@@ +1 @@
+<?php
+
+/**
+ * Tests for duration_format()
+ *
+ * @group functions.php
+ * @ticket 39667
+ */
+class Tests_Functions_Duration_Format extends WP_UnitTestCase {
+
+        public function _data_duration_format() {
+                return array(
+                        array( array(), false ),
+                        array( '30:00', '30 minutes 00 seconds' ),
+                        array( 'Batman Begins !', false ),
+                        array( '', false ),
+                        array( '-1', false ),
+                        array( -1, false ),
+                        array( 0, false ),
+                        array( 1, false ),
+                        array( '00', false ),
+                        array( '00:00', '00 minutes 00 seconds' ),
+                        array( '00:00:00', '00 hours 00 minutes 00 seconds' ),
+                        array( '10:30:34', '10 hours 30 minutes 34 seconds' ),
+                        array( '00:30:34', '00 hours 30 minutes 34 seconds' ),
+                        array( 'MM:30:00', false ),
+                        array( '30:MM', false ),
+                        array( 'MM:00', false ),
+                        array( 'MM:MM', false ),
+                        array( '01:01', '01 minute 01 second' ),
+                        array( '01:01:01', '01 hour 01 minute 01 second' ),
+                        array( '0:05', '05 seconds' ),
+                );
+        }
+
+        /**
+         * @dataProvider _data_duration_format
+         *
+         * @param $input
+         * @param $expected
+         */
+        public function test_duration_format( $input, $expected ) {
+                $this->assertSame( $expected, duration_format( $input ) );
+        }
+
+}