[geeklog-cvs] geeklog: Fixed saving of new elements in Geeklog Configuration (...

[geeklog-cvs at lists.geeklog.net]

changeset 8432:dcd07e580f65
url:  http://project.geeklog.net/cgi-bin/hgwebdir.cgi/geeklog/rev/dcd07e580f65
user: Tom <websitemaster at cogeco.net>
date: Thu Sep 29 20:34:05 2011 -0400
description:
Fixed saving of new elements in Geeklog Configuration (bug #0001396)

diffstat:

 system/classes/config.class.php |  25 ++++++++++++-------------
 1 files changed, 12 insertions(+), 13 deletions(-)

diffs (38 lines):

diff -r d2cb7ed068b2 -r dcd07e580f65 system/classes/config.class.php
--- a/system/classes/config.class.php	Mon Sep 26 13:19:41 2011 +0200
+++ b/system/classes/config.class.php	Thu Sep 29 20:34:05 2011 -0400
@@ -1322,22 +1322,21 @@
                 } else if ( is_array($change_array[$param_name]) ) {
                     /* if array such as mail settings */
                     $_changed = false;
-                    if (count($this->config_array[$group][$param_name]) !=
-                            count($change_array[$param_name])) {
+                    if (count($this->config_array[$group][$param_name]) !=  count($change_array[$param_name])) {
                         $_changed = true;
                     }
                     foreach ( $change_array[$param_name] as $_param_name => $_param_value ) {
-                        if  ( $change_array[$param_name][$_param_name] != 
-                              $this->config_array[$group][$param_name][$_param_name] )
-                        {
-                            $_changed = true;
-                            if ( $this->_validates($param_name . '[' . $_param_name . ']', 
-                                 $group, $change_array[$param_name][$_param_name], $change_array[$param_name]) )
-                            {
-                                $this->changedArray[$group][$param_name][$_param_name] = true;
-                            }
-                        }
-                    }
+                         if (!isset($this->config_array[$group][$param_name][$_param_name])) {
+                             $_changed = true;
+                         } elseif ( $change_array[$param_name][$_param_name] != $this->config_array[$group][$param_name][$_param_name] ) {
+                             $_changed = true;
+                         }
+                         if ($_changed) {
+                             if ( $this->_validates($param_name . '[' . $_param_name . ']', $group, $change_array[$param_name][$_param_name], $change_array[$param_name]) ) {
+                                 $this->changedArray[$group][$param_name][$_param_name] = true;
+                             }
+                         }
+                     }
                     
                     if ( $_changed ) {
                         $pass_validation[$param_name] = $change_array[$param_name];