Remove Title::escapeFragmentForURL, deprecated in 1.30, unused

Change-Id: I649f3f1346c366e722fe4fd8690773bedce7ec54

diff --git a/RELEASE-NOTES-1.33 b/RELEASE-NOTES-1.33
index 168ccc2..51734cf 100644 (file)
--- a/RELEASE-NOTES-1.33
+++ b/RELEASE-NOTES-1.33
@@ -193,6 +193,8 @@ because of Phabricator reports.
 * Class SquidUpdate, deprecated in 1.27, was removed.
 * Title->getSquidURLs(), deprecated in 1.27, was removed. Instead, use
   Title->getCdnUrls().
+* Title::escapeFragmentForURL(), deprecated in 1.30, was removed. Use
+  Sanitizer::escapeIdForLink() or escapeIdForExternalInterwiki() instead.
 
 === Deprecations in 1.33 ===
 * The configuration option $wgUseESI has been deprecated, and is expected

diff --git a/includes/Title.php b/includes/Title.php
index c2d04c3..9303a5c 100644 (file)
--- a/includes/Title.php
+++ b/includes/Title.php
@@ -767,23 +767,6 @@ class Title implements LinkTarget, IDBAccessObject {
                return $name;
        }
 
-       /**
-        * Escape a text fragment, say from a link, for a URL
-        *
-        * @deprecated since 1.30, use Sanitizer::escapeIdForLink() or escapeIdForExternalInterwiki()
-        *
-        * @param string $fragment Containing a URL or link fragment (after the "#")
-        * @return string Escaped string
-        */
-       static function escapeFragmentForURL( $fragment ) {
-               wfDeprecated( __METHOD__, '1.30' );
-               # Note that we don't urlencode the fragment.  urlencoded Unicode
-               # fragments appear not to work in IE (at least up to 7) or in at least
-               # one version of Opera 9.x.  The W3C validator, for one, doesn't seem
-               # to care if they aren't encoded.
-               return Sanitizer::escapeId( $fragment, 'noninitial' );
-       }
-
        /**
         * Callback for usort() to do title sorts by (namespace, title)
         *