made get_profileOptions() non-static

lam/lib/modules.inc

@@ -24,6 +24,8 @@ $Id$
 
 */
 
+include_once("cache.inc");
+
 /* We have to include all modules
 * before start session
 */
@@ -165,15 +167,14 @@ function getAvailableModules($scope) {
 
 // $scope = user, group, host, ...
 function getProfileOptions($scope) {
-	// get module names.
-	$dir = opendir('../../lib/modules');
-	while ($entry = readdir($dir))
-		if (is_file('../../lib/modules/'.$entry)) {
-			$entry = substr($entry, 0, strpos($entry, '.'));
-			if (call_user_func(array($entry, "can_manage"), $scope)) $return[$entry] = call_user_func(array($entry, "get_profileOptions"), $scope);
-		}
-	return $return;
+	// create new account container if needed
+	if (! isset($_SESSION["profile_account_$scope"])) {
+		$_SESSION["profile_account_$scope"] = new accountContainer($scope, "profile_account_$scope");
+		$_SESSION["profile_account_$scope"]->new_account();
 	}
+	// get options
+	return $_SESSION["profile_account_$scope"]->getProfileOptions();
+}
 
 // checks if the profile options are valid
 // $scope: user, group, host, ...
@@ -224,7 +225,7 @@ class accountContainer {
 	/* This variale stores the type
 	* of account. Current unix, group, host are supported
 	*/
-	var $scope;
+	var $type;
 	var $ldap; // This is a reference to the ldap class in session
 	var $config; // This is a reference to the config class in session
 	// Localized part of HTML-Header
@@ -701,22 +702,14 @@ class accountContainer {
 		return 0;
 		}
 
-	// TODO function can be removed?
-	function display_profile($post) {
+	/**
+	* Returns an hash array containing all profile options
+	*/
+	function getProfileOptions() {
 		$return = array();
-		$post = array();
-		$module = array_keys ($this->module);
-		foreach ($module as $singlemodule) {
-			// get list of display functions.
-			$list = $this->module[$singlemodule]->pages();
-			foreach ($list as $item) {
-				$function = 'display_html_' . $item;
-				$page = $this->module[$singlemodule]->$function($post,true);
-				//eval($function);
-				foreach ( $page as $item )
-					$return[$singlemodule][] = $item;
-				//$return = array_merge($return, $page);
-				}
+		$modules = array_keys($this->module);
+		foreach ($modules as $singlemodule) {
+			$return[$singlemodule] = $this->module[$singlemodule]->get_profileOptions();
 		}
 		return $return;
 	}
@@ -742,8 +735,8 @@ class accountContainer {
 	* for a new account
 	*/
 	function new_account() {
-		$function = '$modules = $_SESSION[$this->config]->get_'.ucfirst($this->type).'Modules();';
-		eval ($function);
+		$temp = ucfirst($this->type);
+		$modules = call_user_func(array(&$_SESSION['config'], 'get_' . $temp . 'Modules'));
 		foreach ($modules as $module) $this->module[$module] = new $module($this->base);
 
 		$module = array_keys ($this->module);

lam/lib/modules/account.inc

@@ -198,8 +198,8 @@ class account {
 		return 0;
 		}
 
-	function get_profileOptions($scope) {
-		if ($scope == "host") return array();
+	function get_profileOptions() {
+		return array();
 	}
 
 	// checks if the values of a new or modified profile are valid

lam/lib/modules/inetOrgPerson.inc

@@ -303,8 +303,8 @@ class inetOrgPerson {
 		return 0;
 		}
 
-	function get_profileOptions($scope) {
-		if ($scope == "user") return array();
+	function get_profileOptions() {
+		return array();
 	}
 
 	// checks if the values of a new or modified profile are valid

lam/lib/modules/main.inc

@@ -286,6 +286,10 @@ class main {
 		return $return;
 		}
 
+	function get_profileOptions() {
+		return array();
+	}
+
 	}
 
 ?>

lam/lib/modules/posixAccount.inc

@@ -669,7 +669,7 @@ class posixAccount {
 		}
 
 
-	function get_profileOptions($scope) {
+	function get_profileOptions() {
 		$return = array();
 		// TODO fill in correct groups
 		$return[] = array (0 => array ('kind' => 'text', 'text' => _('Primary group')),

lam/lib/modules/posixGroup.inc

@@ -585,7 +585,7 @@ class posixGroup {
 		return $return;
 		}
 
-	function get_profileOptions($scope) {
+	function get_profileOptions() {
 		$return = array();
 			$return[] = array (0 => array ( 'kind' => 'text', 'text' => _("Groupname")),
 				1 => array ('kind' => 'input', 'name' => 'cn', 'type' => 'text', 'size' => '20', 'maxlength' => '20'),

lam/lib/modules/quota.inc

@@ -293,7 +293,7 @@ class quota {
 		return 0;
 		}
 
-	function get_profileOptions($scope) {
+	function get_profileOptions() {
 		$return = array();
 		return $return;
 	}

lam/lib/modules/sambaAccount.inc

@@ -606,7 +606,7 @@ class sambaAccount {
 		return $return;
 		}
 
-	function get_profileOptions($scope) {
+	function get_profileOptions() {
 		$return = array();
 			$return[] = array ( 0 => array ( 'kind' => 'text', 'text' => _('Domain') ),
 				1 => array ( 'kind' => 'input', 'type' => 'text', 'name' => 'domain', 'size' => '20', 'maxlength' => '255', 'value' => $this->attributes['domain'][0]),

lam/lib/modules/sambaGroupMapping.inc

@@ -305,7 +305,7 @@ class sambaGroupMapping {
 		return 0;
 		}
 
-	function get_profileOptions($scope) {
+	function get_profileOptions() {
 		$return = array();
 		// TODO fill in correct domains
 		$return[] = array (0 => array ('kind' => 'text', 'text' => _('Domain')),

lam/lib/modules/sambaSamAccount.inc

@@ -621,7 +621,7 @@ class sambaSamAccount {
 		return $return;
 		}
 
-	function get_profileOptions($scope) {
+	function get_profileOptions() {
 		$return = array();
 		// TODO fill in correct groups
 		$return[] = array(0 => array('kind' => 'text', 'text' => _('Domain')),

lam/lib/modules/shadowAccount.inc

@@ -256,7 +256,7 @@ class shadowAccount {
 		return 0;
 		}
 
-	function get_profileOptions($scope) {
+	function get_profileOptions() {
 		$return = array();
 		$return[] = array ( 0 => array ( 'kind' => 'text', 'text' => _('Password warn') ),
 			1 => array ( 'kind' => 'input', 'name' => 'shadowWarning', 'type' => 'text', 'size' => '4', 'maxlength' => '4', 'value' => $this->attributes['shadowWarning'][0] ),