Add 2 FIXMEs about confusingly named functions
authorKaldari <rkaldari@wikimedia.org>
Tue, 9 Feb 2016 01:29:09 +0000 (19:29 -0600)
committerKaldari <rkaldari@wikimedia.org>
Sat, 13 Feb 2016 00:27:36 +0000 (18:27 -0600)
With the roll-out of the Gadget namespace, these are going to be
especially misleading and likely to cause bugs.

Change-Id: I61f1f56762711fab173bbc54d110a7c48604d464

includes/Title.php

index 55c7179..da0efee 100644 (file)
@@ -1261,6 +1261,7 @@ class Title implements LinkTarget {
         * show "inactive" CSS or JS.
         *
         * @return bool
+        * @todo FIXME: Rename to isSiteConfigPage() and remove deprecated hook
         */
        public function isCssOrJsPage() {
                $isCssOrJsPage = NS_MEDIAWIKI == $this->mNamespace
@@ -1279,6 +1280,7 @@ class Title implements LinkTarget {
        /**
         * Is this a .css or .js subpage of a user page?
         * @return bool
+        * @todo FIXME: Rename to isUserConfigPage()
         */
        public function isCssJsSubpage() {
                return ( NS_USER == $this->mNamespace && $this->isSubpage()