From c40e1fe9be98f95fa46c83567bdf1c6ba44d190d Mon Sep 17 00:00:00 2001 From: tbrehm <t.brehm@ispconfig.org> Date: Thu, 08 Mar 2007 15:18:23 -0500 Subject: [PATCH] Fixed a bug in the group selector of the user edit form of the admin module. --- interface/lib/classes/listform.inc.php | 26 +++++++++++++++++++++----- 1 files changed, 21 insertions(+), 5 deletions(-) diff --git a/interface/lib/classes/listform.inc.php b/interface/lib/classes/listform.inc.php index e81a256..ea39bee 100644 --- a/interface/lib/classes/listform.inc.php +++ b/interface/lib/classes/listform.inc.php @@ -46,6 +46,7 @@ var $pagingValues; var $searchChanged = 0; var $module; + var $dateformat = 'd.m.Y'; function loadListDef($file,$module = '') { global $app,$conf; @@ -156,7 +157,8 @@ foreach($this->listDef["item"] as $i) { $field = $i["field"]; - if($_REQUEST[$search_prefix.$field] != '') $sql_where .= " $field ".$i["op"]." '".$i["prefix"].$_REQUEST[$search_prefix.$field].$i["suffix"]."' and"; + //if($_REQUEST[$search_prefix.$field] != '') $sql_where .= " $field ".$i["op"]." '".$i["prefix"].$_REQUEST[$search_prefix.$field].$i["suffix"]."' and"; + if($_SESSION["search"][$list_name][$search_prefix.$field] != '') $sql_where .= " $field ".$i["op"]." '".$i["prefix"].$_SESSION["search"][$list_name][$search_prefix.$field].$i["suffix"]."' and"; } if($sql_where != '') { @@ -189,7 +191,7 @@ $sql_von = $_SESSION["search"][$list_name]["page"] * $records_per_page; $record_count = $app->db->queryOneRecord("SELECT count(*) AS anzahl FROM $table WHERE $sql_where"); - $pages = intval($record_count["anzahl"] / $records_per_page); + $pages = intval(($record_count["anzahl"] - 1) / $records_per_page); $vars["list_file"] = $this->listDef["file"]; @@ -203,7 +205,7 @@ if($_SESSION["search"][$list_name]["page"] > 0) $vars["show_page_back"] = 1; - if($_SESSION["search"][$list_name]["page"] <= $seiten - 1) $vars["show_page_next"] = 1; + if($_SESSION["search"][$list_name]["page"] <= $vars["pages"] - 1) $vars["show_page_next"] = 1; $this->pagingValues = $vars; $this->pagingHTML = $this->getPagingHTML($vars); @@ -214,6 +216,17 @@ } function getPagingHTML($vars) { + global $app; + $content = '<a href="'.$vars["list_file"].'?page=0'.$vars["page_params"].'"><img src="../themes/grey/images/btn_left.png" border="0"></a> '; + if($vars["show_page_back"] == 1) $content .= '<a href="'.$vars["list_file"].'?page='.$vars["last_page"].$vars["page_params"].'"><img src="../themes/grey/images/btn_back.png" border="0"></a> '; + $content .= ' '.$app->lng('Page').' '.$vars["next_page"].' '.$app->lng('of').' '.$vars["max_pages"].' '; + if($vars["show_page_next"] == 1) $content .= '<a href="'.$vars["list_file"].'?page='.$vars["next_page"].$vars["page_params"].'"><img src="../themes/grey/images/btn_next.png" border="0"></a> '; + $content .= '<a href="'.$vars["list_file"].'?page='.$vars["pages"].$vars["page_params"].'"> <img src="../themes/grey/images/btn_right.png" border="0"></a>'; + + return $content; + } + + function getPagingHTMLasTXT($vars) { global $app; $content = '[<a href="'.$vars["list_file"].'?page=0'.$vars["page_params"].'">|<< </a>]'; if($vars["show_page_back"] == 1) $content .= '[<< <a href="'.$vars["list_file"].'?page='.$vars["last_page"].$vars["page_params"].'">'.$app->lng('Back').'</a>] '; @@ -254,9 +267,11 @@ break; case 'DATE': - if($val > 0) { + if($record[$key] > 0) { $record[$key] = date($this->dateformat,$record[$key]); - } + } else { + $record[$key] = ''; + } break; case 'INTEGER': @@ -274,6 +289,7 @@ default: $record[$key] = stripslashes($record[$key]); } + } } -- Gitblit v1.9.1