Remove redundant 'type' attributes from style and script tags
authorOri Livneh <ori@wikimedia.org>
Sat, 8 Aug 2015 03:39:56 +0000 (20:39 -0700)
committerTimo Tijhof <krinklemail@gmail.com>
Wed, 9 Dec 2015 20:07:13 +0000 (20:07 +0000)
Already stripped by Html::element before actual output, but
remove them from the attribute arrays as well.

Change-Id: I8699ca7bf40df07e9d4c370f6863003c095ced0e

includes/Html.php
includes/installer/MysqlInstaller.php
resources/src/mediawiki.legacy/wikibits.js

index b7958dd..5f91dac 100644 (file)
@@ -359,8 +359,6 @@ class Html {
                        'keygen' => array( 'keytype' => 'rsa' ),
                        'link' => array( 'media' => 'all' ),
                        'menu' => array( 'type' => 'list' ),
-                       // Note: the use of text/javascript here instead of other JavaScript
-                       // MIME types follows the HTML5 spec.
                        'script' => array( 'type' => 'text/javascript' ),
                        'style' => array(
                                'media' => 'all',
@@ -674,7 +672,6 @@ class Html {
                }
 
                return self::rawElement( 'style', array(
-                       'type' => 'text/css',
                        'media' => $media,
                ), $contents );
        }
@@ -691,7 +688,6 @@ class Html {
                return self::element( 'link', array(
                        'rel' => 'stylesheet',
                        'href' => $url,
-                       'type' => 'text/css',
                        'media' => $media,
                ) );
        }
index e5c1a1d..cbe51f5 100644 (file)
@@ -354,7 +354,7 @@ class MysqlInstaller extends DatabaseInstaller {
                $s .= Xml::closeElement( 'div' );
 
                if ( $this->getVar( '_MysqlEngine' ) != 'MyISAM' ) {
-                       $s .= Xml::openElement( 'script', array( 'type' => 'text/javascript' ) );
+                       $s .= Xml::openElement( 'script' );
                        $s .= '$(\'#dbMyisamWarning\').hide();';
                        $s .= Xml::closeElement( 'script' );
                }
index 7d1f6d7..38271a0 100644 (file)
                loadedScripts[ url ] = true;
                var s = document.createElement( 'script' );
                s.setAttribute( 'src', url );
-               s.setAttribute( 'type', 'text/javascript' );
                document.getElementsByTagName( 'head' )[ 0 ].appendChild( s );
                return s;
        }