Merge pull request #23 from OXIDprojects/keywan-ghadami-oxid-patch-3

fix export bool
OXIDprojects · Jan 14, 2019 · 2c52565 · 2c52565
2 parents 4040be7 + d0a8b4e
commit 2c52565
Showing 1 changed file with 2 additions and 7 deletions.
diff --git a/Core/ConfigExport.php b/Core/ConfigExport.php
@@ -256,11 +256,6 @@ protected function withoutDefaults(&$aGroupedValues)
                     }
                     if (count($aModuleConfig) == 0) {
                         unset($aModuleConfigs[$sModuleId]);
-                    } else {
-                        if ($sDefaultType == 'bool') {
-                            $mCurrentValue = (bool) $mCurrentValue;
-                            $aModuleConfig[$sVarName] = $mCurrentValue;
-                        }
                     }
                 }
             }
@@ -670,9 +665,9 @@ private function varValueWithThemeDisplayInfo($sVarName, $mVarValue, $sVarType,
     protected function convertToBool($mDefaultValue)
     {
         if ($mDefaultValue === 'false') {
-            $mDefaultValue = '';
+            $mDefaultValue = false;
         } else {
-            $mDefaultValue = $mDefaultValue ? '1' : '';
+            $mDefaultValue = $mDefaultValue ? true : false;
         }
         return $mDefaultValue;
     }