From 48ea47541344240d77d7d27c865ac569a488dd8e Mon Sep 17 00:00:00 2001 From: "marcelo.cuiza" Date: Wed, 20 May 2015 18:35:38 -0400 Subject: [PATCH 1/5] PM-874 No se puede importar Report Tables en los procesos Se aumentaron validaciones varias, segun se especifica en el comentario en jira --- .../engine/classes/model/AdditionalTables.php | 8 +- workflow/engine/controllers/pmTablesProxy.php | 81 +++++++++-- workflow/engine/templates/pmTables/list.js | 130 +++++++++++++++++- 3 files changed, 205 insertions(+), 14 deletions(-) diff --git a/workflow/engine/classes/model/AdditionalTables.php b/workflow/engine/classes/model/AdditionalTables.php index e13391417..115c2c76d 100755 --- a/workflow/engine/classes/model/AdditionalTables.php +++ b/workflow/engine/classes/model/AdditionalTables.php @@ -751,9 +751,11 @@ class AdditionalTables extends BaseAdditionalTables } else { // grids foreach ($caseData[$dKey] as $dIndex => $dRow) { - foreach ($dRow as $k => $v) { - if (trim($v) === '') { - $caseData[$dKey][$dIndex][$k] = null; + if (!is_array($dRow)) { + foreach ($dRow as $k => $v) { + if (trim($v) === '') { + $caseData[$dKey][$dIndex][$k] = null; + } } } } diff --git a/workflow/engine/controllers/pmTablesProxy.php b/workflow/engine/controllers/pmTablesProxy.php index 0fd99f919..30892b520 100755 --- a/workflow/engine/controllers/pmTablesProxy.php +++ b/workflow/engine/controllers/pmTablesProxy.php @@ -915,17 +915,32 @@ class pmTablesProxy extends HttpProxyController */ public function import ($httpData) { + if (isset( $_POST["form"]["TYPE_TABLE"] ) && ! empty( $_POST["form"]["TYPE_TABLE"] )) { + if($_POST["form"]["TYPE_TABLE"] == 'admin') { + $fromAdmin = true; + } else { + $fromAdmin = false; + } + } + require_once 'classes/model/AdditionalTables.php'; try { $result = new stdClass(); $errors = ''; + $fromConfirm = false; $overWrite = isset( $_POST['form']['OVERWRITE'] ) ? true : false; + + if (isset( $_POST["form"]["FROM_CONFIRM"] ) && ! empty( $_POST["form"]["FROM_CONFIRM"] )) { + $fromConfirm = $_POST["form"]["FROM_CONFIRM"]; + $_FILES['form'] = $_SESSION['FILES_FORM']; + } //save the file if ($_FILES['form']['error']['FILENAME'] !== 0) { throw new Exception( G::loadTranslation( 'ID_PMTABLE_UPLOADING_FILE_PROBLEM' ) ); } + $_SESSION['FILES_FORM'] = $_FILES['form']; $oAdditionalTables = new AdditionalTables(); $tableNameMap = array (); @@ -935,7 +950,10 @@ class pmTablesProxy extends HttpProxyController $PUBLIC_ROOT_PATH = PATH_DATA . 'sites' . PATH_SEP . SYS_SYS . PATH_SEP . 'public' . PATH_SEP; $filename = $_FILES['form']['name']['FILENAME']; $tempName = $_FILES['form']['tmp_name']['FILENAME']; - G::uploadFile( $tempName, $PUBLIC_ROOT_PATH, $filename ); + + if(!$fromConfirm) { + G::uploadFile( $tempName, $PUBLIC_ROOT_PATH, $filename ); + } $fileContent = file_get_contents( $PUBLIC_ROOT_PATH . $filename ); @@ -947,6 +965,17 @@ class pmTablesProxy extends HttpProxyController $fsData = intval( fread( $fp, 9 ) ); //reading the metadata $sType = fread( $fp, $fsData ); + $pmTables = $this->getList(''); + $proUids = Array(); + if($pmTables['count']) { + $pmTables = $pmTables['rows']; + foreach($pmTables as $val) { + if($val['PRO_UID'] != '') { + $proUids[] = $val['PRO_UID']; + } + } + } + // first create the tables structures while (! feof( $fp )) { @@ -964,7 +993,44 @@ class pmTablesProxy extends HttpProxyController $additionalTable = new additionalTables(); $tableExists = $additionalTable->loadByName( $contentSchema['ADD_TAB_NAME'] ); $tableNameMap[$contentSchema['ADD_TAB_NAME']] = $contentSchema['ADD_TAB_NAME']; - + + $tableData = new stdClass(); + if (isset( $_POST["form"]["PRO_UID"] ) && ! empty( $_POST["form"]["PRO_UID"] )) { + $tableData->PRO_UID = $_POST["form"]["PRO_UID"]; + } else { + $tableData->PRO_UID = isset( $contentSchema["PRO_UID"] ) ? $contentSchema["PRO_UID"] : ""; + } + + if($fromAdmin) { /*from admin tab*/ + if ($tableExists !== false && !$fromConfirm) { + $validationType = 1; + throw new Exception( G::loadTranslation( 'ID_OVERWRITE_PMTABLE' ) ); + } + if(!in_array($tableData->PRO_UID, $proUids)) { + $validationType = 2; + throw new Exception( G::loadTranslation( 'ID_NO_RELATED_PROCESS' ) ); + } + } else { /*from designer tab*/ + if ($tableExists !== false && !$fromConfirm) { + $validationType = 1; + throw new Exception( G::loadTranslation( 'ID_OVERWRITE_PMTABLE' ) ); + } + if (isset( $_SESSION['PROCESS'] ) && !empty( $_SESSION['PROCESS'] )) { + if($_SESSION['PROCESS'] != $tableData->PRO_UID) { + if(!in_array($tableData->PRO_UID, $proUids)) { + $validationType = 2; + if($fromConfirm == $validationType) { + throw new Exception( G::loadTranslation( 'ID_OVERWRITE_RELATED_PROCESS' ) ); + } else { + $tableData->PRO_UID = $_SESSION['PROCESS']; + } + } else { + $validationType = 3; + throw new Exception( G::loadTranslation( 'ID_ALREADY_RELATED_TABLE ' ) ); + } + } + } + } if ($overWrite) { if ($tableExists !== false) { $additionalTable->deleteAll( $tableExists['ADD_TAB_UID'] ); @@ -995,18 +1061,11 @@ class pmTablesProxy extends HttpProxyController $columns[] = $column; } - $tableData = new stdClass(); $tableData->REP_TAB_UID = $contentSchema['ADD_TAB_UID']; $tableData->REP_TAB_NAME = $contentSchema['ADD_TAB_NAME']; $tableData->REP_TAB_DSC = $contentSchema['ADD_TAB_DESCRIPTION']; $tableData->REP_TAB_CONNECTION = $contentSchema['DBS_UID']; - - if (isset( $_POST["form"]["PRO_UID"] ) && ! empty( $_POST["form"]["PRO_UID"] )) { - $tableData->PRO_UID = $_POST["form"]["PRO_UID"]; - } else { - $tableData->PRO_UID = isset( $contentSchema["PRO_UID"] ) ? $contentSchema["PRO_UID"] : ""; - } - + $tableData->REP_TAB_TYPE = isset( $contentSchema['ADD_TAB_TYPE'] ) ? $contentSchema['ADD_TAB_TYPE'] : ''; $tableData->REP_TAB_GRID = isset( $contentSchema['ADD_TAB_GRID'] ) ? $contentSchema['ADD_TAB_GRID'] : ''; $tableData->columns = G::json_encode( $columns ); @@ -1147,6 +1206,8 @@ class pmTablesProxy extends HttpProxyController $result->message = $msg; } catch (Exception $e) { $result = new stdClass(); + $result->fromAdmin = $fromAdmin; + $result->validationType = $validationType; $result->errorType = 'error'; $result->buildResult = ob_get_contents(); ob_end_clean(); diff --git a/workflow/engine/templates/pmTables/list.js b/workflow/engine/templates/pmTables/list.js index c3d8c44c7..3d1762686 100755 --- a/workflow/engine/templates/pmTables/list.js +++ b/workflow/engine/templates/pmTables/list.js @@ -561,6 +561,10 @@ ImportPMTable = function(){ fieldLabel: '', boxLabel: _('ID_OVERWRITE_EXIST'), // 'Overwrite if exists?', name: 'form[OVERWRITE]' + }, { + xtype: 'hidden', + name: 'form[TYPE_TABLE]', + value: (PRO_UID? 'designer' : 'admin') }], buttons: [{ id: 'importPMTableButtonUpload', @@ -613,7 +617,131 @@ ImportPMTable = function(){ PMExt.warning(_('ID_WARNING'), result.message.replace(/\n/g,'
')); } else { - PMExt.error(_('ID_ERROR'), result.message.replace(/\n/g,'
')); + if(result.fromAdmin) { /*from admin tab*/ + if(result.validationType == 1) { + Ext.MessageBox.confirm('Confirmation', result.message.replace(/\n/g,'
'), function(btn, text){ + if (btn == 'yes'){ + Ext.Ajax.request({ + url: 'pmTablesProxy/import', + params: { + 'form[FROM_CONFIRM]':'overWrite', + 'form[TYPE_TABLE]':(PRO_UID? 'designer' : 'admin'), + 'form[OVERWRITE]':true + }, + success: function(resp){ + var result = Ext.util.JSON.decode(resp.responseText); + if (result.success) { + PMExt.notify(_('ID_IMPORT_RESULT'), result.message); + } else { + if(result.validationType == 2) { + PMExt.error(_('ID_ERROR'), result.message.replace(/\n/g,'
')); + } + } + }, + failure: function(obj, resp){ + var result = Ext.util.JSON.decode(resp.responseText); + if(result.validationType == 2) { + PMExt.error(_('ID_ERROR'), result.message.replace(/\n/g,'
')); + } + } + }); + Ext.getCmp('infoGrid').getStore().reload(); + } + }); + return false; + } else { + PMExt.error(_('ID_ERROR'), result.message.replace(/\n/g,'
')); + } + } else { /*from designer tab*/ + if(result.validationType == 1) { + Ext.MessageBox.confirm('Confirmation', result.message.replace(/\n/g,'
'), function(btn, text){ + if (btn == 'yes'){ + Ext.Ajax.request({ + url: 'pmTablesProxy/import', + params: { + 'form[FROM_CONFIRM]':'2', + 'form[TYPE_TABLE]':(PRO_UID? 'designer' : 'admin'), + 'form[OVERWRITE]':true + }, + success: function(resp){ + var result = Ext.util.JSON.decode(resp.responseText); + if (result.success) { + PMExt.notify(_('ID_IMPORT_RESULT'), result.message); + } else { + if(result.validationType == 2) { + Ext.MessageBox.confirm('Confirmation', result.message.replace(/\n/g,'
'), function(btn, text){ + if (btn == 'yes'){ + Ext.Ajax.request({ + url: 'pmTablesProxy/import', + params: { + 'form[FROM_CONFIRM]':'overWrite', + 'form[TYPE_TABLE]':(PRO_UID? 'designer' : 'admin'), + 'form[OVERWRITE]':true + }, + success: function(resp){ + var result = Ext.util.JSON.decode(resp.responseText); + if (result.success) { + PMExt.notify(_('ID_IMPORT_RESULT'), result.message); + } else { + PMExt.error(_('ID_ERROR'), result.message.replace(/\n/g,'
')); + } + }, + failure: function(obj, resp){ + PMExt.error(_('ID_ERROR'), result.message.replace(/\n/g,'
')); + } + }); + Ext.getCmp('infoGrid').getStore().reload(); + } + }); + return false; + } + else { + PMExt.error(_('ID_ERROR'), result.message.replace(/\n/g,'
')); + } + } + }, + failure: function(obj, resp){ + var result = Ext.util.JSON.decode(resp.responseText); + if(result.validationType == 2) { + PMExt.error(_('ID_ERROR'), result.message.replace(/\n/g,'
')); + } + } + }); + Ext.getCmp('infoGrid').getStore().reload(); + } + }); + return false; + } + if(result.validationType == 2) { + Ext.MessageBox.confirm('Confirmation', result.message.replace(/\n/g,'
'), function(btn, text){ + if (btn == 'yes'){ + Ext.Ajax.request({ + url: 'pmTablesProxy/import', + params: { + 'form[FROM_CONFIRM]':'overWrite', + 'form[TYPE_TABLE]':(PRO_UID? 'designer' : 'admin'), + 'form[OVERWRITE]':true + }, + success: function(resp){ + var result = Ext.util.JSON.decode(resp.responseText); + if (result.success) { + PMExt.notify(_('ID_IMPORT_RESULT'), result.message); + } else { + PMExt.error(_('ID_ERROR'), result.message.replace(/\n/g,'
')); + } + }, + failure: function(obj, resp){ + PMExt.error(_('ID_ERROR'), result.message.replace(/\n/g,'
')); + } + }); + Ext.getCmp('infoGrid').getStore().reload(); + } + }); + return false; + } else { + PMExt.error(_('ID_ERROR'), result.message.replace(/\n/g,'
')); + } + } } } }); From b4420c2614a74d0fb58e634a1d36b092c5ff2a60 Mon Sep 17 00:00:00 2001 From: "marcelo.cuiza" Date: Mon, 25 May 2015 14:34:51 -0400 Subject: [PATCH 2/5] PM-874 No se puede importar Report Tables en los procesos Se aumentaron validaciones varias, segun se especifica en el comentario en jira --- workflow/engine/controllers/pmTablesProxy.php | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/workflow/engine/controllers/pmTablesProxy.php b/workflow/engine/controllers/pmTablesProxy.php index 30892b520..5cf282954 100755 --- a/workflow/engine/controllers/pmTablesProxy.php +++ b/workflow/engine/controllers/pmTablesProxy.php @@ -915,12 +915,11 @@ class pmTablesProxy extends HttpProxyController */ public function import ($httpData) { + $fromAdmin = false; if (isset( $_POST["form"]["TYPE_TABLE"] ) && ! empty( $_POST["form"]["TYPE_TABLE"] )) { if($_POST["form"]["TYPE_TABLE"] == 'admin') { $fromAdmin = true; - } else { - $fromAdmin = false; - } + } } require_once 'classes/model/AdditionalTables.php'; From 4ef02142adeef0c4ee83a14c54b365ea024d1a2c Mon Sep 17 00:00:00 2001 From: "marcelo.cuiza" Date: Wed, 27 May 2015 11:37:32 -0400 Subject: [PATCH 3/5] PM-874 No se puede importar Report Tables en los procesos Se aumentaron validaciones varias, segun se especifica en el comentario en jira --- workflow/engine/controllers/pmTablesProxy.php | 43 ++++++---- workflow/engine/templates/pmTables/list.js | 81 ++++++++++++++++++- 2 files changed, 105 insertions(+), 19 deletions(-) diff --git a/workflow/engine/controllers/pmTablesProxy.php b/workflow/engine/controllers/pmTablesProxy.php index 5cf282954..f3af71cf6 100755 --- a/workflow/engine/controllers/pmTablesProxy.php +++ b/workflow/engine/controllers/pmTablesProxy.php @@ -953,6 +953,10 @@ class pmTablesProxy extends HttpProxyController if(!$fromConfirm) { G::uploadFile( $tempName, $PUBLIC_ROOT_PATH, $filename ); } + + if ($fromConfirm == 'clear') { + $fromConfirm = true; + } $fileContent = file_get_contents( $PUBLIC_ROOT_PATH . $filename ); @@ -1000,36 +1004,45 @@ class pmTablesProxy extends HttpProxyController $tableData->PRO_UID = isset( $contentSchema["PRO_UID"] ) ? $contentSchema["PRO_UID"] : ""; } - if($fromAdmin) { /*from admin tab*/ + $isPmTable = false; /*is a report table*/ + if($contentSchema["PRO_UID"] == "" ) { + $isPmTable = true; + } + if (isset( $_POST["form"]["PRO_UID_HELP"] ) && !empty($_POST["form"]["PRO_UID_HELP"])) { + $currentPRO_UID = $_POST["form"]["PRO_UID_HELP"]; + } else { + $currentPRO_UID = (isset( $_SESSION['PROCESS'] ) && !empty( $_SESSION['PROCESS'] )) ? $_SESSION['PROCESS'] : ''; + } + + if($fromAdmin) { /* from admin tab */ if ($tableExists !== false && !$fromConfirm) { $validationType = 1; throw new Exception( G::loadTranslation( 'ID_OVERWRITE_PMTABLE' ) ); } - if(!in_array($tableData->PRO_UID, $proUids)) { + if(!in_array($tableData->PRO_UID, $proUids) && !$isPmTable) { $validationType = 2; throw new Exception( G::loadTranslation( 'ID_NO_RELATED_PROCESS' ) ); } - } else { /*from designer tab*/ + } else { /* from designer tab */ if ($tableExists !== false && !$fromConfirm) { $validationType = 1; throw new Exception( G::loadTranslation( 'ID_OVERWRITE_PMTABLE' ) ); - } - if (isset( $_SESSION['PROCESS'] ) && !empty( $_SESSION['PROCESS'] )) { - if($_SESSION['PROCESS'] != $tableData->PRO_UID) { - if(!in_array($tableData->PRO_UID, $proUids)) { - $validationType = 2; - if($fromConfirm == $validationType) { - throw new Exception( G::loadTranslation( 'ID_OVERWRITE_RELATED_PROCESS' ) ); - } else { - $tableData->PRO_UID = $_SESSION['PROCESS']; - } + } + if($currentPRO_UID != $tableData->PRO_UID) { + if(!in_array($tableData->PRO_UID, $proUids)) { + $validationType = 2; + if(($fromConfirm == $validationType || !$fromConfirm) && !$isPmTable) { + throw new Exception( G::loadTranslation( 'ID_OVERWRITE_RELATED_PROCESS' ) ); } else { - $validationType = 3; - throw new Exception( G::loadTranslation( 'ID_ALREADY_RELATED_TABLE ' ) ); + $tableData->PRO_UID = $currentPRO_UID; } + } else { + $validationType = 3; + throw new Exception( G::loadTranslation( 'ID_ALREADY_RELATED_TABLE ' ) ); } } } + if ($overWrite) { if ($tableExists !== false) { $additionalTable->deleteAll( $tableExists['ADD_TAB_UID'] ); diff --git a/workflow/engine/templates/pmTables/list.js b/workflow/engine/templates/pmTables/list.js index 3d1762686..a6497ceb2 100755 --- a/workflow/engine/templates/pmTables/list.js +++ b/workflow/engine/templates/pmTables/list.js @@ -617,7 +617,7 @@ ImportPMTable = function(){ PMExt.warning(_('ID_WARNING'), result.message.replace(/\n/g,'
')); } else { - if(result.fromAdmin) { /*from admin tab*/ + if(result.fromAdmin) { /* from admin tab */ if(result.validationType == 1) { Ext.MessageBox.confirm('Confirmation', result.message.replace(/\n/g,'
'), function(btn, text){ if (btn == 'yes'){ @@ -632,6 +632,7 @@ ImportPMTable = function(){ var result = Ext.util.JSON.decode(resp.responseText); if (result.success) { PMExt.notify(_('ID_IMPORT_RESULT'), result.message); + Ext.getCmp('infoGrid').getStore().reload(); } else { if(result.validationType == 2) { PMExt.error(_('ID_ERROR'), result.message.replace(/\n/g,'
')); @@ -645,14 +646,27 @@ ImportPMTable = function(){ } } }); - Ext.getCmp('infoGrid').getStore().reload(); + } else { + Ext.Ajax.request({ + url: 'pmTablesProxy/import', + params: { + 'form[FROM_CONFIRM]':'clear', + 'form[TYPE_TABLE]':(PRO_UID? 'designer' : 'admin') + }, + success: function(resp) { + var result = Ext.util.JSON.decode(resp.responseText); + PMExt.notify(_('ID_IMPORT_RESULT'), result.message); + Ext.getCmp('infoGrid').getStore().reload(); + } + }); } + Ext.getCmp('infoGrid').getStore().reload(); }); return false; } else { PMExt.error(_('ID_ERROR'), result.message.replace(/\n/g,'
')); } - } else { /*from designer tab*/ + } else { /* from designer tab */ if(result.validationType == 1) { Ext.MessageBox.confirm('Confirmation', result.message.replace(/\n/g,'
'), function(btn, text){ if (btn == 'yes'){ @@ -667,6 +681,7 @@ ImportPMTable = function(){ var result = Ext.util.JSON.decode(resp.responseText); if (result.success) { PMExt.notify(_('ID_IMPORT_RESULT'), result.message); + Ext.getCmp('infoGrid').getStore().reload(); } else { if(result.validationType == 2) { Ext.MessageBox.confirm('Confirmation', result.message.replace(/\n/g,'
'), function(btn, text){ @@ -682,6 +697,7 @@ ImportPMTable = function(){ var result = Ext.util.JSON.decode(resp.responseText); if (result.success) { PMExt.notify(_('ID_IMPORT_RESULT'), result.message); + Ext.getCmp('infoGrid').getStore().reload(); } else { PMExt.error(_('ID_ERROR'), result.message.replace(/\n/g,'
')); } @@ -708,7 +724,62 @@ ImportPMTable = function(){ } }); Ext.getCmp('infoGrid').getStore().reload(); + } else { + Ext.Ajax.request({ + url: 'pmTablesProxy/import', + params: { + 'form[FROM_CONFIRM]':'2', + 'form[TYPE_TABLE]':(PRO_UID? 'designer' : 'admin'), + 'form[PRO_UID_HELP]':PRO_UID + }, + success: function(resp) { + var result = Ext.util.JSON.decode(resp.responseText); + if(result.validationType == 2) { + /*add code if related process*/ + Ext.MessageBox.confirm('Confirmation', result.message.replace(/\n/g,'
'), function(btn, text){ + if (btn == 'yes'){ + Ext.Ajax.request({ + url: 'pmTablesProxy/import', + params: { + 'form[FROM_CONFIRM]':'overWrite', + 'form[TYPE_TABLE]':(PRO_UID? 'designer' : 'admin'), + 'form[PRO_UID_HELP]':PRO_UID + }, + success: function(resp){ + var result = Ext.util.JSON.decode(resp.responseText); + if (result.success) { + PMExt.notify(_('ID_IMPORT_RESULT'), result.message); + Ext.getCmp('infoGrid').getStore().reload(); + } else { + PMExt.error(_('ID_ERROR'), result.message.replace(/\n/g,'
')); + } + }, + failure: function(obj, resp){ + PMExt.error(_('ID_ERROR'), result.message.replace(/\n/g,'
')); + } + }); + Ext.getCmp('infoGrid').getStore().reload(); + } + }); + return false; + } else { + var result = Ext.util.JSON.decode(resp.responseText); + if (result.success) { + PMExt.notify(_('ID_IMPORT_RESULT'), result.message); + Ext.getCmp('infoGrid').getStore().reload(); + } else { + PMExt.error(_('ID_ERROR'), result.message.replace(/\n/g,'
')); + } + } + //PMExt.notify(_('ID_IMPORT_RESULT'), result.message); + }, + failure: function(obj, resp){ + var result = Ext.util.JSON.decode(resp.responseText); + PMExt.error(_('ID_ERROR'), result.message.replace(/\n/g,'
')); + } + }); } + Ext.getCmp('infoGrid').getStore().reload(); }); return false; } @@ -720,12 +791,14 @@ ImportPMTable = function(){ params: { 'form[FROM_CONFIRM]':'overWrite', 'form[TYPE_TABLE]':(PRO_UID? 'designer' : 'admin'), - 'form[OVERWRITE]':true + 'form[OVERWRITE]':true, + 'form[PRO_UID_HELP]':PRO_UID }, success: function(resp){ var result = Ext.util.JSON.decode(resp.responseText); if (result.success) { PMExt.notify(_('ID_IMPORT_RESULT'), result.message); + Ext.getCmp('infoGrid').getStore().reload(); } else { PMExt.error(_('ID_ERROR'), result.message.replace(/\n/g,'
')); } From cf1ccbb15d77d679c00d0f8eb1301866f8ab0bab Mon Sep 17 00:00:00 2001 From: "marcelo.cuiza" Date: Fri, 29 May 2015 11:33:05 -0400 Subject: [PATCH 4/5] PM-874 --- workflow/engine/controllers/pmTablesProxy.php | 3 +++ 1 file changed, 3 insertions(+) diff --git a/workflow/engine/controllers/pmTablesProxy.php b/workflow/engine/controllers/pmTablesProxy.php index f3af71cf6..cf7851e50 100755 --- a/workflow/engine/controllers/pmTablesProxy.php +++ b/workflow/engine/controllers/pmTablesProxy.php @@ -1024,6 +1024,9 @@ class pmTablesProxy extends HttpProxyController throw new Exception( G::loadTranslation( 'ID_NO_RELATED_PROCESS' ) ); } } else { /* from designer tab */ + if($isPmTable){ + throw new Exception( G::loadTranslation( 'ID_NO_REPORT_TABLE' ) ); + } if ($tableExists !== false && !$fromConfirm) { $validationType = 1; throw new Exception( G::loadTranslation( 'ID_OVERWRITE_PMTABLE' ) ); From cfcebd3da5e073b4e86c58a1eea6f055a5c74853 Mon Sep 17 00:00:00 2001 From: "marcelo.cuiza" Date: Fri, 29 May 2015 16:07:30 -0400 Subject: [PATCH 5/5] pm-874 --- workflow/engine/controllers/pmTablesProxy.php | 1 + 1 file changed, 1 insertion(+) diff --git a/workflow/engine/controllers/pmTablesProxy.php b/workflow/engine/controllers/pmTablesProxy.php index cf7851e50..988d4b04a 100755 --- a/workflow/engine/controllers/pmTablesProxy.php +++ b/workflow/engine/controllers/pmTablesProxy.php @@ -1025,6 +1025,7 @@ class pmTablesProxy extends HttpProxyController } } else { /* from designer tab */ if($isPmTable){ + $validationType = ''; throw new Exception( G::loadTranslation( 'ID_NO_REPORT_TABLE' ) ); } if ($tableExists !== false && !$fromConfirm) {