Nikerabbit has uploaded a new change for review.

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


Change subject: Fix Undefined offset notice
......................................................................

Fix Undefined offset notice

Bug: 45327
Change-Id: I1f70ef0ec4f2434f9f072e718140ff8050b81ba3
---
M data/LanguageNameSearch.php
1 file changed, 8 insertions(+), 5 deletions(-)


  git pull 
ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/UniversalLanguageSelector 
refs/changes/31/60831/1

diff --git a/data/LanguageNameSearch.php b/data/LanguageNameSearch.php
index b662afc..c1b237e 100644
--- a/data/LanguageNameSearch.php
+++ b/data/LanguageNameSearch.php
@@ -28,11 +28,17 @@
                if ( self::$languagenames === null ) {
                        self::init();
                }
+
+               // @todo: Shouldn't this be unicode aware?
                $searchKey = strtolower( $searchKey );
-               $bucket = self::$languagenames[self::getIndex( $searchKey )];
-               if ( !$bucket ) {
+               $index = self::getIndex( $searchKey );
+
+               if ( !self::$languagenames[$index] ) {
                        return array();
                }
+
+               $bucket = self::$languagenames[$index];
+
                $results = array();
                foreach ( $bucket as $name => $code ) {
                        // Prefix search
@@ -54,9 +60,6 @@
                        $bucket = $codepoint;
                } else {
                        $bucket = $codepoint % 1000;
-               }
-               if ( !isset( $buckets[$bucket] ) ) {
-                       $buckets[$bucket] = array();
                }
 
                return $bucket;

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I1f70ef0ec4f2434f9f072e718140ff8050b81ba3
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/UniversalLanguageSelector
Gerrit-Branch: master
Gerrit-Owner: Nikerabbit <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to