From 7fe908c50c8dbc5cc05f571dbe11d66141caacd4 Mon Sep 17 00:00:00 2001 From: Marius Cramer <m.cramer@pixcept.de> Date: Thu, 14 Nov 2013 09:01:22 -0500 Subject: [PATCH] Cleaning up code to match coding guidelines --- interface/lib/classes/searchform_actions.inc.php | 99 +++++++++++++++++++++++++------------------------ 1 files changed, 50 insertions(+), 49 deletions(-) diff --git a/interface/lib/classes/searchform_actions.inc.php b/interface/lib/classes/searchform_actions.inc.php index a451f4a..e48eb85 100644 --- a/interface/lib/classes/searchform_actions.inc.php +++ b/interface/lib/classes/searchform_actions.inc.php @@ -29,44 +29,44 @@ */ class searchform_actions { - + var $id; var $idx_key; var $DataRowColor; var $SQLExtWhere = ''; var $SQLOrderBy = ''; - + function onLoad() { global $app, $conf, $list_def_file; - + if(!is_object($app->tpl)) $app->uses('tpl'); if(!is_object($app->searchform)) $app->uses('searchform'); if(!is_object($app->tform)) $app->uses('tform'); - + // Load list definition $app->searchform->loadListDef($list_def_file); - + // Delete the search form contents, if requested if($_REQUEST["empty_searchfields"] == 'yes') { $list_name = $app->searchform->listDef["name"]; unset($_SESSION["search"][$list_name]); } - + // Save the search for later usage if($_REQUEST["btn_submit_search_save"] && $_REQUEST["search_save_as"] != '') { $app->searchform->saveSearchSettings($_REQUEST["search_save_as"]); } - + // Set th returnto value for forms $_SESSION["s"]["form"]["return_to_url"] = $app->searchform->listDef["file"]; - + if(!is_file('templates/'.$app->searchform->listDef["name"].'_search.htm')) { $app->uses('searchform_tpl_generator'); $app->searchform_tpl_generator->buildHTML($app->searchform->listDef); } - + $app->tpl->newTemplate("searchpage.tpl.htm"); - $app->tpl->setInclude('content_tpl','templates/'.$app->searchform->listDef["name"].'_search.htm'); + $app->tpl->setInclude('content_tpl', 'templates/'.$app->searchform->listDef["name"].'_search.htm'); // Getting Datasets from DB $records = $app->db->queryAllRecords($this->getQueryString()); @@ -74,30 +74,30 @@ $this->DataRowColor = "#FFFFFF"; if(is_array($records)) { - $this->idx_key = $app->searchform->listDef["table_idx"]; + $this->idx_key = $app->searchform->listDef["table_idx"]; foreach($records as $rec) { $records_new[] = $this->prepareDataRow($rec); } } - $app->tpl->setLoop('records',$records_new); - + $app->tpl->setLoop('records', $records_new); + //print_r($records_new); $this->onShow(); - - + + } - + function prepareDataRow($rec) { global $app; - + $rec = $app->searchform->decode($rec); // Alternating datarow colors $this->DataRowColor = ($this->DataRowColor == "#FFFFFF")?"#EEEEEE":"#FFFFFF"; $rec["bgcolor"] = $this->DataRowColor; - + // substitute value for select fields foreach($app->searchform->listDef["item"] as $field) { $key = $field["field"]; @@ -110,16 +110,16 @@ $rec[$key] = $field['value'][$rec[$key]]; } } - + // The variable "id" contains always the index variable $rec["id"] = $rec[$this->idx_key]; - + return $rec; } - + function getQueryString() { global $app; - + // Generate the search sql if($app->searchform->listDef["auth"] != 'no') { if($_SESSION["s"]["user"]["typ"] == "admin") { @@ -128,74 +128,75 @@ $sql_where = $app->tform->getAuthSQL('r')." and"; } } - + if($this->SQLExtWhere != '') { $sql_where .= " ".$this->SQLExtWhere." and"; } $sql_where = $app->searchform->getSearchSQL($sql_where); $app->tpl->setVar($app->searchform->searchValues); - + $order_by_sql = $this->SQLOrderBy; // Generate SQL for paging $limit_sql = $app->searchform->getPagingSQL($sql_where); - $app->tpl->setVar("paging",$app->searchform->pagingHTML); + $app->tpl->setVar("paging", $app->searchform->pagingHTML); return "SELECT * FROM ".$app->searchform->listDef["table"]." WHERE $sql_where $order_by_sql $limit_sql"; - + } - - + + function onShow() { global $app; - + // Language File setzen $lng_file = ISPC_WEB_PATH.'/lang/lib/lang/'.$_SESSION['s']['language'].'_list.lng'; if(!file_exists($lng_file)) $lng_file = ISPC_WEB_PATH.'/lang/lib/lang/en_'.'_list.lng'; - include($lng_file); + include $lng_file; $lng_file = "lib/lang/".$_SESSION["s"]["language"]."_".$app->searchform->listDef['name']."_search.lng"; if(!file_exists($lng_file)) $lng_file = 'lib/lang/en_'.$app->searchform->listDef['name']."_search.lng"; - include($lng_file); + include $lng_file; $app->tpl->setVar($wb); - $app->tpl->setVar("form_action",$app->searchform->listDef["file"]); - + $app->tpl->setVar("form_action", $app->searchform->listDef["file"]); + // Parse the templates and send output to the browser $this->onShowEnd(); } - + function onShowEnd() { global $app; if(count($_REQUEST) > 0) { - $app->tpl->setVar('searchresult_visible',1); - if($_REQUEST['searchresult_visible'] == 'no') $app->tpl->setVar('searchresult_visible',0); - + $app->tpl->setVar('searchresult_visible', 1); + if($_REQUEST['searchresult_visible'] == 'no') $app->tpl->setVar('searchresult_visible', 0); + if($_REQUEST['searchform_visible'] == 'yes') { - $app->tpl->setVar('searchform_visible',1); + $app->tpl->setVar('searchform_visible', 1); } else { - $app->tpl->setVar('searchform_visible',0); + $app->tpl->setVar('searchform_visible', 0); } } else { - $app->tpl->setVar('searchform_visible',1); - if($_REQUEST['searchform_visible'] == 'no') $app->tpl->setVar('searchform_visible',0); - + $app->tpl->setVar('searchform_visible', 1); + if($_REQUEST['searchform_visible'] == 'no') $app->tpl->setVar('searchform_visible', 0); + if($_REQUEST['searchresult_visible'] == 'yes') { - $app->tpl->setVar('searchresult_visible',1); + $app->tpl->setVar('searchresult_visible', 1); } else { - $app->tpl->setVar('searchresult_visible',0); + $app->tpl->setVar('searchresult_visible', 0); } } - + // make columns visible - $visible_columns = explode(",",$app->searchform->listDef['default_columns']); + $visible_columns = explode(",", $app->searchform->listDef['default_columns']); foreach($visible_columns as $col) { - $app->tpl->setVar($col.'_visible',1); + $app->tpl->setVar($col.'_visible', 1); } - + $app->tpl_defaults(); $app->tpl->pparse(); } + } -?> \ No newline at end of file +?> -- Gitblit v1.9.1