Merge pull request #599 from OPUS4/issue598

#598 Trim configured 'supportedLanguages'

Author: j3nsch

library/Application/Configuration.php

@@ -125,7 +125,7 @@ public function getSupportedLanguages()
         if (is_null($this->supportedLanguages)) {
             $config = $this->getConfig();
             if (isset($config->supportedLanguages)) {
-                $this->supportedLanguages = explode(',', $config->supportedLanguages);
+                $this->supportedLanguages = array_map('trim', explode(',', $config->supportedLanguages));
                 /* TODO only used for debugging - remove?
                 $this->getLogger()->debug(
                     Zend_Debug::dump(

tests/library/Application/ConfigurationTest.php

@@ -124,6 +124,15 @@ public function testIsLanguageSelectionEnabledTrue()
         $this->assertTrue($this->config->isLanguageSelectionEnabled());
     }
 
+    public function testGetSupportedLanguagesValuesAreTrimmed()
+    {
+        $this->adjustConfiguration([
+            'supportedLanguages' => 'en, de'
+        ]);
+
+        $this->assertEquals(['en', 'de'], $this->config->getSupportedLanguages());
+    }
+
     public function testIsLanguageSelectionEnabledFalse()
     {
         Config::get()->supportedLanguages = 'de';