Tim Starling has submitted this change and it was merged.

Change subject: Revert "(bug 29788) Swedish Collation (uppercase-sv). Swaps Ä 
and Æ"
......................................................................


Revert "(bug 29788) Swedish Collation (uppercase-sv). Swaps Ä and Æ"

This workaround is unnecessary now that I838484b9 was merged.

This reverts commit 13dc8ff88f48e0bb18750f5e98d50cdb8057c0d3.

Change-Id: I2cd22ad87eb7a56c5742b20c6089a4b8607e5614
---
M RELEASE-NOTES-1.21
M includes/AutoLoader.php
M includes/Collation.php
3 files changed, 0 insertions(+), 21 deletions(-)

Approvals:
  Tim Starling: Verified; Looks good to me, approved
  jenkins-bot: Checked



diff --git a/RELEASE-NOTES-1.21 b/RELEASE-NOTES-1.21
index 3737932..cbfafa2 100644
--- a/RELEASE-NOTES-1.21
+++ b/RELEASE-NOTES-1.21
@@ -96,8 +96,6 @@
 * (bug 5346) Categories that are redirects will be displayed italic in
   the category links section at the bottom of a page.
 * (bug 43915) New maintenance script deleteEqualMessages.php.
-* New collation uppercase-sv, which is like uppercase, but adapted
-  to Swedish sort order.
 * WikiText now permits the use of WAI-ARIA's role="presentation" inside of
   html elements and tables. This allows presentational markup, especially
   tables. To be marked up as such.
diff --git a/includes/AutoLoader.php b/includes/AutoLoader.php
index 5b64e65..e1caddc 100644
--- a/includes/AutoLoader.php
+++ b/includes/AutoLoader.php
@@ -258,7 +258,6 @@
        'UnlistedSpecialPage' => 'includes/SpecialPage.php',
        'UploadSourceAdapter' => 'includes/Import.php',
        'UppercaseCollation' => 'includes/Collation.php',
-       'UppercaseSvCollation' => 'includes/Collation.php',
        'User' => 'includes/User.php',
        'UserArray' => 'includes/UserArray.php',
        'UserArrayFromResult' => 'includes/UserArray.php',
diff --git a/includes/Collation.php b/includes/Collation.php
index f57a14a..87afc10 100644
--- a/includes/Collation.php
+++ b/includes/Collation.php
@@ -43,8 +43,6 @@
                switch( $collationName ) {
                        case 'uppercase':
                                return new UppercaseCollation;
-                       case 'uppercase-sv':
-                               return new UppercaseSvCollation;
                        case 'identity':
                                return new IdentityCollation;
                        case 'uca-default':
@@ -124,22 +122,6 @@
                        $string = substr( $string, 1 );
                }
                return $this->lang->ucfirst( $this->lang->firstChar( $string ) 
);
-       }
-}
-
-/**
- * Like UppercaseCollation but swaps Ä and Æ.
- *
- * This provides an ordering suitable for Swedish.
- * @author Lejonel
- */
-class UppercaseSvCollation extends UppercaseCollation {
-
-       /* Unicode code point order is ÄÅÆÖ, Swedish order is ÅÄÖ and Æ is 
often sorted as Ä.
-        * Replacing Ä for Æ should give a better collation. */
-       function getSortKey( $string ) {
-               $uppercase = $this->lang->uc( $string );
-               return strtr( $uppercase, array( 'Ä' => 'Æ', 'Æ' => 'Ä' ) );
        }
 }
 

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I2cd22ad87eb7a56c5742b20c6089a4b8607e5614
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Matmarex <matma....@gmail.com>
Gerrit-Reviewer: Brian Wolff <bawolff...@gmail.com>
Gerrit-Reviewer: Tim Starling <tstarl...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot

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

Reply via email to