[mms] Fix Horde_Prefs#isDefault() to return true if the given pref na…

…me doesn't exist in a scope. Since the pref is not defined, whatever value given should be considered the default. Ticket #13732
horde · Dec 3, 2014 · 690d731 · 690d731
1 parent 20c98f3
commit 690d731
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/framework/Prefs/lib/Horde/Prefs.php b/framework/Prefs/lib/Horde/Prefs.php
@@ -330,7 +330,7 @@ public function isDefault($pref)
     {
         return ($scope = $this->_getScope($pref))
             ? $this->_scopes[$scope]->isDefault($pref)
-            : false;
+            : true;
     }
 
     /**

diff --git a/framework/Prefs/package.xml b/framework/Prefs/package.xml
@@ -33,7 +33,7 @@
  </stability>
  <license uri="http://www.horde.org/licenses/lgpl21">LGPL-2.1</license>
  <notes>
-* 
+* [mms] Fix Horde_Prefs#isDefault() to return true if the given pref name doesn&apos;t exist in a scope.
  </notes>
  <contents>
   <dir baseinstalldir="/" name="/">
@@ -1130,7 +1130,7 @@ Initial release as a PEAR package
    <date>2014-10-21</date>
    <license uri="http://www.horde.org/licenses/lgpl21">LGPL-2.1</license>
    <notes>
-* 
+* [mms] Fix Horde_Prefs#isDefault() to return true if the given pref name doesn&apos;t exist in a scope.
    </notes>
   </release>
  </changelog>