diff --git a/interface/lib/app.inc.php b/interface/lib/app.inc.php
index 7a382a426d34c4ced591e3b51d3afc36220b9d14..7c63a560270516a2c0fe154779ef38bd72f8a665 100644
--- a/interface/lib/app.inc.php
+++ b/interface/lib/app.inc.php
@@ -59,9 +59,9 @@ class app {
 			session_start();
 			
 			//* Initialize session variables
-			if(!isset($_SESSION['s']['id']) ) $_SESSION["s"]['id'] = session_id();
-			if(empty($_SESSION["s"]["theme"])) $_SESSION["s"]['theme'] = $conf['theme'];
-			if(empty($_SESSION["s"]["language"])) $_SESSION["s"]['language'] = $conf['language'];
+			if(!isset($_SESSION['s']['id']) ) $_SESSION['s']['id'] = session_id();
+			if(empty($_SESSION['s']['theme'])) $_SESSION['s']['theme'] = $conf['theme'];
+			if(empty($_SESSION['s']['language'])) $_SESSION['s']['language'] = $conf['language'];
 		}
 	}
 
diff --git a/interface/lib/classes/listform.inc.php b/interface/lib/classes/listform.inc.php
index 41dfaff12886d0f0359bf68994c1c372bb02c23f..27d1c8a977e6568fbec2b85caad8be837b254ce1 100644
--- a/interface/lib/classes/listform.inc.php
+++ b/interface/lib/classes/listform.inc.php
@@ -38,307 +38,306 @@ EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 
 class listform {
 
-        var $debug = 0;
-        var $errorMessage;
-    var $listDef;
-        var $searchValues;
-        var $pagingHTML;
-        var $pagingValues;
-        var $searchChanged = 0;
-        var $module;
-		var $dateformat = 'd.m.Y';
-
-    function loadListDef($file,$module = '') {
-                global $app,$conf;
-                if(!is_file($file)) die("List-Definition: $file not found.");
-                include_once($file);
-                $this->listDef = $liste;
-                $this->module = $module;
-				
-				// Fill datasources
-				foreach($this->listDef["item"] as $key => $field) {
-					if(@is_array($field['datasource'])) {
-                    	$this->listDef["item"][$key]["value"] = $this->getDatasourceData($field);
-                    }
-				}
-				
-                return true;
-        }
+    private $debug = 0;
+    private $errorMessage;
+    public  $listDef;
+    public  $searchValues;
+    public  $pagingHTML;
+    private $pagingValues;
+    private $searchChanged = 0;
+    private $module;
+	private $dateformat = 'd.m.Y';
+
+    public function loadListDef($file,$module = '')
+    {
+        global $app,$conf;
+        if(!is_file($file)) die("List-Definition: $file not found.");
+        require_once($file);
+        $this->listDef = $liste;
+        $this->module = $module;
 		
-		/**
-        * Get the key => value array of a form filed from a datasource definitiom
-        *
-        * @param field = array with field definition
-        * @param record = Dataset as array
-        * @return key => value array for the value field of a form
-        */
-
-        function getDatasourceData($field) {
-                global $app;
-
-                $values = array();
-
-                if($field["datasource"]["type"] == 'SQL') {
-
-                        // Preparing SQL string. We will replace some
-                        // common placeholders
-                        $querystring = $field["datasource"]["querystring"];
-                        $querystring = str_replace("{USERID}",$_SESSION["s"]["user"]["userid"],$querystring);
-                        $querystring = str_replace("{GROUPID}",$_SESSION["s"]["user"]["default_group"],$querystring);
-                        $querystring = str_replace("{GROUPS}",$_SESSION["s"]["user"]["groups"],$querystring);
-                        //$table_idx = $this->formDef['db_table_idx'];
-                        //$querystring = str_replace("{RECORDID}",$record[$table_idx],$querystring);
-						$app->uses("tform");
-                        $querystring = str_replace("{AUTHSQL}",$app->tform->getAuthSQL('r'),$querystring);
-
-                        // Getting the records
-                        $tmp_records = $app->db->queryAllRecords($querystring);
-                        if($app->db->errorMessage != '') die($app->db->errorMessage);
-                        if(is_array($tmp_records)) {
-                                $key_field = $field["datasource"]["keyfield"];
-                                $value_field = $field["datasource"]["valuefield"];
-                                foreach($tmp_records as $tmp_rec) {
-                                        $tmp_id = $tmp_rec[$key_field];
-                                        $values[$tmp_id] = $tmp_rec[$value_field];
-                                }
-                        }
-                }
-
-                if($field["datasource"]["type"] == 'CUSTOM') {
-                        // Calls a custom class to validate this record
-                        if($field["datasource"]['class'] != '' and $field["datasource"]['function'] != '') {
-                                $datasource_class = $field["datasource"]['class'];
-                                $datasource_function = $field["datasource"]['function'];
-                                $app->uses($datasource_class);
-								$record = array();
-                                $values = $app->$datasource_class->$datasource_function($field, $record);
-                        } else {
-                                $this->errorMessage .= "Custom datasource class or function is empty<br>\r\n";
-                        }
+		//* Fill datasources
+		foreach($this->listDef['item'] as $key => $field) {
+			if(@is_array($field['datasource'])) {
+                $this->listDef['item'][$key]['value'] = $this->getDatasourceData($field);
+            }
+		}
+        return true;
+    }
+		
+	/**
+    * Get the key => value array of a form filed from a datasource definitiom
+    *
+    * @param field = array with field definition
+    * @param record = Dataset as array
+    * @return array key => value array for the value field of a form
+    */
+    private function getDatasourceData($field)
+    {
+        global $app;
+        $values = array();
+
+        if($field['datasource']['type'] == 'SQL') {
+
+            //** Preparing SQL string. We will replace some common placeholders
+            $querystring = $field['datasource']['querystring'];
+            $querystring = str_replace('{USERID}', $_SESSION['s']['user']['userid'], $querystring);
+            $querystring = str_replace('{GROUPID}', $_SESSION['s']['user']['default_group'], $querystring);
+            $querystring = str_replace('{GROUPS}', $_SESSION['s']['user']['groups'], $querystring);
+            //TODO:
+            //$table_idx = $this->formDef['db_table_idx'];
+            //$querystring = str_replace("{RECORDID}",$record[$table_idx],$querystring);
+			$app->uses('tform');
+            $querystring = str_replace("{AUTHSQL}",$app->tform->getAuthSQL('r'),$querystring);
+
+            //* Getting the records
+            $tmp_records = $app->db->queryAllRecords($querystring);
+            if($app->db->errorMessage != '') die($app->db->errorMessage);
+            if(is_array($tmp_records)) {
+                $key_field = $field['datasource']['keyfield'];
+                $value_field = $field['datasource']['valuefield'];
+                foreach($tmp_records as $tmp_rec) {
+                    $tmp_id = $tmp_rec[$key_field];
+                    $values[$tmp_id] = $tmp_rec[$value_field];
                 }
-
-                return $values;
-
+            }
         }
 
-        function getSearchSQL($sql_where = "") {
-                global $db;
-
-                // Hole Config Variablen
-                $list_name = $this->listDef["name"];
-                $search_prefix = $this->listDef["search_prefix"];
-
-                // speichere Suchanfrage
-                foreach($this->listDef["item"] as $i) {
-                        $field = $i["field"];
-
-                        // hat sich die suche geändert
-                        if(isset($_REQUEST[$search_prefix.$field]) and $_REQUEST[$search_prefix.$field] != $_SESSION["search"][$list_name][$search_prefix.$field]) $this->searchChanged = 1;
-
-                        // suchfeld in session speichern.
-                        if(isset($_REQUEST[$search_prefix.$field])) $_SESSION["search"][$list_name][$search_prefix.$field] = $_REQUEST[$search_prefix.$field];
-
-                        if($i["formtype"] == "SELECT") {
-                                if(is_array($i['value'])) {
-                                        $out = '<option value=""></option>';
-                                        foreach($i['value'] as $k => $v) {
-                                                $selected = (isset($_SESSION["search"][$list_name][$search_prefix.$field]) && $k == $_SESSION["search"][$list_name][$search_prefix.$field] && $_SESSION["search"][$list_name][$search_prefix.$field] != '')?' SELECTED':'';
-                                                $out .= "<option value='$k'$selected>$v</option>\r\n";
-                                        }
-                                }
-                                $this->searchValues[$search_prefix.$field] = $out;
-                        } else {
-                                if(isset($_SESSION["search"][$list_name][$search_prefix.$field])) $this->searchValues[$search_prefix.$field] = $_SESSION["search"][$list_name][$search_prefix.$field];
-                        }
-                }
-
-                // Speichere Variablen in Objekt zum späteren einparsen in Template
-                // $this->searchValues = $_SESSION["search"][$list_name];
-
-                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(isset($_SESSION["search"][$list_name][$search_prefix.$field]) && $_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($field['datasource']['type'] == 'CUSTOM') {
+            //* Calls a custom class to validate this record
+            if($field['datasource']['class'] != '' and $field['datasource']['function'] != '') {
+                $datasource_class = $field['datasource']['class'];
+                $datasource_function = $field['datasource']['function'];
+                $app->uses($datasource_class);
+				$record = array();
+                $values = $app->$datasource_class->$datasource_function($field, $record);
+            } else {
+                $this->errorMessage .= "Custom datasource class or function is empty<br>\r\n";
+            }
+        }
+        return $values;
+    }
+
+    public function getSearchSQL($sql_where = '') 
+    {
+        global $db;
+
+        //* Get config variable
+        $list_name = $this->listDef['name'];
+        $search_prefix = $this->listDef['search_prefix'];
+
+        //* store retrieval query
+        foreach($this->listDef['item'] as $i) {
+            $field = $i['field'];
+
+            //*TODO: comment =  hat sich die suche ge�ndert
+            if(isset($_REQUEST[$search_prefix.$field]) and $_REQUEST[$search_prefix.$field] != $_SESSION['search'][$list_name][$search_prefix.$field]){
+                    $this->searchChanged = 1;
+            }
+
+            //* Store field in session
+            if(isset($_REQUEST[$search_prefix.$field])){
+                $_SESSION['search'][$list_name][$search_prefix.$field] = $_REQUEST[$search_prefix.$field];
+            }
+
+            if(isset($i['formtype']) && $i['formtype'] == 'SELECT'){
+                if(is_array($i['value'])) {
+                    $out = '<option value=""></option>';
+                    foreach($i['value'] as $k => $v) {
+                        // TODO: this could be more elegant
+                        $selected = (isset($_SESSION['search'][$list_name][$search_prefix.$field]) 
+                                        && $k == $_SESSION['search'][$list_name][$search_prefix.$field] 
+                                        && $_SESSION['search'][$list_name][$search_prefix.$field] != '')
+                                        ? ' SELECTED' : '';
+                        $out .= "<option value='$k'$selected>$v</option>\r\n";
+                    }
                 }
-
-                if($sql_where != '') {
-                        $sql_where = substr($sql_where,0,-3);
-                } else {
-                        $sql_where = "1";
+                    $this->searchValues[$search_prefix.$field] = $out;
+            } else {
+                if(isset($_SESSION['search'][$list_name][$search_prefix.$field])){
+                    $this->searchValues[$search_prefix.$field] = $_SESSION['search'][$list_name][$search_prefix.$field];
                 }
-
-
-                return $sql_where;
+            }
         }
 
-        function getPagingSQL($sql_where = "1") {
-                global $app, $conf;
-
-                // Hole Config Variablen
-                $list_name                         = $this->listDef["name"];
-                $search_prefix                 = $this->listDef["search_prefix"];
-                $records_per_page         = $this->listDef["records_per_page"];
-                $table                                 = $this->listDef["table"];
-
-                // setze page auf null, wenn in session nicht gesetzt
-                if(!isset($_SESSION["search"][$list_name]["page"]) || $_SESSION["search"][$list_name]["page"] == '') $_SESSION["search"][$list_name]["page"] = 0;
-
-                // setze page auf wert der request variablen "page"
-                if(isset($_REQUEST["page"])) $_SESSION["search"][$list_name]["page"] = $_REQUEST["page"];
-
-                // page auf 0 setzen, wenn suche sich geändert hat.
-                if($this->searchChanged == 1) $_SESSION["search"][$list_name]["page"] = 0;
-
-                $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"] - 1) / $records_per_page);
-
-
-                $vars["list_file"] = $_SESSION["s"]["module"]["name"].'/'.$this->listDef["file"];
-                $vars["page"] = $_SESSION["search"][$list_name]["page"];
-                $vars["last_page"] = $_SESSION["search"][$list_name]["page"] - 1;
-                $vars["next_page"] = $_SESSION["search"][$list_name]["page"] + 1;
-                $vars["pages"] = $pages;
-                $vars["max_pages"] = $pages + 1;
-                $vars["records_gesamt"] = $record_count["anzahl"];
-                $vars["page_params"] = (isset($this->listDef["page_params"]))? $this->listDef["page_params"]:'';
-				//$vars["module"] = $_SESSION["s"]["module"]["name"];
-
+        //* Store variables in object | $this->searchValues = $_SESSION["search"][$list_name];
+        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(isset($_SESSION['search'][$list_name][$search_prefix.$field]) && $_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($_SESSION["search"][$list_name]["page"] > 0) $vars["show_page_back"] = 1;
-                if($_SESSION["search"][$list_name]["page"] <= $vars["pages"] - 1) $vars["show_page_next"] = 1;
+        return ( $sql_where != '' ) ? $sql_where = substr($sql_where,0,-3) : '1';
+    }
 
-                $this->pagingValues = $vars;
-                $this->pagingHTML = $this->getPagingHTML($vars);
+    public function getPagingSQL($sql_where = '1') 
+    {
+        global $app, $conf;
 
-                $limit_sql = "LIMIT $sql_von, $records_per_page";
+        //* Get Config variables
+        $list_name          = $this->listDef['name'];
+        $search_prefix      = $this->listDef['search_prefix'];
+        $records_per_page   = $this->listDef['records_per_page'];
+        $table              = $this->listDef['table'];
 
-                return $limit_sql;
+        //* set PAGE to zero, if in session not set
+        if(!isset($_SESSION['search'][$list_name]['page']) || $_SESSION['search'][$list_name]['page'] == ''){
+            $_SESSION['search'][$list_name]['page'] = 0;
         }
 
-        function getPagingHTML($vars) {
-                global $app;
-                $content = '<a href="'."javascript:loadContent('".$vars["list_file"].'?page=0'.$vars["page_params"]."');".'"><img src="themes/grey/images/btn_left.png" border="0"></a> &nbsp; ';
-                if(isset($vars["show_page_back"]) && $vars["show_page_back"] == 1) $content .= '<a href="'."javascript:loadContent('".$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(isset($vars["show_page_next"]) && $vars["show_page_next"] == 1) $content .= '<a href="'."javascript:loadContent('".$vars["list_file"].'?page='.$vars["next_page"].$vars["page_params"]."');".'"><img src="themes/grey/images/btn_next.png" border="0"></a> &nbsp; ';
-                $content .= '<a href="'."javascript:loadContent('".$vars["list_file"].'?page='.$vars["pages"].$vars["page_params"]."');".'"> <img src="themes/grey/images/btn_right.png" border="0"></a>';
-
-                return $content;
+        //* set PAGE to worth request variable "PAGE" - ? setze page auf wert der request variablen "page"
+        if(isset($_REQUEST["page"])) $_SESSION["search"][$list_name]["page"] = $_REQUEST["page"];
+
+        //* PAGE to 0 set, if look for themselves ?  page auf 0 setzen, wenn suche sich ge�ndert hat.
+        if($this->searchChanged == 1) $_SESSION['search'][$list_name]['page'] = 0;
+
+        $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'] - 1) / $records_per_page);
+
+
+        $vars['list_file']      = $_SESSION['s']['module']['name'].'/'.$this->listDef['file'];
+        $vars['page']           = $_SESSION['search'][$list_name]['page'];
+        $vars['last_page']      = $_SESSION['search'][$list_name]['page'] - 1;
+        $vars['next_page']      = $_SESSION['search'][$list_name]['page'] + 1;
+        $vars['pages']          = $pages;
+        $vars['max_pages']      = $pages + 1;
+        $vars['records_gesamt'] = $record_count['anzahl'];
+        $vars['page_params']    = (isset($this->listDef['page_params'])) ? $this->listDef['page_params'] : '';
+        //$vars['module'] = $_SESSION['s']['module']['name'];
+
+        if($_SESSION['search'][$list_name]['page'] > 0) $vars['show_page_back'] = 1;
+        if($_SESSION['search'][$list_name]['page'] <= $vars['pages'] - 1) $vars['show_page_next'] = 1;
+
+        $this->pagingValues = $vars;
+        $this->pagingHTML = $this->getPagingHTML($vars);
+
+        //* Return limit sql
+        return "LIMIT $sql_von, $records_per_page";
+    }
+
+    public function getPagingHTML($vars)
+    {
+        global $app;
+        $content = '<a href="'."javascript:loadContent('".$vars['list_file'].'?page=0'.$vars['page_params']."');".'">'
+                    .'<img src="themes/grey/images/btn_left.png" border="0"></a> &nbsp; ';
+        //* Show Back 
+        if(isset($vars['show_page_back']) && $vars['show_page_back'] == 1){
+            $content .= '<a href="'."javascript:loadContent('".$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'].' ';
+        //* Show Next
+        if(isset($vars['show_page_next']) && $vars['show_page_next'] == 1){
+            $content .= '<a href="'."javascript:loadContent('".$vars['list_file'].'?page='.$vars['next_page'].$vars['page_params']."');".'">'
+                        .'<img src="themes/grey/images/btn_next.png" border="0"></a> &nbsp; ';
         }
+        $content .= '<a href="'."javascript:loadContent('".$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"].'">|&lt;&lt; </a>]';
-                if($vars["show_page_back"] == 1) $content .= '[<< <a href="'.$vars["list_file"].'?page='.$vars["last_page"].$vars["page_params"].'">'.$app->lng('Back').'</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"].'">'.$app->lng('Next').' >></a>] ';
-                $content .= '[<a href="'.$vars["list_file"].'?page='.$vars["pages"].$vars["page_params"].'"> &gt;&gt;|</a>]';
-
-                return $content;
+	public function getPagingHTMLasTXT($vars)
+    {
+        global $app;
+        $content = '[<a href="'.$vars['list_file'].'?page=0'.$vars['page_params'].'">|&lt;&lt; </a>]';
+        if($vars['show_page_back'] == 1){
+            $content .= '[<< <a href="'.$vars['list_file'].'?page='.$vars['last_page'].$vars['page_params'].'">'.$app->lng('Back').'</a>] ';
         }
-
-        function getSortSQL() {
-                global $app, $conf;
-
-                // Hole Config Variablen
-                $sort_field = $this->listDef["sort_field"];
-                $sort_direction = $this->listDef["sort_direction"];
-
-                $sql_sort = '';
-
-                if($sort_field != '' && $sort_direction != '') {
-                        $sql_sort = "ORDER BY $sort_field $sort_direction";
-                }
-
-                return $sql_sort;
+        $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'].'">'.$app->lng('Next').' >></a>] ';
         }
-
-        function decode($record) {
-                if(is_array($record)) {
-                        foreach($this->listDef["item"] as $field) {
-                                $key = $field["field"];
-                                switch ($field['datatype']) {
-                                case 'VARCHAR':
-                                        $record[$key] = stripslashes($record[$key]);
-                                break;
-
-                                case 'TEXT':
-                                        $record[$key] = stripslashes($record[$key]);
-                                break;
-
-                                case 'DATE':
-                                        if($record[$key] > 0) {
-                                                $record[$key] = date($this->dateformat,$record[$key]);
-                                        } else {
-											$record[$key] = '';
-										}
-                                break;
-
-                                case 'INTEGER':
-                                        $record[$key] = intval($record[$key]);
-                                break;
-
-                                case 'DOUBLE':
-                                        $record[$key] = $record[$key];
-                                break;
-
-                                case 'CURRENCY':
-                                        $record[$key] = number_format($record[$key], 2, ',', '');
-                                break;
-
-                                default:
-                                        $record[$key] = stripslashes($record[$key]);
-                                }
-								
-                        }
-
-                }
-        return $record;
+        $content .= '[<a href="'.$vars['list_file'].'?page='.$vars['pages'].$vars['page_params'].'"> &gt;&gt;|</a>]';
+        return $content;
+    }
+
+    public function getSortSQL()
+    {
+        global $app, $conf;
+        //* Get config vars
+        $sort_field = $this->listDef['sort_field'];
+        $sort_direction = $this->listDef['sort_direction'];
+        return ($sort_field != '' && $sort_direction != '') ? "ORDER BY $sort_field $sort_direction" : '';
+    }
+
+    public function decode($record) 
+    {
+        if(is_array($record)) {
+            foreach($this->listDef['item'] as $field){
+                $key = $field['field'];
+                switch ($field['datatype']){
+
+                    case 'VARCHAR':
+                    case 'TEXT':
+                        $record[$key] = stripslashes($record[$key]);
+                         break;
+
+                    case 'DATE':
+                        $record[$key] = ($record[$key] > 0) ? date($this->dateformat,$record[$key]) : '';
+                        break;
+
+                    case 'INTEGER':
+                        $record[$key] = intval($record[$key]);
+                        break;
+
+                    case 'DOUBLE':
+                        $record[$key] = $record[$key];
+                        break;
+
+                    case 'CURRENCY':
+                        $record[$key] = number_format($record[$key], 2, ',', '');
+                        break;
+
+                    default:
+                        $record[$key] = stripslashes($record[$key]);
+                }	
+            }
         }
+        return $record;
+    }
+
+    public function encode($record)
+    {
+        if(is_array($record)) {
+            foreach($this->listDef['item'] as $field){
+                $key = $field['field'];
+                switch($field['datatype']){
+
+                    case 'VARCHAR':
+                    case 'TEXT':
+                        if(!is_array($record[$key])) {
+                            $record[$key] = addslashes($record[$key]);
+                        } else {
+                            $record[$key] = implode($this->tableDef[$key]['separator'],$record[$key]);
+                        }
+                        break;
+                    
+                    case 'DATE':
+                        if($record[$key] > 0) {
+                            list($tag,$monat,$jahr) = explode('.',$record[$key]);
+                            $record[$key] = mktime(0,0,0,$monat,$tag,$jahr);
+                        }
+                        break;
 
+                    case 'INTEGER':
+                        $record[$key] = intval($record[$key]);
+                        break;
 
-        function encode($record) {
-
-                if(is_array($record)) {
-                        foreach($this->listDef["item"] as $field) {
-                                $key = $field["field"];
-                                switch ($field['datatype']) {
-                                case 'VARCHAR':
-                                        if(!is_array($record[$key])) {
-                                                $record[$key] = addslashes($record[$key]);
-                                        } else {
-                                                $record[$key] = implode($this->tableDef[$key]['separator'],$record[$key]);
-                                        }
-                                break;
-                                case 'TEXT':
-                                        if(!is_array($record[$key])) {
-                                                $record[$key] = addslashes($record[$key]);
-                                        } else {
-                                                $record[$key] = implode($this->tableDef[$key]['separator'],$record[$key]);
-                                        }
-                                break;
-                                case 'DATE':
-                                        if($record[$key] > 0) {
-                                                list($tag,$monat,$jahr) = explode('.',$record[$key]);
-                                                $record[$key] = mktime(0,0,0,$monat,$tag,$jahr);
-                                        }
-                                break;
-                                case 'INTEGER':
-                                        $record[$key] = intval($record[$key]);
-                                break;
-                                case 'DOUBLE':
-                                        $record[$key] = addslashes($record[$key]);
-                                break;
-                                case 'CURRENCY':
-                                        $record[$key] = str_replace(",",".",$record[$key]);
-                                break;
-                                }
+                    case 'DOUBLE':
+                        $record[$key] = addslashes($record[$key]);
+                        break;
 
-                        }
+                    case 'CURRENCY':
+                        $record[$key] = str_replace(',', '.', $record[$key]);
+                        break;
                 }
-                return $record;
+            }
         }
+        return $record;
+    }
 
 }
 
diff --git a/interface/lib/classes/listform_actions.inc.php b/interface/lib/classes/listform_actions.inc.php
index 804699baa022bc6841eb9cee7417497d876610b3..cfe880c8f080526c9f581779eb2616ca1a2136ee 100644
--- a/interface/lib/classes/listform_actions.inc.php
+++ b/interface/lib/classes/listform_actions.inc.php
@@ -37,21 +37,23 @@ EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 
 class listform_actions {
 	
-	var $id;
-	var $idx_key;
-	var $DataRowColor;
-	var $SQLExtWhere = '';
-	var $SQLOrderBy = '';
+	private $id;
+	private $idx_key;
+	private $DataRowColor;
+	private $SQLExtWhere = '';
+	private $SQLOrderBy = '';
 	
-	function onLoad() {
+	public function onLoad()
+    {
 		global $app, $conf, $list_def_file;
 		
+        //TODO: this is_object checking should be in the $appobject - pedro notes
 		if(!@is_object($app->tpl)) $app->uses('tpl');
 		if(!@is_object($app->listform)) $app->uses('listform');
 		if(!@is_object($app->tform)) $app->uses('tform');
 		
-		// Clear session variable that is used when lists are embedded with the listview plugin
-		$_SESSION["s"]["form"]["return_to"] = '';
+		//* Clear session variable that is used when lists are embedded with the listview plugin
+		$_SESSION['s']['form']['return_to'] = '';
 		
 		// Load list definition
 		$app->listform->loadListDef($list_def_file);
@@ -84,43 +86,43 @@ class listform_actions {
 		
 	}
 	
-	function prepareDataRow($rec) {
+	private function prepareDataRow($rec)
+    {
 		global $app;
 		
 		$rec = $app->listform->decode($rec);
 
-		// Alternating datarow colors
-		$this->DataRowColor = ($this->DataRowColor == "#FFFFFF")?"#EEEEEE":"#FFFFFF";
-		$rec["bgcolor"] = $this->DataRowColor;
+		//* Alternating datarow colors
+		$this->DataRowColor = ($this->DataRowColor == '#FFFFFF') ? '#EEEEEE' : '#FFFFFF';
+		$rec['bgcolor'] = $this->DataRowColor;
 		
-		// substitute value for select fields
-		foreach($app->listform->listDef["item"] as $field) {
-			$key = $field["field"];
-			if($field['formtype'] == "SELECT") {
+		//* substitute value for select fields
+		foreach($app->listform->listDef['item'] as $field) {
+			$key = $field['field'];
+			if(isset($field['formtype']) && $field['formtype'] == 'SELECT') {
 				$rec[$key] = $field['value'][$rec[$key]];
 			}
 		}
 		
-		// The variable "id" contains always the index variable
-		$rec["id"] = $rec[$this->idx_key];
-		
+		//* The variable "id" contains always the index variable
+		$rec['id'] = $rec[$this->idx_key];
 		return $rec;
 	}
 	
-	function getQueryString() {
+	private function getQueryString() {
 		global $app;
-		
-		// Generate the search sql
-		if($app->listform->listDef["auth"] != 'no') {
-			if($_SESSION["s"]["user"]["typ"] == "admin") {
-				$sql_where = "";
+		$sql_where = '';
+
+		//* Generate the search sql
+		if($app->listform->listDef['auth'] != 'no') {
+			if($_SESSION['s']['user']['typ'] == "admin") {
+				$sql_where = '';
 			} else {
-				$sql_where = $app->tform->getAuthSQL('r')." and";
+				$sql_where = $app->tform->getAuthSQL('r').' and';
 			}
-		}
-		
+		}		
 		if($this->SQLExtWhere != '') {
-			$sql_where .= " ".$this->SQLExtWhere." and";
+			$sql_where .= ' '.$this->SQLExtWhere.' and';
 		}
 
 		$sql_where = $app->listform->getSearchSQL($sql_where);
@@ -128,31 +130,31 @@ class listform_actions {
 		
 		$order_by_sql = $this->SQLOrderBy;
 
-		// Generate SQL for paging
+		//* Generate SQL for paging
 		$limit_sql = $app->listform->getPagingSQL($sql_where);
-		$app->tpl->setVar("paging",$app->listform->pagingHTML);
+		$app->tpl->setVar('paging',$app->listform->pagingHTML);
 
-		return "SELECT * FROM ".$app->listform->listDef["table"]." WHERE $sql_where $order_by_sql $limit_sql";
-		
+		return 'SELECT * FROM '.$app->listform->listDef['table']." WHERE $sql_where $order_by_sql $limit_sql";
 	}
 	
 	
-	function onShow() {
+	private function onShow()
+    {
 		global $app;
 		
-		// Language File setzen
-		$lng_file = "lib/lang/".$_SESSION["s"]["language"]."_".$app->listform->listDef['name']."_list.lng";
+		//* Set Language File
+		$lng_file = 'lib/lang/'.$_SESSION['s']['language'].'_'.$app->listform->listDef['name'].'_list.lng';
 		include($lng_file);
 		$app->tpl->setVar($wb);
-		$app->tpl->setVar("form_action",$app->listform->listDef["file"]);
+		$app->tpl->setVar('form_action', $app->listform->listDef['file']);
 		
-		// Parse the templates and send output to the browser
+		//* Parse the templates and send output to the browser
 		$this->onShowEnd();
 	}
 	
-	function onShowEnd() {
+	private function onShowEnd()
+    {
 		global $app;
-		
 		$app->tpl_defaults();
 		$app->tpl->pparse();
 	}