BUG 8457 Problems reassigning cases to other users under HOME>Reassign SOLVED

- We found some loose details in the reassignment.
- It was added a note for the correct use of the reassignment and also a warning before continuing doing reassignment.

workflow/engine/templates/cases/casesList.js

@@ -485,58 +485,47 @@ Ext.onReady ( function() {
       //storeReassignCases.reload();
     }
   });
+  
+  var ExecReassign = function () {
+    newPopUp.hide();
+    var rs = storeReassignCases.getModifiedRecords();
 
-  var btnExecReassignSelected = new Ext.Button ({
+    var sv = [];
-    text: _('ID_REASSIGN'),
+    for(var i = 0; i <= rs.length-1; i++){
-    // text: 'Reassign',
+      sv[i]= rs[i].data;
-    //    text: TRANSLATIONS.LABEL_SELECT_ALL,
-    handler: function(){
-      newPopUp.hide();
-      var rs = storeReassignCases.getModifiedRecords();
-      var sv = [];
-      for(var i = 0; i <= rs.length-1; i++){
-        //sv[i]= rs[i].data['name'];
-        sv[i]= rs[i].data;
-      }
-      var gridData = storeReassignCases.getModifiedRecords();
-      Ext.MessageBox.show({ msg: _('ID_PROCESSING'), wait:true,waitConfig: {interval:200} });
-      Ext.Ajax.request({
-        url: 'proxySaveReassignCasesList',
-        success: function(response) {
-          Ext.MessageBox.hide();
-          storeCases.reload();
-          var ajaxServerResponse = Ext.util.JSON.decode(response.responseText);
-          var count;
-          var message = '';
-
-          for (count in ajaxServerResponse) {
-            if ( ajaxServerResponse[count]['TAS_TITLE'] != undefined ){
-              message = message + _('ID_CASE') + ": " + ajaxServerResponse[count]['APP_TITLE'] + " - " + _('ID_REASSIGNED_TO') + ": " + ajaxServerResponse[count]['APP_REASSIGN_USER'] + "<br>" ;
-            };
-          }
-
-          if (ajaxServerResponse['TOTAL']!=undefined&&ajaxServerResponse['TOTAL']!=-1){
-            message = message + "<br> " + _('ID_TOTAL_CASES_REASSIGNED') + ": " + ajaxServerResponse['TOTAL'];
-          } else {
-            message = "";
-          };
-
-          if (message!=""){
-            Ext.MessageBox.alert( _('ID_STATUS_REASSIGNMENT'), message, '' );
-          }
-        },
-        params: { APP_UIDS:ids, data:Ext.util.JSON.encode(sv), selected:true }
-      });
-
-      /*storeReassignCases.setBaseParam('selected', true);
-      var result = storeReassignCases.save();
-      //storeCases.load({params:{process: filterProcess, start : 0 , limit : pageSize}});
-      newPopUp.hide();
-      storeCases.reload();
-      //storeReassignCases.reload();
-      */
     }
-  });
+    var gridData = storeReassignCases.getModifiedRecords();
+    Ext.MessageBox.show({ msg: _('ID_PROCESSING'), wait:true,waitConfig: {interval:200} });
+    Ext.Ajax.request({
+      url: 'proxySaveReassignCasesList',
+      success: function(response) {
+        Ext.MessageBox.hide();
+        storeCases.reload();
+        var ajaxServerResponse = Ext.util.JSON.decode(response.responseText);
+        var count;
+        var message = '';
+
+        for (count in ajaxServerResponse) {
+          if ( ajaxServerResponse[count]['TAS_TITLE'] != undefined ){
+            message = message + _('ID_CASE') + ": " + ajaxServerResponse[count]['APP_TITLE'] + " - " + _('ID_REASSIGNED_TO') + ": " + ajaxServerResponse[count]['APP_REASSIGN_USER'] + "<br>" ;
+          };
+        }
+
+        if (ajaxServerResponse['TOTAL']!=undefined&&ajaxServerResponse['TOTAL']!=-1){
+          message = message + "<br> " + _('ID_TOTAL_CASES_REASSIGNED') + ": " + ajaxServerResponse['TOTAL'];
+        } else {
+          message = "";
+        };
+
+        if (message!=""){
+          Ext.MessageBox.alert( _('ID_STATUS_REASSIGNMENT'), message, '' );
+        }
+      },
+      params: { APP_UIDS:ids, data:Ext.util.JSON.encode(sv), selected:true }
+    });
+  }
+  
+
 
   // Create HttpProxy instance, all CRUD requests will be directed to single proxy url.
   var proxyCasesList = new Ext.data.HttpProxy({
@@ -791,7 +780,7 @@ Ext.onReady ( function() {
     // text: 'Reassign',
     // text: TRANSLATIONS.LABEL_UNSELECT_ALL,
     handler: function(){
-        reassign();
+      reassign();
     }
   });
 
@@ -1532,6 +1521,7 @@ Ext.onReady ( function() {
     listeners: {
       rowdblclick: openCase,
       render: function(){
+        
         //this.loadMask = new Ext.LoadMask(this.body, {msg:TRANSLATIONS.LABEL_GRID_LOADING});
         //this.ownerCt.doLayout();
       }
@@ -1580,12 +1570,27 @@ Ext.onReady ( function() {
     region: 'center',
     store: storeReassignCases,
     cm: reassignCm,
+
     autoHeight: true,   
     viewConfig: {
       forceFit:true
     }
   });
 
+  var btnExecReassignSelected = new Ext.Button ({
+    text: _('ID_REASSIGN'),
+    handler: function(){
+      var rs = storeReassignCases.getModifiedRecords();
+      
+      if (rs.length < storeReassignCases.totalLength) {
+        Ext.Msg.confirm( _('ID_CONFIRM'), _('ID_CONFIRM_TO_REASSIGN'), function (btn, text) {
+          if ( btn == 'yes' ) {
+            ExecReassign();
+          }
+        })
+      }
+    }
+  });
   
 var gridForm = new Ext.FormPanel({
         id: 'reassign-form',
@@ -1594,6 +1599,8 @@ var gridForm = new Ext.FormPanel({
         //title: 'Company data',
         bodyStyle:'padding:5px',
         width: 750,
+        
+
         layout: 'column',    // Specifies that the items will now be arranged in columns
         items: [{
             id : 'tasksGrid',
@@ -1617,10 +1624,12 @@ var gridForm = new Ext.FormPanel({
                 title  : _('ID_CASES_TO_REASSIGN_TASK_LIST'),
                 border : true,
 
-                listeners: {                   
+                listeners: {
-                    click: function() {                                 
+                  
+                    click: function() {
                         rows = this.getSelectionModel().getSelections();
                         var application = '';
+                        comboUsersToReassign.disable();
                         if( rows.length > 0 ) {
                           comboUsersToReassign.enable();
                             var ids = '';
@@ -1629,13 +1638,13 @@ var gridForm = new Ext.FormPanel({
                                application = rows[i].get('APP_UID');
                             }
                         } else {
-                          comboUsersToReassign.disable();                              
+                                                        
                         }
                         comboUsersToReassign.clearValue();
                         storeUsersToReassign.removeAll();
                         storeUsersToReassign.setBaseParam('application',application);
                              
-                        //storeUsersToReassign.load();
+                        storeUsersToReassign.load();
                         //alert(record.USERS);
                     } // Allow rows to be rendered.
               
@@ -1646,16 +1655,27 @@ var gridForm = new Ext.FormPanel({
             xtype: 'fieldset',
             labelWidth: 50,
             title: _('ID_USER_LIST'),
-            defaults: {width: 170, border:false},    // Default config options for child items
+            defaults: {width: 200, border:false},    // Default config options for child items
             defaultType: 'textfield',
             autoHeight: true,
-            bodyStyle: Ext.isIE ? 'text-align: right;padding:0 0 5px 15px;' : 'text-align: right; padding:10px 15px;',
+            bodyStyle: Ext.isIE ? 'text-align: left;padding:0 0 5px 15px;' : 'text-align: left; padding:10px 5px;',
             border: false,
-            style: {                
+            //style: {                
-                "margin-left": "10px", // when you add custom margin in IE 6...
+            //    "margin-left": "10px", // when you add custom margin in IE 6...
-                "margin-right": Ext.isIE6 ? (Ext.isStrict ? "-10px" : "-13px") : "0"  // you have to adjust for it somewhere else
+            //    "margin-right": Ext.isIE6 ? (Ext.isStrict ? "-10px" : "-13px") : "0"  // you have to adjust for it somewhere else
-            },
+            //},
-            items: comboUsersToReassign
+            items:
+              [
+                comboUsersToReassign,
+                {
+                  xtype: 'fieldset',
+                  border : true,
+                  defaultType: 'textfield',
+                  title: _('ID_INSTRUCTIONS'),
+                  autoHeight:true,
+                  html: _('ID_INSTRUCTIONS_TEXT')
+                }
+              ]
         }]
         //renderTo: bd
     });
@@ -1781,9 +1801,11 @@ var gridForm = new Ext.FormPanel({
 
 
 function reassign(){
+  storeReassignCases.removeAll();
   var rows  = grid.getSelectionModel().getSelections();
   storeReassignCases.rejectChanges();
   var tasks = [];
+  var sw = 0;
   if( rows.length > 0 ) {
     ids = '';
     for(i=0; i<rows.length; i++) {
@@ -1794,6 +1816,7 @@ function reassign(){
     }
     storeReassignCases.setBaseParam( 'APP_UIDS', ids);
     storeReassignCases.load();
+    
     newPopUp.show();
     comboUsersToReassign.disable();
 

workflow/engine/templates/cases/reassignList.js

@@ -32,6 +32,7 @@
   
   var storeUsersToReassign = new Ext.data.Store({
     remoteSort: false,
+    autoLoad:true,
     proxy : proxyUsersToReassignList,
     reader: readerUsersToReassignList,
     writer: writerUsersToReassignList,  // <-- plug a DataWriter into the store just as you would a Reader
@@ -63,11 +64,10 @@
     valueField    : 'userId',
     displayField  : 'userFullname',
     selectOnFocus : true,
-    typeAhead     : true,
+    typeAhead     : false,
     autocomplete  : true,
     hideTrigger   : Boolean,
-    alignTo : 'right',
+    alignTo       : 'right',
-    selectOnFocus:true,
     mode          : 'remote',
     triggerAction : 'all',
     emptyText     : _('ID_ENTER_SEARCH_TERM'),