From: James D. Forrester <jforrester@wikimedia.org>
Date: Fri, 8 Feb 2019 01:02:15 +0000 (-0800)
Subject: Remove Title::escapeFragmentForURL, deprecated in 1.30, unused
X-Git-Tag: 1.34.0-rc.0~2893^2
X-Git-Url: https://git.heureux-cyclage.org/?a=commitdiff_plain;h=4e9741a71a96bcbe3ed9ce249bb4b43b3950c910;p=lhc%2Fweb%2Fwiklou.git

Remove Title::escapeFragmentForURL, deprecated in 1.30, unused

Change-Id: I649f3f1346c366e722fe4fd8690773bedce7ec54
---

diff --git a/RELEASE-NOTES-1.33 b/RELEASE-NOTES-1.33
index 168ccc2ba5..51734cfb18 100644
--- 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 c2d04c390d..9303a5cd5f 100644
--- 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)
 	 *