From f0619af1087cbfa827049e1a8679adac836e7f5b Mon Sep 17 00:00:00 2001 From: jennylee Date: Tue, 14 Oct 2014 17:38:50 -0400 Subject: [PATCH] PM-604 Listado de Report Tables(old version) dentro de Designer. MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit PROBLEMA: El listado de RT dentro de un proceso tenia 2 problemas: 1. Los RT generados con el generador antiguo (Old Version) siempre se listaban sin diferencia en la paginacion. En todas las paginas aparecian los mismos RT(old version) 2. Los RT generados con el generador antiguo (Old Version) siempre se listaban al hacer una busqueda. En todas las busquedas siempre se incluian todos los RT(old version) sin excluir a los que no cumplian con el filtro de la busqueda. Ambos problemas sucedian porque la inclusion de los RT(old version) en el listado a mostrarse, se realizaba solo por la busqueda del processUID, nunca se incluian otros filtros (esto porq la funcion que lista a los RT(old version) solo acepta el UID del proceso) y siempre se aniadian este listado al listado que si usa filtros de RT(new version). SOLUCION: Se agrego una validacion para usar el filtro en las busquedas y una validacion para que los RT(Old version solo se añn al final del listado total de R RT del proceso). --- workflow/engine/controllers/pmTablesProxy.php | 16 +++++++++++++--- 1 file changed, 13 insertions(+), 3 deletions(-) diff --git a/workflow/engine/controllers/pmTablesProxy.php b/workflow/engine/controllers/pmTablesProxy.php index 1238b7ea0..2641d8334 100755 --- a/workflow/engine/controllers/pmTablesProxy.php +++ b/workflow/engine/controllers/pmTablesProxy.php @@ -53,9 +53,19 @@ class pmTablesProxy extends HttpProxyController } $addTables['count'] += count( $reportTablesOldList ); - foreach ($reportTablesOldList as $i => $oldRepTab) { - $addTables['rows'][] = array ('ADD_TAB_UID' => $oldRepTab['REP_TAB_UID'],'PRO_UID' => $oldRepTab['PRO_UID'],'DBS_UID' => ($oldRepTab['REP_TAB_CONNECTION'] == 'wf' ? 'workflow' : 'rp'),'ADD_TAB_DESCRIPTION' => $oldRepTab['REP_TAB_TITLE'],'ADD_TAB_NAME' => $oldRepTab['REP_TAB_NAME'],'ADD_TAB_TYPE' => $oldRepTab['REP_TAB_TYPE'],'TYPE' => 'CLASSIC' ); - } + if(($start+$limit) > $addTables['count']){ + foreach ($reportTablesOldList as $i => $oldRepTab) { + if($filter != ''){ + $oldTableName = strtolower($oldRepTab['REP_TAB_NAME']); + $oldTableDesc = strtolower($oldRepTab['REP_TAB_TITLE']); + if((strpos($oldTableName, $filter) !== false) || (strpos($oldTableDesc, $filter) !== false)){ + $addTables['rows'][] = array ('ADD_TAB_UID' => $oldRepTab['REP_TAB_UID'],'PRO_UID' => $oldRepTab['PRO_UID'],'DBS_UID' => ($oldRepTab['REP_TAB_CONNECTION'] == 'wf' ? 'workflow' : 'rp'),'ADD_TAB_DESCRIPTION' => $oldRepTab['REP_TAB_TITLE'],'ADD_TAB_NAME' => $oldRepTab['REP_TAB_NAME'],'ADD_TAB_TYPE' => $oldRepTab['REP_TAB_TYPE'],'TYPE' => 'CLASSIC' ); + } + } else { + $addTables['rows'][] = array ('ADD_TAB_UID' => $oldRepTab['REP_TAB_UID'],'PRO_UID' => $oldRepTab['PRO_UID'],'DBS_UID' => ($oldRepTab['REP_TAB_CONNECTION'] == 'wf' ? 'workflow' : 'rp'),'ADD_TAB_DESCRIPTION' => $oldRepTab['REP_TAB_TITLE'],'ADD_TAB_NAME' => $oldRepTab['REP_TAB_NAME'],'ADD_TAB_TYPE' => $oldRepTab['REP_TAB_TYPE'],'TYPE' => 'CLASSIC' ); + } + } + } } else { $addTables = AdditionalTables::getAll( $start, $limit, $filter ); }