Check for string initialization in lcfirst() for HHVM 3.18
authorAntoine Musso <hashar@free.fr>
Fri, 24 Mar 2017 14:19:36 +0000 (15:19 +0100)
committerAntoine Musso <hashar@free.fr>
Fri, 24 Mar 2017 14:19:36 +0000 (15:19 +0100)
HHVM 3.18 emits a notice when attempting to access the first offset of
an empty string.  We had that fixed for ucfirst() in 3605066c96. This is
the same for lcfirst().

Bug: T161095
Change-Id: I1456611222c24290f259298e883ca89dd830c74b

languages/classes/LanguageKk.php

index 0460a6c..3a50987 100644 (file)
@@ -408,7 +408,7 @@ class LanguageKk extends LanguageKk_cyrl {
         * @return string
         */
        function lcfirst( $string ) {
-               if ( $string[0] == 'I' ) {
+               if ( substr( $string, 0, 1 ) === 'I' ) {
                        $variant = $this->getPreferredVariant();
                        if ( $variant == 'kk-latn' || $variant == 'kk-tr' ) {
                                return 'ı' . substr( $string, 1 );