Restructured session start code so it doesnt check every hit, silenced some warnings

Author: pedro_morgan

TODO.txt

@@ -32,7 +32,7 @@ Mail module
 DNS module
 --------------------------------------
 
-- Add some kind of whizard to create DNS records easily. The idea is to have some
+- Add some kind of wizard to create DNS records easily. The idea is to have some
   kind of template(s) to create all needed A and MX records defined in the template
   with one click and the user has just to enter IP, domain, nameserver and select the
   template that he wants to use.
@@ -43,7 +43,7 @@ Administration module
 
 - Add a firewall configuration form. Any suggestions for a good firewall
   script that runs on many linux distributions, or shall we stay with bastille
-  firewall thet is used in ISPConfig 2?
+  firewall that is used in ISPConfig 2?
 
 
 Clients module

interface/lib/app.inc.php

@@ -44,12 +44,11 @@ public function __construct() {
 		}
 
 		if($conf['start_session'] == true) {
-				session_start();
-				$_SESSION['s']['id'] = session_id();
-				if(!isset($_SESSION['s']['theme']) || $_SESSION['s']['theme'] == ''){
-					 $_SESSION['s']['theme'] = $conf['theme'];
-				}
-				if($_SESSION['s']['language'] == '') $_SESSION['s']['language'] = $conf['language'];
+			session_start();
+            //* Initialise vars if session is not set
+            if( !isset($_SESSION['s']['id']) ){
+                $_SESSION['s'] = array('id' => session_id(), 'theme' => $conf['theme'], 'language' => $conf['language']);
+            }
 		}
 	}