Revert "Deprecate no longer used Skin::getCommonStylePath."
authorDaniel Friesen <pub-github@nadir-seen-fire.com>
Fri, 15 Jun 2012 01:08:29 +0000 (18:08 -0700)
committerDaniel Friesen <pub-github@nadir-seen-fire.com>
Fri, 15 Jun 2012 01:21:01 +0000 (18:21 -0700)
This reverts commit 79dcf7686d857e1dcbc3f1127a106d847de10d9e.

getCommonStylePath and getSkinStylePath are a pair of methods for use in 3rd party skins (NOT core).
They cut down on the boilerplate for including skin urls into skins for things like <img> tags.
Without these methods skin authors will be forced to revert to unintuitive boilerplate to add style paths and style versions into their code.
Most won't even bother to include the style version.
And ResourceLoader is a tangent to the whole topic because no matter what the advantages to CSS are developers writing custom skins for a single site from scratch will not care and they will choose CSS or <img> based on situations on hand in their own markup that make one or the other harder to handle.
This addition was part of the effort to fix the longstanding issue that our our skin system does not properly account for 3rd party skin developers.

Change-Id: I98f4fd83c93aba7681306d0a1edb58f798057543

RELEASE-NOTES-1.20
includes/Skin.php

index e222db7..7b53839 100644 (file)
@@ -161,7 +161,6 @@ changes to languages because of Bugzilla reports.
 * Removed ./tests/qunit/index.html from core. It wasn't actively maintained and
   has been made obsolete when [[Special:JavaScriptTest/qunit]] was introduced,
   which actually uses ResourceLoader, LocalSettings and the Skin.
-* Skin::getCommonStylePath has been deprecated. Its use is obsolete.
 
 == Compatibility ==
 
index a2d5e0f..8d47b83 100644 (file)
@@ -1033,16 +1033,11 @@ abstract class Skin extends ContextSource {
         * Return a fully resolved style path url to images or styles stored in the common folder.
         * This method returns a url resolved using the configured skin style path
         * and includes the style version inside of the url.
-        * @deprecated since 1.20: obsolete
-        *
         * @param $name String: The name or path of a skin resource file
         * @return String The fully resolved style path url including styleversion
         */
        function getCommonStylePath( $name ) {
                global $wgStylePath, $wgStyleVersion;
-
-               wfDeprecated( __METHOD__, '1.20' );
-
                return "$wgStylePath/common/$name?$wgStyleVersion";
        }