From 0a2ecfc7f0924c2f97973ae1940b3a007b4a8764 Mon Sep 17 00:00:00 2001
From: Brayan Osmar Pereyra Suxo <brayan@colosa.com>
Date: Tue, 29 May 2012 12:36:38 -0400
Subject: [PATCH] BUG 8819 Remove the config to load the list  SOLVED

DESCRIPTION the bug hide columns SUMMARY and CASES NOTES.. this happened because load config the BD.
I removed part code to this config.
---
 .../engine/methods/cases/casesListExtJs.php   | 83 ++++++++-----------
 1 file changed, 35 insertions(+), 48 deletions(-)

diff --git a/workflow/engine/methods/cases/casesListExtJs.php b/workflow/engine/methods/cases/casesListExtJs.php
index 719cbed42..0662d790d 100755
--- a/workflow/engine/methods/cases/casesListExtJs.php
+++ b/workflow/engine/methods/cases/casesListExtJs.php
@@ -31,7 +31,7 @@
   $confReassignList = getReassignList();
 
   // evaluates an action and the configuration for the list that will be rendered
-  $config       = getAdditionalFields($action, $confCasesList);
+  $config       = getAdditionalFields($action);
   $columns      = $config['caseColumns'];
   $readerFields = $config['caseReaderFields'];
   $reassignColumns      = $confReassignList['caseColumns'];
@@ -738,55 +738,42 @@
    * @param  String $action
    * @return Array $config
    */
-function getAdditionalFields($action, $confCasesList){
-  $caseColumns = array();
-  $caseReaderFields = array();
 
-  if ( !empty($confCasesList) && !empty($confCasesList['second']['data']) ) {
-    foreach($confCasesList['second']['data'] as $fieldData){
-      if ( $fieldData['fieldType']!='key' ) {
-        $label = $fieldData['label'];
-        $caseColumns[]      = array( 'header' => $label, 'dataIndex' => $fieldData['name'], 'width' => $fieldData['width'], 'align' => $fieldData['align'] );
-        $caseReaderFields[] = array( 'name'   => $fieldData['name'] );
-      }
-    }
-    return array ( 'caseColumns' => $caseColumns, 'caseReaderFields' => $caseReaderFields, 'rowsperpage' => $confCasesList['rowsperpage'], 'dateformat' => $confCasesList['dateformat'] );
-  }
-  else {  //seems this is only in case this user dont have the configuration for this action.
-    switch ( $action ) {
-      case 'draft' :
-        $config = getDraft();
-        break;
-      case 'simple_search':
-      case 'search' :
-        $config = getSearch();
-        break;
-      case 'participated' :
-      case 'sent' :
-        $config = getParticipated();
-        break;
-      case 'unassigned' :
-        $config = getUnassigned();
-        break;
-      case 'paused' :
-        $config = getPaused();
-        break;
-      case 'to_revise' :
-        $config = getToRevise();
-        break;
-      case 'to_reassign' :
-        $config = getToReassign();
-        break;
-      case 'gral' :
-        $config = getGeneral();
-        break;
-      case 'todo' :
-      default :
-        $action = 'todo';
-        $config = getToDo();
+function getAdditionalFields($action){
+  switch ( $action ) {
+    case 'draft' :
+      $config = getDraft();
       break;
-    }
-    return $config;
+    case 'simple_search':
+    case 'search' :
+      $config = getSearch();
+      break;
+    case 'participated' :
+    case 'sent' :
+      $config = getParticipated();
+      break;
+    case 'unassigned' :
+      $config = getUnassigned();
+      break;
+    case 'paused' :
+      $config = getPaused();
+      break;
+    case 'to_revise' :
+      $config = getToRevise();
+      break;
+    case 'to_reassign' :
+      $config = getToReassign();
+      break;
+    case 'gral' :
+      $config = getGeneral();
+      break;
+    case 'todo' :
+    default :
+      $action = 'todo';
+      $config = getToDo();
+    break;
   }
+  
+  return $config;
 }