fernando/luos
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Causa: Esto fue debido a los siguientes motivos,inicialmente si funcionaba... se realizo un leve cambio parecida a la siguiente instrucció

Browse Source 
$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"];
                            }
This commit is contained in:
dheeyi
2015-02-12 13:07:01 -04:00
parent 9f8be84cae
commit e932246469
2 changed files with 4 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
workflow/engine/classes/class.configuration.php
View File

@@ -373,7 +373,9 @@ class Configurations // extends Configuration
*/
public function getFormats()
{
$this->UserConfig = array();
if (!isset($this->UserConfig)) {
$this->UserConfig = array();
}
if (empty($this->UserConfig)) {
$this->UserConfig = $this->getConfiguration("ENVIRONMENT_SETTINGS", "");
}

2
workflow/engine/methods/setup/environmentSettingsAjax.php
View File

@@ -55,7 +55,7 @@ switch ($request) {
$conf->aConfig = $config;
$conf->saveConfig( "ENVIRONMENT_SETTINGS", "" );
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"]));
G::auditLog("UpdateEnvironmentSettings", "UserNameDisplayFormat -> ".(isset($_POST["userFormat"]) ? $_POST["userFormat"] : '').", GlobalDateFormat -> ".(isset($_POST["dateFormat"]) ? $_POST["dateFormat"] : '').", HideProcessInformation -> ".(string)isset($_POST["hideProcessInf"]).", DateFormat -> ".(isset($_POST["casesListDateFormat"]) ? $_POST["casesListDateFormat"] : '').", NumberOfRowsPerPage -> ".(isset($_POST["casesListRowNumber"]) ? $_POST["casesListRowNumber"] : '').", RefreshTimeSeconds -> ".(isset($_POST["txtCasesRefreshTime"]) ? $_POST["txtCasesRefreshTime"] : ''));
$response = new stdclass();
$response->success = true;