From 53b4e0ce3fe7026ae4388312ca2b2664f9b0c6f4 Mon Sep 17 00:00:00 2001 From: Hector Cortez Date: Thu, 15 Aug 2013 16:34:02 -0400 Subject: [PATCH 1/7] BUG 12702 No se pueden editar usuarios SOLVED - No se pueden editar usuarios - Adjustment in the validation of the roles permissions. --- workflow/engine/templates/users/users.js | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/workflow/engine/templates/users/users.js b/workflow/engine/templates/users/users.js index 54788e47b..e03d4dee2 100755 --- a/workflow/engine/templates/users/users.js +++ b/workflow/engine/templates/users/users.js @@ -1126,7 +1126,9 @@ function saveUser() var confPass = frmDetails.getForm().findField('USR_CNF_PASS').getValue(); if (confPass === newPass) { +console.log("Dentro de password no modificado"); if(typeof(EDITPROFILE) != "undefined" && EDITPROFILE == 1 && newPass != "") { +console.log("Entro por then"); var currentPassword = Ext.getCmp("currentPassword").getValue(); if(currentPassword != "") { @@ -1159,6 +1161,9 @@ function saveUser() } else { userFrmEditSubmit(); } + } else { + // Another field changed + userFrmEditSubmit(); } } }, @@ -1194,6 +1199,9 @@ function saveUser() } else { userFrmEditSubmit(); } + } else { + // Another field changed + userFrmEditSubmit(); } } }, From 3c3414999a3f4197149bd6d155728fcf93cf1d85 Mon Sep 17 00:00:00 2001 From: Omar Suca Date: Fri, 16 Aug 2013 11:07:15 -0400 Subject: [PATCH 2/7] BUG-12258 Preview in Messages history doesn't work in IE. Descripcion:Al ver la previsualizacion de las notificaciones enviadas por caso, se pudo verificar que para el caso de IE no muestra la informacion. Se adjuntan imagenes donde se muestra el problema. El label del footer de la grilla esta mal ya que esta mostrando una etiqueta de process. Cuando visualizas un preview de un item y despues seleccionas otro item para visualizar sin cerrar el actual, los campos de la ventana se distorsionan ya que nuevamente se hace el llamado del metodo "show" del widget "Window". Solucion: El problema de visualizacion fue constatado para los casos en los cuales el cuerpo del mensaje es html y dentro de este se encuentra cierto que causa conflicto con IE in quirks mode, para corregir este problema se manda una cabecera especificamente para IE. Se cambio el label relacionado a procesos a uno de items. Para el problema de campos distorsionados dentro de una ventana se valido la existencia del widget para posteriormente removerlo y volverlo a crear. Resuelto para la version PM 2.5.2 --- workflow/engine/classes/class.case.php | 4 +- .../templates/cases/caseMessageHistory.js | 184 +----------------- 2 files changed, 11 insertions(+), 177 deletions(-) diff --git a/workflow/engine/classes/class.case.php b/workflow/engine/classes/class.case.php index 9db2d6d56..95bcb235f 100755 --- a/workflow/engine/classes/class.case.php +++ b/workflow/engine/classes/class.case.php @@ -5890,6 +5890,8 @@ class Cases $aMessages = array(); while ($aRow = $oDataset->getRow()) { + //Head for IE quirks mode + $sBody = ''.$aRow['APP_MSG_BODY']; $aMessages[] = array('APP_MSG_UID' => $aRow['APP_MSG_UID'], 'APP_UID' => $aRow['APP_UID'], 'DEL_INDEX' => $aRow['DEL_INDEX'], @@ -5897,7 +5899,7 @@ class Cases 'APP_MSG_SUBJECT' => $aRow['APP_MSG_SUBJECT'], 'APP_MSG_FROM' => $aRow['APP_MSG_FROM'], 'APP_MSG_TO' => $aRow['APP_MSG_TO'], - 'APP_MSG_BODY' => $aRow['APP_MSG_BODY'], + 'APP_MSG_BODY' => $sBody, 'APP_MSG_DATE' => $aRow['APP_MSG_DATE'], 'APP_MSG_CC' => $aRow['APP_MSG_CC'], 'APP_MSG_BCC' => $aRow['APP_MSG_BCC'], diff --git a/workflow/engine/templates/cases/caseMessageHistory.js b/workflow/engine/templates/cases/caseMessageHistory.js index bef133c2f..70c3d929b 100644 --- a/workflow/engine/templates/cases/caseMessageHistory.js +++ b/workflow/engine/templates/cases/caseMessageHistory.js @@ -2,12 +2,14 @@ * @author: Douglas Medrano * May 03, 2011 */ - + //var windowMessage; + windowMessage = new Object(); function onResizeIframe(idIframe){ window.parent.tabIframeWidthFix2(idIframe); } function windowPreviewMessage(rowSelected) { + windowMessage = new Ext.Window({ title: '', width: 600, @@ -135,111 +137,6 @@ } - previewMessage = function() { - var rowSelected = Ext.getCmp('processesGrid').getSelectionModel().getSelected(); - if (rowSelected) { - windowPreviewMessage(rowSelected); - /* - windowMessage = new Ext.Window({ - title: '', - width: 600, - height: 420, - border: false, - layout : 'fit', - items: - [ - { - xtype: 'form', - frame: true, - border: false, - defaults: { - width: 150 - }, - items: [ - { - xtype: 'textfield', - fieldLabel: _("ID_FROM"), - id:'From', - anchor: '100%', - arrowAlign:'center', - readOnly: true, - name: 'From' - }, - { - xtype: 'textfield', - fieldLabel: _("ID_TO"), - id: 'To', - anchor: '100%', - arrowAlign:'center', - readOnly: true, - name: 'To' - }, - { - xtype: 'textfield', - fieldLabel: _('ID_SUBJECT'), - id: 'Subjet', - anchor: '100%', - arrowAlign:'center', - readOnly: true, - name: 'Subjet' - }, - { - xtype: 'textfield', - fieldLabel: _("DATE_LABEL"), - id: 'date', - arrowAlign:'center', - readOnly: true, - name: 'Status' - }, - { - name : 'body', - id:'body', - hideLabel:true, - xtype: 'htmleditor', - autoScroll: true, - readOnly: true, - x: 1, - y: 1, - enableAlignments:false, - enableColors:false, - enableFont:false, - enableFontSize:false, - enableFormat:false, - enableLinks:false, - enableLists:false, - enableSourceEdit:false, - anchor: '100%', - height: 260 - } - ] - } - ] - }); - - //load fields from rowSelect - Ext.getCmp('From').setValue(rowSelected.data.APP_MSG_FROM); - Ext.getCmp('To').setValue(rowSelected.data.APP_MSG_TO); - Ext.getCmp('Subjet').setValue(rowSelected.data.APP_MSG_SUBJECT); - Ext.getCmp('date').setValue(rowSelected.data.APP_MSG_DATE); - Ext.getCmp('body').setValue(rowSelected.data.APP_MSG_BODY); - - //show windows message - windowMessage.show(windowMessage); - */ - - } - else { - Ext.Msg.show({ - title:'', - msg: _("ID_NO_SELECTION_WARNING"), - buttons: Ext.Msg.INFO, - fn: function(){}, - animEl: 'elId', - icon: Ext.MessageBox.INFO, - buttons: Ext.MessageBox.OK - }); - } - } function ajaxPostRequest(url, callback_function, id){ var d = new Date(); @@ -419,12 +316,6 @@ var ActionTabFrameGlobal = ''; buttons: Ext.MessageBox.OK }); - - - - - - Ext.destroy(Ext.getCmp('processesGrid')); messageHistoryGridList(); @@ -576,7 +467,10 @@ var ActionTabFrameGlobal = ''; handler: function(grid, rowIndex, colIndex) { var rec = store.getAt(rowIndex); if (rec.get('MSGS_HISTORY') === 'VIEW' || rec.get('MSGS_HISTORY') === 'RESEND') { - windowPreviewMessage(rec); + if ( typeof windowMessage === 'object' && typeof windowMessage.close === 'function'){ + windowMessage.close(); + } + windowPreviewMessage(rec); } } } @@ -587,79 +481,17 @@ var ActionTabFrameGlobal = ''; ] }), store: store, -/* - tbar:[ - { - text:_("ID_RESEND"), - id:'sendMailMessageFormRadioId', - iconCls: 'button_menu_ext', - icon: '/images/mail-send16x16.png', - handler: function(){ - var rowSelected = processesGrid.getSelectionModel().getSelected(); - - if( rowSelected ){ - resendDialog(rowSelected); - } - else{ - Ext.Msg.show({ - title:'', - msg: TRANSLATIONS.ID_NO_SELECTION_WARNING, - buttons: Ext.Msg.INFO, - fn: function(){}, - animEl: 'elId', - icon: Ext.MessageBox.INFO, - buttons: Ext.MessageBox.OK - - }); - } - - }, - disabled:false - }, - { - xtype: 'tbseparator' - }, - { - text:_("ID_PREVIEW"), - id:'viewMailMessageFormRadioId', - iconCls: 'button_menu_ext', - icon: '/images/documents/_filefind.png', - handler: function(){ - var rowSelected = processesGrid.getSelectionModel().getSelected(); - - if (rowSelected) { - previewMessage(); - } else { - Ext.Msg.show({ - title:'', - msg: _("ID_NO_SELECTION_WARNING"), - buttons: Ext.Msg.INFO, - fn: function(){}, - animEl: 'elId', - icon: Ext.MessageBox.INFO, - buttons: Ext.MessageBox.OK - }); - } - }, - disabled:false - }, - { - xtype: 'tbfill' - } - ], -*/ bbar: new Ext.PagingToolbar({ pageSize: 20, store: store, displayInfo: true, - displayMsg: _('ID_DISPLAY_PROCESSES'), + displayMsg: _('ID_GRID_PAGE_DISPLAYING_ITEMS'), emptyMsg: "", items:[] }), listeners: { - // rowdblclick: previewMessage, render: function(){ this.loadMask = new Ext.LoadMask(this.body, {msg:'Loading...'}); processesGrid.getSelectionModel().on('rowselect', function(){ From ba9ade3c6f685e0e883076dc0a226e59c17ac902 Mon Sep 17 00:00:00 2001 From: Hector Cortez Date: Fri, 16 Aug 2013 11:14:39 -0400 Subject: [PATCH 3/7] BUG 12702 No se pueden editar usuarios SOLVED - No se pueden editar usuarios - Deleted console log tracer. --- workflow/engine/templates/users/users.js | 2 -- 1 file changed, 2 deletions(-) diff --git a/workflow/engine/templates/users/users.js b/workflow/engine/templates/users/users.js index e03d4dee2..2423d4496 100755 --- a/workflow/engine/templates/users/users.js +++ b/workflow/engine/templates/users/users.js @@ -1126,9 +1126,7 @@ function saveUser() var confPass = frmDetails.getForm().findField('USR_CNF_PASS').getValue(); if (confPass === newPass) { -console.log("Dentro de password no modificado"); if(typeof(EDITPROFILE) != "undefined" && EDITPROFILE == 1 && newPass != "") { -console.log("Entro por then"); var currentPassword = Ext.getCmp("currentPassword").getValue(); if(currentPassword != "") { From a480f03d723dfb30525f756b9f82f3cd61b5a6e5 Mon Sep 17 00:00:00 2001 From: Omar Suca Date: Fri, 16 Aug 2013 12:05:13 -0400 Subject: [PATCH 4/7] BUG-11291 No se deberia permitir la redimension de la ventana para pausar casos MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Descripcion: Al pausar un caso y abrirse la ventana para elegir la fecha y describir el motivo, la misma puede redimensionarse ya sea para aumentar o disminuir su tamaño(como muestra la imagen), lo cual no deberia permitirse ya que debería conservar su tamaño original. Las ventanas de Pause and Reassign tambien pueden ser redimensionados Solucion: El redimensionamiento con la ventana de Pause se constato que se encuentra con la propiedad correcta pero no asi las ventanas de Cancel y Reassign que se les agrego la propiedad "resizable:false". El problema del resizable con la ventana de Pause ocurre en la version 2.5.0 Resuelto para la version PM 2.5.2 --- workflow/engine/templates/cases/open.js | 2 ++ 1 file changed, 2 insertions(+) diff --git a/workflow/engine/templates/cases/open.js b/workflow/engine/templates/cases/open.js index 11e952775..c15712846 100755 --- a/workflow/engine/templates/cases/open.js +++ b/workflow/engine/templates/cases/open.js @@ -762,6 +762,7 @@ Ext.onReady(function(){ width:500, plain: true, modal: true, + resizable: false, title: _('ID_CONFIRM'), items: [ new Ext.FormPanel({ @@ -904,6 +905,7 @@ Ext.onReady(function(){ layout:'fit', autoScroll:true, modal: true, + resizable: false, maximizable: false, items: [grid] }); From a4f4e89cb8039ac1086ea53254791931d822f574 Mon Sep 17 00:00:00 2001 From: Omar Suca Date: Fri, 16 Aug 2013 14:24:34 -0400 Subject: [PATCH 5/7] BUG-11902 Campos de report table se desfasan cuando se mueven o amplian las cabeceras de las columnas. SOLVED. MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Descripcion: Cuando se crea un report table y se amplian las cabeceras, los campos de creacion no se ajustan al alineamiento y se desfasan. Comportamiento esperado. Los campos para llenar la creación del report table deben ajustarse al alineamiento de las cabeceras. Solucion: Se deshabilito el movimiento de las columnas para la grilla de edicion. Se agrego la propiedad "enableColumnMove: false". Resuelto para la version PM-2.5.2 --- workflow/engine/templates/pmTables/edit.js | 1 + 1 file changed, 1 insertion(+) diff --git a/workflow/engine/templates/pmTables/edit.js b/workflow/engine/templates/pmTables/edit.js index 55d88e3d1..2cfb170a3 100755 --- a/workflow/engine/templates/pmTables/edit.js +++ b/workflow/engine/templates/pmTables/edit.js @@ -446,6 +446,7 @@ Ext.onReady(function(){ ddGroup : 'availableGridDDGroup', enableDragDrop : true, enableColumnResize : true, + enableColumnMove: false, viewConfig : {forceFit:true}, cm: cm, sm: sm, From b5a00a54364da431c7b4c8f885ad93f98e925ccd Mon Sep 17 00:00:00 2001 From: Hector Cortez Date: Fri, 16 Aug 2013 16:29:46 -0400 Subject: [PATCH 6/7] BUG 12118 Fatal error en un dynaform cuando se habilita el html SOLVED MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - Cuando se da click en enable html para habilitar la edición de un dynaform, luego de guardar y volver a preview se da un fatal error. - Change how smarty tags filtering. --- workflow/engine/classes/class.dynaformEditor.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/workflow/engine/classes/class.dynaformEditor.php b/workflow/engine/classes/class.dynaformEditor.php index c746a99d0..398a1a967 100755 --- a/workflow/engine/classes/class.dynaformEditor.php +++ b/workflow/engine/classes/class.dynaformEditor.php @@ -543,7 +543,7 @@ class dynaformEditorAjax extends dynaformEditor implements iDynaformEditorAjax public function set_htmlcode($A, $htmlcode) { try { - $iOcurrences = preg_match_all('/\{\$\S*\s*\}/im', $htmlcode, $matches); + $iOcurrences = preg_match_all('/\{\$.*?\}/im', $htmlcode, $matches); if ($iOcurrences) { if (isset($matches[0])) { $tagsHtml = $matches[0]; From 3823f345992b2bda91ac3a087ed5697b487ba8c0 Mon Sep 17 00:00:00 2001 From: Omar Suca Date: Fri, 16 Aug 2013 16:47:56 -0400 Subject: [PATCH 7/7] BUG-11977 Popup de pausa de casos se puede desplazar fuera del marco MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Descripcion: Cuando se pausa un caso, el popup de pausas se puede desplazar fuera del marco y no se lo puede reestablecer, se tiene que volver a ejecutar el caso. Solucion: Se deahabilito el movimiento de la ventana y corrigio problemas con tamaño del formulario contenedor. Resuelto para la version PM-2.5.2 --- workflow/engine/templates/cases/open.js | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/workflow/engine/templates/cases/open.js b/workflow/engine/templates/cases/open.js index 11e952775..95305da55 100755 --- a/workflow/engine/templates/cases/open.js +++ b/workflow/engine/templates/cases/open.js @@ -976,6 +976,7 @@ Ext.onReady(function(){ var fieldset = { xtype : 'fieldset', labelWidth: 150, + width:380, //autoHeight : true, defaults : { width : 170, @@ -1045,12 +1046,13 @@ Ext.onReady(function(){ var win = new Ext.Window({ title: _('ID_PAUSE_CASE'), width: 380, - height: 250, + height: 260, layout:'fit', autoScroll:true, modal: true, maximizable: false, resizable: false, + draggable: false, items: [frm] }); Ext.Ajax.request({