Siebrand has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/104033


Change subject: Remove long deprecated methods from Linker
......................................................................

Remove long deprecated methods from Linker

- Linker::makeBrokenLink() (deprecated in 1.16)
- Linker::makeBrokenLinkObj() (deprecated in 1.16)
- Linker::makeColouredLinkObj() (deprecated in 1.16)
- Linker::makeSizeLinkObj() (deprecated in 1.17)

Change-Id: I230096cbf964cdff980f33f76790a524d318408d
---
M RELEASE-NOTES-1.23
M includes/Linker.php
2 files changed, 5 insertions(+), 106 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core 
refs/changes/33/104033/1

diff --git a/RELEASE-NOTES-1.23 b/RELEASE-NOTES-1.23
index 6078234..b933a31 100644
--- a/RELEASE-NOTES-1.23
+++ b/RELEASE-NOTES-1.23
@@ -100,6 +100,11 @@
   - LoadBalancer_Single to LoadBalancerSingle
   - LoadMonitor_MySQL to LoadMonitorMySQL
   - LoadMonitor_Null to LoadMonitorNull
+* Removed methods:
+  - Linker::makeBrokenLink() (deprecated in 1.16)
+  - Linker::makeBrokenLinkObj() (deprecated in 1.16)
+  - Linker::makeColouredLinkObj() (deprecated in 1.16)
+  - Linker::makeSizeLinkObj() (deprecated in 1.17)
 
 === Languages updated in 1.23 ===
 
diff --git a/includes/Linker.php b/includes/Linker.php
index f303a17..27f8ab4 100644
--- a/includes/Linker.php
+++ b/includes/Linker.php
@@ -394,29 +394,6 @@
        }
 
        /**
-        * Generate either a normal exists-style link or a stub link, depending
-        * on the given page size.
-        *
-        * @param $size Integer
-        * @param $nt Title object.
-        * @param $text String
-        * @param $query String
-        * @param $trail String
-        * @param $prefix String
-        * @return string HTML of link
-        * @deprecated since 1.17
-        */
-       static function makeSizeLinkObj( $size, $nt, $text = '', $query = '', 
$trail = '', $prefix = '' ) {
-               global $wgUser;
-               wfDeprecated( __METHOD__, '1.17' );
-
-               $threshold = $wgUser->getStubThreshold();
-               $colour = ( $size < $threshold ) ? 'stub' : '';
-               // @todo FIXME: Replace deprecated makeColouredLinkObj by link()
-               return self::makeColouredLinkObj( $nt, $colour, $text, $query, 
$trail, $prefix );
-       }
-
-       /**
         * Make appropriate markup for a link to the current article. This is 
currently rendered
         * as the bold link text. The calling sequence is the same as the other 
make*LinkObj static functions,
         * despite $query not being used.
@@ -2244,32 +2221,6 @@
        /* Deprecated methods */
 
        /**
-        * @deprecated since 1.16 Use link()
-        *
-        * This function is a shortcut to 
makeBrokenLinkObj(Title::newFromText($title),...). Do not call
-        * it if you already have a title object handy. See makeBrokenLinkObj 
for further documentation.
-        *
-        * @param string $title The text of the title
-        * @param string $text Link text
-        * @param string $query Optional query part
-        * @param string $trail Optional trail. Alphabetic characters at the 
start of this string will
-        *               be included in the link text. Other characters will be 
appended after
-        *               the end of the link.
-        * @return string
-        */
-       static function makeBrokenLink( $title, $text = '', $query = '', $trail 
= '' ) {
-               wfDeprecated( __METHOD__, '1.16' );
-
-               $nt = Title::newFromText( $title );
-               if ( $nt instanceof Title ) {
-                       return self::makeBrokenLinkObj( $nt, $text, $query, 
$trail );
-               } else {
-                       wfDebug( 'Invalid title passed to 
self::makeBrokenLink(): "' . $title . "\"\n" );
-                       return $text == '' ? $title : $text;
-               }
-       }
-
-       /**
         * @deprecated since 1.16 Use link(); warnings since 1.21
         *
         * Make a link for a title which may or may not be in the database. If 
you need to
@@ -2340,63 +2291,6 @@
 
                wfProfileOut( __METHOD__ );
                return $ret;
-       }
-
-       /**
-        * @deprecated since 1.16 Use link()
-        *
-        * Make a red link to the edit page of a given title.
-        *
-        * @param $title Title object of the target page
-        * @param $text  String: Link text
-        * @param string $query Optional query part
-        * @param string $trail Optional trail. Alphabetic characters at the 
start of this string will
-        *                      be included in the link text. Other characters 
will be appended after
-        *                      the end of the link.
-        * @param string $prefix Optional prefix
-        * @return string
-        */
-       static function makeBrokenLinkObj( $title, $text = '', $query = '', 
$trail = '', $prefix = '' ) {
-               wfDeprecated( __METHOD__, '1.16' );
-
-               wfProfileIn( __METHOD__ );
-
-               list( $inside, $trail ) = self::splitTrail( $trail );
-               if ( $text === '' ) {
-                       $text = self::linkText( $title );
-               }
-
-               $ret = self::link( $title, "$prefix$text$inside", array(),
-                       wfCgiToArray( $query ), 'broken' ) . $trail;
-
-               wfProfileOut( __METHOD__ );
-               return $ret;
-       }
-
-       /**
-        * @deprecated since 1.16 Use link()
-        *
-        * Make a coloured link.
-        *
-        * @param $nt Title object of the target page
-        * @param $colour Integer: colour of the link
-        * @param $text   String:  link text
-        * @param $query  String:  optional query part
-        * @param $trail  String:  optional trail. Alphabetic characters at the 
start of this string will
-        *                      be included in the link text. Other characters 
will be appended after
-        *                      the end of the link.
-        * @param string $prefix Optional prefix
-        * @return string
-        */
-       static function makeColouredLinkObj( $nt, $colour, $text = '', $query = 
'', $trail = '', $prefix = '' ) {
-               wfDeprecated( __METHOD__, '1.16' );
-
-               if ( $colour != '' ) {
-                       $style = self::getInternalLinkAttributesObj( $nt, 
$text, $colour );
-               } else {
-                       $style = '';
-               }
-               return self::makeKnownLinkObj( $nt, $text, $query, $trail, 
$prefix, '', $style );
        }
 
        /**

-- 
To view, visit https://gerrit.wikimedia.org/r/104033
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I230096cbf964cdff980f33f76790a524d318408d
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Siebrand <siebr...@wikimedia.org>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to