Changeset 2713

Timestamp:

03/21/10 23:09:57 (23 months ago)

Author:

nuxwin

Message:

• [GUI] Fixed #2266: Logical error in gui/client/alias_add.php

Location:

trunk

Files:

• CHANGELOG (modified) (1 diff)
• gui/client/alias_add.php (modified) (1 diff)

trunk/CHANGELOG

@@ -5 +5 @@
         - GUI
                 * Fixed #2195: Users gui properties not deleted
+                * Fixed #2266: Logical error in gui/client/alias_add.php
 
 2010-03-20 Laurent Declercq

trunk/gui/client/alias_add.php

@@ -153 +153 @@
         if (isset($_POST['status']) && $_POST['status'] == 1) {
                 $forward_prefix = clean_input($_POST['forward_prefix']);
-                if($_POST['status'] == 1) {
+
                         $check_en = 'checked="checked"';
                         $check_dis = '';
                         $forward = strtolower(clean_input($_POST['forward']));
+
                         $tpl->assign(
-                                        array(
-                                                'READONLY_FORWARD' => '',
-                                                'DISABLE_FORWARD' => '',
-                                                )
-                                        );
-                } else {
-                        $check_en = '';
-                        $check_dis = 'checked="checked"';
-                        $forward = '';
-                        $tpl->assign(
-                                        array(
-                                                'READONLY_FORWARD' => ' readonly',
-                                                'DISABLE_FORWARD' => ' disabled="disabled"',
-                                                )
-                                        );
-                }
-                $tpl->assign(
                                 array(
+                                        'READONLY_FORWARD' => '',
+                                        'DISABLE_FORWARD' => '',
                                         'HTTP_YES' => ($forward_prefix === 'http://') ? 'selected="selected"' : '',
                                         'HTTPS_YES' => ($forward_prefix === 'https://') ? 'selected="selected"' : '',
                                         'FTP_YES' => ($forward_prefix === 'ftp://') ? 'selected="selected"' : ''
-                                        )
-                                );
+                                )
+                        );
         } else {
                 $check_en = '';
                 $check_dis = 'checked="checked"';
                 $forward = '';
+
                 $tpl->assign(
-                                array(
-                                        'READONLY_FORWARD' => ' readonly',
-                                        'DISABLE_FORWARD' => ' disabled="disabled"',
-                                        )
-                                );
-        }
-        
+                        array(
+                                'READONLY_FORWARD' => ' readonly',
+                                'DISABLE_FORWARD' => ' disabled="disabled"',
+                                )
+                        );
+        }
+
         $tpl->assign(
                 array(