From 2dde55a1aa48d980ddb8cb5dde5128adf02db4e5 Mon Sep 17 00:00:00 2001
From: dheeyi <dheeyi@gmail.com>
Date: Wed, 11 Feb 2015 17:04:04 -0400
Subject: [PATCH] PM-1399 En la configuracion de "Enviroment" no se aplica el
 formato para el User Name.
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Causa. Esto fue debido a los siguientes motivos.

		Inicialmente si funcionaba... se realizo un leve cambio parecida a la siguiente
		instrucció
		$this->UserConfig = array(); /* Esto no existia en un principio y la logica de funcionamiento era el correcto*/

        if (!isset($this->UserConfig)) {
            $this->UserConfig = $this->getConfiguration("ENVIRONMENT_SETTINGS", "");
        }

        Sin enbargo al adicionar esa linea(Motivo Warnings de PHP), el if que se menciona tubo otra logica de trabajo es por eso que no se reflejaba los cambios realizados por el usuario.

Solució	Lo que se realizo fue cambiar la instruccion isset por empty, de la siguiente 			manera.

			if (empty($this->UserConfig)) {
            $this->UserConfig = $this->getConfiguration("ENVIRONMENT_SETTINGS", "");
       		}

       		Con esto fue resuelto el error.
       		Nota.- A medida que se corrigio este error, tambien fue corregido los Notices PHP generados(Index Indefinido) y asi tener mas limpio el codigo.

       		Ejem.

       		Estado Inicial >>  $config['format'] = $_POST["userFormat"];
       		Estado final   >>  if (isset($_POST["dateFormat"])) {
						            $config['dateFormat'] = $_POST["dateFormat"];
						        }
---
 .../engine/classes/class.configuration.php    |  2 +-
 .../methods/setup/environmentSettingsAjax.php | 28 ++++++++++++++-----
 2 files changed, 22 insertions(+), 8 deletions(-)

diff --git a/workflow/engine/classes/class.configuration.php b/workflow/engine/classes/class.configuration.php
index 5ec6c2346..917c260ba 100755
--- a/workflow/engine/classes/class.configuration.php
+++ b/workflow/engine/classes/class.configuration.php
@@ -374,7 +374,7 @@ class Configurations // extends Configuration
     public function getFormats()
     {
     	$this->UserConfig = array();
-        if (!isset($this->UserConfig)) {
+        if (empty($this->UserConfig)) {
             $this->UserConfig = $this->getConfiguration("ENVIRONMENT_SETTINGS", "");
         }
 
diff --git a/workflow/engine/methods/setup/environmentSettingsAjax.php b/workflow/engine/methods/setup/environmentSettingsAjax.php
index e03a107b0..d888d6159 100755
--- a/workflow/engine/methods/setup/environmentSettingsAjax.php
+++ b/workflow/engine/methods/setup/environmentSettingsAjax.php
@@ -5,7 +5,6 @@
  * @date Sept 13th, 2010
  *
  */
-
 G::LoadClass( "configuration" );
 
 $request = isset( $_POST["request"] ) ? $_POST["request"] : (isset( $_GET["request"] ) ? $_GET["request"] : null);
@@ -36,17 +35,32 @@ switch ($request) {
     case "save":
         $conf = new Configurations();
         $config = $conf->getConfiguration("ENVIRONMENT_SETTINGS", "" );
-        $config['format'] = $_POST["userFormat"];
-        $config['dateFormat'] = $_POST["dateFormat"];
+        //$config['format'] = $_POST["userFormat"];
+        if (isset($_POST["userFormat"])) {
+            $config['format'] = $_POST["userFormat"]; 
+        }        
+        //$config['dateFormat'] = $_POST["dateFormat"];
+        if (isset($_POST["dateFormat"])) {
+            $config['dateFormat'] = $_POST["dateFormat"]; 
+        }
         $config['startCaseHideProcessInf'] = ((isset( $_POST["hideProcessInf"] )) ? true : false);
-        $config['casesListDateFormat'] = $_POST["casesListDateFormat"];
-        $config['casesListRowNumber'] = intval( $_POST["casesListRowNumber"] );
-        $config['casesListRefreshTime'] = intval( $_POST["txtCasesRefreshTime"]);
+        //$config['casesListDateFormat'] = $_POST["casesListDateFormat"];
+        if (isset($_POST["casesListDateFormat"])) {
+            $config['casesListDateFormat'] = $_POST["casesListDateFormat"]; 
+        }        
+        //$config['casesListRowNumber'] = intval( $_POST["casesListRowNumber"] );
+        if (isset($_POST["casesListDateFormat"])) {
+            $config['casesListRowNumber'] = intval( $_POST["casesListRowNumber"] );
+        }
+        //$config['casesListRefreshTime'] = intval( $_POST["txtCasesRefreshTime"]);
+        if (isset($_POST["txtCasesRefreshTime"])) {
+            $config['casesListRefreshTime'] = intval( $_POST["txtCasesRefreshTime"]);
+        }
 
         $conf->aConfig = $config;
         $conf->saveConfig( "ENVIRONMENT_SETTINGS", "" );
 
-        G::auditLog("UpdateEnvironmentSettings", "UserNameDisplayFormat -> ".$_POST["userFormat"].", GlobalDateFormat -> ".$_POST["dateFormat"].", HideProcessInformation -> ".(string)isset($_POST["hideProcessInf"]).", DateFormat -> ".$_POST["casesListDateFormat"].", NumberOfRowsPerPage -> ".$_POST["casesListRowNumber"].", RefreshTimeSeconds -> ".$_POST["txtCasesRefreshTime"]);
+        G::auditLog("UpdateEnvironmentSettings", "UserNameDisplayFormat -> ".isset($_POST["userFormat"]).", GlobalDateFormat -> ".isset($_POST["dateFormat"]).", HideProcessInformation -> ".(string)isset($_POST["hideProcessInf"]).", DateFormat -> ".isset($_POST["casesListDateFormat"]).", NumberOfRowsPerPage -> ".isset($_POST["casesListRowNumber"]).", RefreshTimeSeconds -> ".isset($_POST["txtCasesRefreshTime"]));
 
         $response = new stdclass();
         $response->success = true;