WordPress.org
Get WordPress

Make WordPress Core

Ticket #60415: 60415.10.patch

File 60415.10.patch, 13.0 KB (added by viralsampat, 2 years ago)

I have checked above mentioned issue and founds few files. Here, I have added its patch.

  • src/wp-includes/ID3/module.audio.flac.php 

    diff --git src/wp-includes/ID3/module.audio.flac.php src/wp-includes/ID3/module.audio.flac.php
index 014061da94..5655a5df99 100644
    class getid3_flac extends getid3_handler  
    6565                                break;
    6666                        }
    6767                        if ($BlockLength < 1) {
    68                                 if ($BlockTypeText != 'reserved') {
     68                                if ($BlockTypeText !== 'reserved') {
    6969                                        // probably supposed to be zero-length
    7070                                        $this->warning('METADATA_BLOCK_HEADER.BLOCK_LENGTH ('.$BlockTypeText.') at offset '.$BlockOffset.' is zero bytes');
    7171                                        continue;
    … … class getid3_flac extends getid3_handler  
    8282                        $BlockTypeText_raw['block_type']      = $BlockType;
    8383                        $BlockTypeText_raw['block_type_text'] = $BlockTypeText;
    8484                        $BlockTypeText_raw['block_length']    = $BlockLength;
    85                         if ($BlockTypeText_raw['block_type'] != 0x06) { // do not read attachment data automatically
     85                        if ($BlockTypeText_raw['block_type'] !== 0x06) { // do not read attachment data automatically
    8686                                $BlockTypeText_raw['block_data']  = $this->fread($BlockLength);
    8787                        }
    8888

  • src/wp-includes/ID3/module.tag.apetag.php 

    diff --git src/wp-includes/ID3/module.tag.apetag.php src/wp-includes/ID3/module.tag.apetag.php
index 1305cfb509..ba1c34ef63 100644
    class getid3_apetag extends getid3_handler  
    6969                } else {
    7070
    7171                        $this->fseek($this->overrideendoffset - $apetagheadersize);
    72                         if ($this->fread(8) == 'APETAGEX') {
     72                        if ($this->fread(8) === 'APETAGEX') {
    7373                                $info['ape']['tag_offset_end'] = $this->overrideendoffset;
    7474                        }
    7575

  • src/wp-includes/ID3/module.tag.id3v2.php 

    diff --git src/wp-includes/ID3/module.tag.id3v2.php src/wp-includes/ID3/module.tag.id3v2.php
index ec448be87b..ded91e54f0 100644
    class getid3_id3v2 extends getid3_handler  
    6060
    6161                $this->fseek($this->StartingOffset);
    6262                $header = $this->fread(10);
    63                 if (substr($header, 0, 3) == 'ID3'  &&  strlen($header) == 10) {
     63                if (substr($header, 0, 3) === 'ID3'  &&  strlen($header) === 10) {
    6464
    6565                        $thisfile_id3v2['majorversion'] = ord($header[3]);
    6666                        $thisfile_id3v2['minorversion'] = ord($header[4]);
    … … class getid3_id3v2 extends getid3_handler  
    186186                                        }
    187187                                        $extended_header_offset += $thisfile_id3v2['exthead']['padding_size'];
    188188
    189                                 } elseif ($id3v2_majorversion == 4) {
     189                                } elseif ($id3v2_majorversion === 4) {
    190190
    191191                                        // v2.4 definition:
    192192                                        //Extended header size   4 * %0xxxxxxx // 28-bit synchsafe integer
    … … class getid3_id3v2 extends getid3_handler  
    246246                                                $thisfile_id3v2['exthead']['flags']['restrictions_text']['imgsize']  = $this->LookupExtendedHeaderRestrictionsImageSizeSize($thisfile_id3v2['exthead']['flags']['restrictions']['imgsize']);
    247247                                        }
    248248
    249                                         if ($thisfile_id3v2['exthead']['length'] != $extended_header_offset) {
     249                                        if ($thisfile_id3v2['exthead']['length'] !== $extended_header_offset) {
    250250                                                $this->warning('ID3v2.4 extended header length mismatch (expecting '.intval($thisfile_id3v2['exthead']['length']).', found '.intval($extended_header_offset).')');
    251251                                        }
    252252                                }
    … … class getid3_id3v2 extends getid3_handler  
    263263                                        $thisfile_id3v2['padding']['length'] = strlen($framedata);
    264264                                        $thisfile_id3v2['padding']['valid']  = true;
    265265                                        for ($i = 0; $i < $thisfile_id3v2['padding']['length']; $i++) {
    266                                                 if ($framedata[$i] != "\x00") {
     266                                                if ($framedata[$i] !== "\x00") {
    267267                                                        $thisfile_id3v2['padding']['valid'] = false;
    268268                                                        $thisfile_id3v2['padding']['errorpos'] = $thisfile_id3v2['padding']['start'] + $i;
    269269                                                        $this->warning('Invalid ID3v2 padding found at offset '.$thisfile_id3v2['padding']['errorpos'].' (the remaining '.($thisfile_id3v2['padding']['length'] - $i).' bytes are considered invalid)');
    … … class getid3_id3v2 extends getid3_handler  
    276276                                $frame_name   = null;
    277277                                $frame_size   = null;
    278278                                $frame_flags  = null;
    279                                 if ($id3v2_majorversion == 2) {
     279                                if ($id3v2_majorversion === 2) {
    280280                                        // Frame ID  $xx xx xx (three characters)
    281281                                        // Size      $xx xx xx (24-bit integer)
    282282                                        // Flags     $xx xx
    … … class getid3_id3v2 extends getid3_handler  
    307307                                                $nextFrameID = substr($framedata, $frame_size, 4);
    308308                                                if ($this->IsValidID3v2FrameName($nextFrameID, $id3v2_majorversion)) {
    309309                                                        // next frame is OK
    310                                                 } elseif (($frame_name == "\x00".'MP3') || ($frame_name == "\x00\x00".'MP') || ($frame_name == ' MP3') || ($frame_name == 'MP3e')) {
     310                                                } elseif (($frame_name === "\x00".'MP3') || ($frame_name === "\x00\x00".'MP') || ($frame_name === ' MP3') || ($frame_name === 'MP3e')) {
    311311                                                        // MP3ext known broken frames - "ok" for the purposes of this test
    312312                                                } elseif (($id3v2_majorversion == 4) && ($this->IsValidID3v2FrameName(substr($framedata, getid3_lib::BigEndian2Int(substr($frame_header, 4, 4), 0), 4), 3))) {
    313313                                                        $this->warning('ID3v2 tag written as ID3v2.4, but with non-synchsafe integers (ID3v2.3 style). Older versions of (Helium2; iTunes) are known culprits of this. Tag has been parsed as ID3v2.3');
    … … class getid3_id3v2 extends getid3_handler  
    320320                                        $frame_flags = getid3_lib::BigEndian2Int(substr($frame_header, 8, 2));
    321321                                }
    322322
    323                                 if ((($id3v2_majorversion == 2) && ($frame_name == "\x00\x00\x00")) || ($frame_name == "\x00\x00\x00\x00")) {
     323                                if ((($id3v2_majorversion === 2) && ($frame_name === "\x00\x00\x00")) || ($frame_name === "\x00\x00\x00\x00")) {
    324324                                        // padding encountered
    325325
    326326                                        $thisfile_id3v2['padding']['start']  = $framedataoffset;
    … … class getid3_id3v2 extends getid3_handler  
    329329
    330330                                        $len = strlen($framedata);
    331331                                        for ($i = 0; $i < $len; $i++) {
    332                                                 if ($framedata[$i] != "\x00") {
     332                                                if ($framedata[$i] !== "\x00") {
    333333                                                        $thisfile_id3v2['padding']['valid'] = false;
    334334                                                        $thisfile_id3v2['padding']['errorpos'] = $thisfile_id3v2['padding']['start'] + $i;
    335335                                                        $this->warning('Invalid ID3v2 padding found at offset '.$thisfile_id3v2['padding']['errorpos'].' (the remaining '.($thisfile_id3v2['padding']['length'] - $i).' bytes are considered invalid)');
    … … class getid3_id3v2 extends getid3_handler  
    435435
    436436                if (isset($thisfile_id3v2_flags['isfooter']) && $thisfile_id3v2_flags['isfooter']) {
    437437                        $footer = $this->fread(10);
    438                         if (substr($footer, 0, 3) == '3DI') {
     438                        if (substr($footer, 0, 3) === '3DI') {
    439439                                $thisfile_id3v2['footer'] = true;
    440440                                $thisfile_id3v2['majorversion_footer'] = ord($footer[3]);
    441441                                $thisfile_id3v2['minorversion_footer'] = ord($footer[4]);
    … … class getid3_id3v2 extends getid3_handler  
    633633                }
    634634
    635635                if (!empty($parsedFrame['flags']['DataLengthIndicator'])) {
    636                         if ($parsedFrame['data_length_indicator'] != strlen($parsedFrame['data'])) {
     636                        if ($parsedFrame['data_length_indicator'] !== strlen($parsedFrame['data'])) {
    637637                                $this->warning('ID3v2 frame "'.$parsedFrame['frame_name'].'" should be '.$parsedFrame['data_length_indicator'].' bytes long according to DataLengthIndicator, but found '.strlen($parsedFrame['data']).' bytes of data');
    638638                        }
    639639                }
    … … class getid3_id3v2 extends getid3_handler  
    651651                        }
    652652                        $this->warning($warning);
    653653
    654                 } elseif ((($id3v2_majorversion >= 3) && ($parsedFrame['frame_name'] == 'UFID')) || // 4.1   UFID Unique file identifier
    655                         (($id3v2_majorversion == 2) && ($parsedFrame['frame_name'] == 'UFI'))) {  // 4.1   UFI  Unique file identifier
     654                } elseif ((($id3v2_majorversion >= 3) && ($parsedFrame['frame_name'] === 'UFID')) || // 4.1   UFID Unique file identifier
     655                        (($id3v2_majorversion == 2) && ($parsedFrame['frame_name'] === 'UFI'))) {  // 4.1   UFI  Unique file identifier
    656656                        //   There may be more than one 'UFID' frame in a tag,
    657657                        //   but only one with the same 'Owner identifier'.
    658658                        // <Header for 'Unique file identifier', ID: 'UFID'>
    … … class getid3_id3v2 extends getid3_handler  
    662662                        $parsedFrame['ownerid'] = (isset($exploded[0]) ? $exploded[0] : '');
    663663                        $parsedFrame['data']    = (isset($exploded[1]) ? $exploded[1] : '');
    664664
    665                 } elseif ((($id3v2_majorversion >= 3) && ($parsedFrame['frame_name'] == 'TXXX')) || // 4.2.2 TXXX User defined text information frame
    666                                 (($id3v2_majorversion == 2) && ($parsedFrame['frame_name'] == 'TXX'))) {    // 4.2.2 TXX  User defined text information frame
     665                } elseif ((($id3v2_majorversion >= 3) && ($parsedFrame['frame_name'] === 'TXXX')) || // 4.2.2 TXXX User defined text information frame
     666                                (($id3v2_majorversion == 2) && ($parsedFrame['frame_name'] === 'TXX'))) {    // 4.2.2 TXX  User defined text information frame
    667667                        //   There may be more than one 'TXXX' frame in each tag,
    668668                        //   but only one with the same description.
    669669                        // <Header for 'User defined text information frame', ID: 'TXXX'>
    … … class getid3_id3v2 extends getid3_handler  
    701701                        //unset($parsedFrame['data']); do not unset, may be needed elsewhere, e.g. for replaygain
    702702
    703703
    704                 } elseif ($parsedFrame['frame_name'][0] == 'T') { // 4.2. T??[?] Text information frame
     704                } elseif ($parsedFrame['frame_name'][0] === 'T') { // 4.2. T??[?] Text information frame
    705705                        //   There may only be one text information frame of its kind in an tag.
    706706                        // <Header for 'Text information frame', ID: 'T000' - 'TZZZ',
    707707                        // excluding 'TXXX' described in 4.2.6.>
    … … class getid3_id3v2 extends getid3_handler  
    754754                                unset($string, $wordsize, $i, $Txxx_elements, $Txxx_element, $Txxx_elements_start_offset);
    755755                        }
    756756
    757                 } elseif ((($id3v2_majorversion >= 3) && ($parsedFrame['frame_name'] == 'WXXX')) || // 4.3.2 WXXX User defined URL link frame
    758                                 (($id3v2_majorversion == 2) && ($parsedFrame['frame_name'] == 'WXX'))) {    // 4.3.2 WXX  User defined URL link frame
     757                } elseif ((($id3v2_majorversion >= 3) && ($parsedFrame['frame_name'] === 'WXXX')) || // 4.3.2 WXXX User defined URL link frame
     758                                (($id3v2_majorversion == 2) && ($parsedFrame['frame_name'] === 'WXX'))) {    // 4.3.2 WXX  User defined URL link frame
    759759                        //   There may be more than one 'WXXX' frame in each tag,
    760760                        //   but only one with the same description
    761761                        // <Header for 'User defined URL link frame', ID: 'WXXX'>
    … … class getid3_id3v2 extends getid3_handler  
    787787                        unset($parsedFrame['data']);
    788788
    789789
    790                 } elseif ($parsedFrame['frame_name'][0] == 'W') { // 4.3. W??? URL link frames
     790                } elseif ($parsedFrame['frame_name'][0] === 'W') { // 4.3. W??? URL link frames
    791791                        //   There may only be one URL link frame of its kind in a tag,
    792792                        //   except when stated otherwise in the frame description
    793793                        // <Header for 'URL link frame', ID: 'W000' - 'WZZZ', excluding 'WXXX'
    … … class getid3_id3v2 extends getid3_handler  
    801801                        unset($parsedFrame['data']);
    802802
    803803
    804                 } elseif ((($id3v2_majorversion == 3) && ($parsedFrame['frame_name'] == 'IPLS')) || // 4.4  IPLS Involved people list (ID3v2.3 only)
    805                                 (($id3v2_majorversion == 2) && ($parsedFrame['frame_name'] == 'IPL'))) {     // 4.4  IPL  Involved people list (ID3v2.2 only)
     804                } elseif ((($id3v2_majorversion == 3) && ($parsedFrame['frame_name'] === 'IPLS')) || // 4.4  IPLS Involved people list (ID3v2.3 only)
     805                                (($id3v2_majorversion == 2) && ($parsedFrame['frame_name'] === 'IPL'))) {     // 4.4  IPL  Involved people list (ID3v2.2 only)
    806806                        // http://id3.org/id3v2.3.0#sec4.4
    807807                        //   There may only be one 'IPL' frame in each tag
    808808                        // <Header for 'User defined URL link frame', ID: 'IPL'>
    … … class getid3_id3v2 extends getid3_handler  
    881881                        }
    882882
    883883
    884                 } elseif ((($id3v2_majorversion >= 3) && ($parsedFrame['frame_name'] == 'MCDI')) || // 4.4   MCDI Music CD identifier
    885                                 (($id3v2_majorversion == 2) && ($parsedFrame['frame_name'] == 'MCI'))) {     // 4.5   MCI  Music CD identifier
     884                } elseif ((($id3v2_majorversion >= 3) && ($parsedFrame['frame_name'] === 'MCDI')) || // 4.4   MCDI Music CD identifier
     885                                (($id3v2_majorversion == 2) && ($parsedFrame['frame_name'] === 'MCI'))) {     // 4.5   MCI  Music CD identifier
    886886                        //   There may only be one 'MCDI' frame in each tag
    887887                        // <Header for 'Music CD identifier', ID: 'MCDI'>
    888888                        // CD TOC                <binary data>
    … … class getid3_id3v2 extends getid3_handler  
    892892                        }
    893893
    894894
    895                 } elseif ((($id3v2_majorversion >= 3) && ($parsedFrame['frame_name'] == 'ETCO')) || // 4.5   ETCO Event timing codes
    896                                 (($id3v2_majorversion == 2) && ($parsedFrame['frame_name'] == 'ETC'))) {     // 4.6   ETC  Event timing codes
     895                } elseif ((($id3v2_majorversion >= 3) && ($parsedFrame['frame_name'] === 'ETCO')) || // 4.5   ETCO Event timing codes
     896                                (($id3v2_majorversion == 2) && ($parsedFrame['frame_name'] === 'ETC'))) {     // 4.6   ETC  Event timing codes
    897897                        //   There may only be one 'ETCO' frame in each tag
    898898                        // <Header for 'Event timing codes', ID: 'ETCO'>
    899899                        // Time stamp format    $xx

  • src/wp-includes/comment-template.php 

    diff --git src/wp-includes/comment-template.php src/wp-includes/comment-template.php
index d125f0c2dd..7bba06ba55 100644
    function get_comment_class( $css_class = '', $comment_id = null, $post = null )  
    552552        ++$comment_alt;
    553553
    554554        // Alt for top-level comments.
    555         if ( 1 == $comment_depth ) {
     555        if ( 1 === $comment_depth ) {
    556556                if ( $comment_thread_alt % 2 ) {
    557557                        $classes[] = 'thread-odd';
    558558                        $classes[] = 'thread-alt';