From ecd107afb2a62274050d2233130ae2254ec9cc87 Mon Sep 17 00:00:00 2001 From: Tim Starling Date: Mon, 29 Aug 2011 04:01:40 +0000 Subject: [PATCH] In Language::isValidBuiltInCode(), reject the empty string per Nikerabbit's request on CR r82927. I'm not sure what the reason for the request was, but I suppose an empty language code could result in the base Language class being used, which could be problematic for some caller. It doesn't do any harm to reject it here. --- languages/Language.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/languages/Language.php b/languages/Language.php index 474fde02cd..49067be2cd 100644 --- a/languages/Language.php +++ b/languages/Language.php @@ -215,7 +215,7 @@ class Language { * @return bool */ public static function isValidBuiltInCode( $code ) { - return preg_match( '/^[a-z0-9-]*$/i', $code ); + return preg_match( '/^[a-z0-9-]+$/i', $code ); } public static function classFromCode( $code ) { -- 2.20.1