more plugin.xml tweaking and fixes

e107_admin/plugin.php

@@ -11,8 +11,8 @@
 |     GNU General Public License (http://gnu.org).
 |
 |     $Source: /cvs_backup/e107_0.8/e107_admin/plugin.php,v $
-|     $Revision: 1.10 $
+|     $Revision: 1.11 $
-|     $Date: 2008-01-27 01:34:59 $
+|     $Date: 2008-02-01 14:11:27 $
 |     $Author: mcfly_e107 $
 +----------------------------------------------------------------------------+
 */
@@ -523,7 +523,8 @@ function render_plugs($pluginList)
 				$img = "<img src='".e_IMAGE."packs/".$imode."/admin_images/upgrade.png' alt='' />";
 			}
 
-			$plugin_icon = $plug_vars['administration']['icon'] ? "<img src='".e_PLUGIN.$plug_vars['administration']['icon']."' alt='' style='border:0px;vertical-align: bottom; width: 32px; height: 32px' />" : E_32_CAT_PLUG;
+			$icon_src = (isset($plug_vars['plugin_php']) ? e_PLUGIN : $_path).$plug_vars['administration']['icon'];
+			$plugin_icon = $plug_vars['administration']['icon'] ? "<img src='{$icon_src}' alt='' style='border:0px;vertical-align: bottom; width: 32px; height: 32px' />" : E_32_CAT_PLUG;
 
 			if ($plug_vars['administration']['configFile'] && $plug['plugin_installflag'] == true)
 			{

e107_files/shortcode/admin_alt_nav.sc

@@ -60,18 +60,24 @@ if (ADMIN) {
 	$render_plugins = FALSE;
 	include_once(e_HANDLER.'plugin_class.php');
 	$plug = new e107plugin;
-	if($sql -> db_Select("plugin", "*", "plugin_installflag=1 ORDER BY plugin_path")){
+	if($sql -> db_Select("plugin", "*", "plugin_installflag=1 ORDER BY plugin_path"))
-		while($row = $sql -> db_Fetch()){
+	{
-			if(getperms('P'.$row['plugin_id'])){
+		while($row = $sql -> db_Fetch())
-				if($plug->parse_plugin(e_PLUGIN.$row['plugin_path'].'/'))
+		{
+			if(getperms('P'.$row['plugin_id']))
+			{
+				$_path = e_PLUGIN.$row['plugin_path'].'/';
+				if($plug->parse_plugin($_path))
 				{
 					$plug_vars = $plug->plug_vars;
 //					print_a($plug_vars);
 //					exit;
 //					include_once(e_PLUGIN.$row['plugin_path']."/plugin.php");
-					if($plug_vars['administration']['configFile']){
+					if($plug_vars['administration']['configFile'])
+					{
 						$plug_vars['name'] = $tp->toHTML($plug_vars['name'], FALSE, "defs");
-						$plugin_icon = $plug_vars['administration']['iconSmall'] ? "<img src='".e_PLUGIN_ABS.$plug_vars['administration']['iconSmall']."' alt='".$plug_vars['administration']['caption']."' style='border:0px; vertical-align:bottom; width: 16px; height: 16px' />" : E_16_PLUGIN;
+						$icon_src = (isset($plug_vars['plugin_php']) ? e_PLUGIN_ABS : e_PLUGIN_ABS.$row['plugin_path'].'/') .$plug_vars['administration']['iconSmall'];
+						$plugin_icon = $plug_vars['administration']['iconSmall'] ? "<img src='{$icon_src}' alt='".$plug_vars['administration']['caption']."' style='border:0px; vertical-align:bottom; width: 16px; height: 16px' />" : E_16_PLUGIN;
 						$plugin_array[ucfirst($plug_vars['name'])] = adnav_main($plug_vars['name'], e_PLUGIN.$row['plugin_path']."/".$plug_vars['administration']['configFile'], $plugin_icon);
 					}
 					$render_plugins = TRUE;

e107_handlers/plugin_class.php

@@ -11,8 +11,8 @@
 |     GNU General Public License (http://gnu.org).
 |
 |     $Source: /cvs_backup/e107_0.8/e107_handlers/plugin_class.php,v $
-|     $Revision: 1.22 $
+|     $Revision: 1.23 $
-|     $Date: 2008-02-01 11:52:27 $
+|     $Date: 2008-02-01 14:11:27 $
 |     $Author: mcfly_e107 $
 +----------------------------------------------------------------------------+
 */
@@ -230,7 +230,8 @@ class e107plugin
 	*/
 	function getinfo($id, $force=false)
 	{
-		global $sql, $getinfo_results;
+		global $sql;
+		static $getinfo_results;
 		if(!is_array($getinfo_results)) { $getinfo_results = array(); }
 
 		$id = (int)$id;
@@ -238,7 +239,7 @@ class e107plugin
 		{
 			if ($sql->db_Select('plugin', '*', "plugin_id = ".$id))
 			{
-				$getinfo_results[$id]; $sql->db_Fetch();
+				$getinfo_results[$id] = $sql->db_Fetch();
 			}
 			else
 			{
@@ -598,12 +599,22 @@ class e107plugin
 
 	function manage_plugin_xml($id, $function='')
 	{
+		global $sql;
 		$id = (int)$id;
 		$plug = $this->getinfo($id);
 		$path = e_PLUGIN.$plug['plugin_path'].'/';
+		$addons = explode(',', $plug['plugin_addons']);
+		$sql_list = array();
+		foreach($addons as $addon)
+		{
+			if(substr($addon, -4) == '_sql')
+			{
+				$sql_list[] = $addon.'.php';
+			}
+		}
 
 		//Will just install using plugin.php file for now.
-		return $this->install_plugin_php($path);
+		//return $this->install_plugin_php($path);
 
 		//New code to install using plugin.xml below.
 		if(!file_exists($path.'plugin.xml') || $function == '')
@@ -622,26 +633,30 @@ class e107plugin
 			return false;
 		}
 
-		//tables
+		// tables
-		if(($function == 'install' || $function == 'uninstall') || isset($plug_vars['sqlFile']))
+		// This will load each _sql.php file found in the plugin directory and parse it.
+		if(($function == 'install' || $function == 'uninstall') && count($sql_list))
 		{
-			if($sql_data = file_get_contents($path.$plug_vars['sqlFile']))
+			foreach($sql_list as $sql_file)
 			{
-				preg_match_all("/create(.*?)myisam.*?;/si", $sql_data, $result );
+				if($sql_data = file_get_contents($path.$sql_file))
-				foreach ($result[0] as $sql_table)
 				{
-					if($function == 'uninstall')
+					preg_match_all("/create(.*?)myisam.*?;/si", $sql_data, $result );
+					foreach ($result[0] as $sql_table)
 					{
-						preg_match("/CREATE TABLE(.*?)\(/si", $sql_table, $match);
+						if($function == 'uninstall')
-						$tablename = trim($match[1]);
+						{
-						echo "Removing table $tablename <br />";
+							preg_match("/CREATE TABLE(.*?)\(/si", $sql_table, $match);
-//						$this->manage_tables('remove', array($tablename));
+							$tablename = trim($match[1]);
-					}
+							echo "Removing table $tablename <br />";
-					if($function == 'install')
+							//	$this->manage_tables('remove', array($tablename));
-					{
+						}
-						$sql_table = preg_replace("/create table\s+/si", "CREATE TABLE #", $sql_table);
+						if($function == 'install')
-						echo "Adding table: <pre>{$sql_table}</pre><br />";
+						{
-//						$this->manage_tables('add', array($sql_table));
+							$sql_table = preg_replace("/create table\s+/si", "CREATE TABLE #", $sql_table);
+							echo "Adding table: <pre>{$sql_table}</pre><br />";
+							//	$this->manage_tables('add', array($sql_table));
+						}
 					}
 				}
 			}
@@ -662,26 +677,26 @@ class e107plugin
 				{
 					case 'upgrade':
 					case 'install':
-						// Add any active link
+					// Add any active link
-						if(!isset($attrib['active']) || $attrib['active'] == 'true')
+					if(!isset($attrib['active']) || $attrib['active'] == 'true')
-						{
+					{
-							$perm = (isset($attrib['perm']) ? $attrib['perm'] : 0);
+						$perm = (isset($attrib['perm']) ? $attrib['perm'] : 0);
-							echo "Adding link {$attrib['name']} with url [{$attrib['url']}] and perm {$perm} <br />";
+						echo "Adding link {$attrib['name']} with url [{$attrib['url']}] and perm {$perm} <br />";
-//							manage_link('add', $attrib['url'], $attrib['name'], $perm);
+						//	manage_link('add', $attrib['url'], $attrib['name'], $perm);
-						}
+					}
-						//remove inactive links on upgrade
+					//remove inactive links on upgrade
-						if($function == 'upgrade' && isset($attrib['active']) && $attrib['active'] == 'false')
+					if($function == 'upgrade' && isset($attrib['active']) && $attrib['active'] == 'false')
-						{
+					{
-							echo "Removing link {$attrib['name']} with url [{$attrib['url']}] <br />";
+						echo "Removing link {$attrib['name']} with url [{$attrib['url']}] <br />";
-//							manage_link('remove', $attrib['url'], $attrib['name']);
+						//	manage_link('remove', $attrib['url'], $attrib['name']);
-						}
+					}
-						break;
+					break;
 
 					case 'uninstall':
-						//remove all links
+					//remove all links
-						echo "Removing link {$attrib['name']} with url [{$attrib['url']}] <br />";
+					echo "Removing link {$attrib['name']} with url [{$attrib['url']}] <br />";
-//						manage_link('remove', $attrib['url'], $attrib['name']);
+					//	manage_link('remove', $attrib['url'], $attrib['name']);
-						break;
+					break;
 				}
 			}
 		}
@@ -699,11 +714,10 @@ class e107plugin
 				{
 					$pref_list = $plug_vars['mainPrefs']['pref'];
 				}
-				
 				$list = array();
-				foreach($pref_list as $pref)
+				foreach($pref_list as $_pref)
 				{
-					$attrib = $pref['@attributes'];
+					$attrib = $_pref['@attributes'];
 					$list['all'][$attrib['name']] = $attrib['value'];
 					if(!isset($attrib['active']) || $attrib['active'] == 'true')
 					{
@@ -718,25 +732,28 @@ class e107plugin
 				{
 					case 'install':
 					case 'upgrade':
-						if(!isset($attrib['active']) || $attrib['active'] == 'true')
+					if(is_array($list['active']))
-						{
+					{
-							echo "Adding prefs ".print_a($list['active'], true)."<br />";
+						echo "Adding prefs ".print_a($list['active'], true)."<br />";
-//							manage_prefs('add', $list['active']);
+						//	manage_prefs('add', $list['active']);
-						}
+					}
 
-						//If upgrading, removing any inactive pref
+					//If upgrading, removing any inactive pref
-						if($function == 'upgrade')
+					if($function == 'upgrade' && is_array($list['inactive']))
-						{
+					{
-							echo "Removing prefs ".print_a($list['inactive'], true)."<br />";
+						echo "Removing prefs ".print_a($list['inactive'], true)."<br />";
-//							manage_prefs('remove', $list['inactive']);
+						//	manage_prefs('remove', $list['inactive']);
-						}
+					}
-						break;
+					break;
 
 					//If uninstalling, remove all prefs (active or inactive)
 					case 'uninstall':
+					if(is_array($list['all']))
+					{
 						echo "Removing prefs ".print_a($list['all'], true)."<br />";
-//						manage_prefs('remove', $list['all']);
+						//	manage_prefs('remove', $list['all']);
-						break;
+					}
+					break;
 				}
 			}
 		}
@@ -745,43 +762,46 @@ class e107plugin
 		//$this->manage_userclass('add', $eplug_userclass, $eplug_userclass_description);
 		if(isset($plug_vars['userclasses']))
 		{
-
 			if(isset($plug_vars['userclasses']['userclass']))
 			{
 				if(!is_array($plug_vars['userclasses']['userclass']))
 				{
-					$uclass_list = array($plug_vars['mainPrefs']['pref']);
+					$uclass_list = array($plug_vars['userclasses']['userclass']);
 				}
 				else
 				{
-					$uclass_list = $plug_vars['mainPrefs']['pref'];
+					$uclass_list = $plug_vars['userclasses']['userclass'];
 				}
+				print_a($uclass_list);
+				
 				foreach($uclass_list as $uclass)
 				{
+					print_a($uclass);
 					$attrib = $uclass['@attributes'];
 					switch($function)
 					{
 						case 'install':
 						case 'upgrade':
-							if(!isset($attrib['active']) || $attrib['active'] == 'true')
+						// Add all active userclasses						
-							{
+						if(!isset($attrib['active']) || $attrib['active'] == 'true')
-								echo "Adding userclass ".$attrib['name']."<br />";
+						{
-//								manage_userclass('add', $attrib['name'], $attrib['description']);
+							echo "Adding userclass ".$attrib['name']."<br />";
-							}
+							//	manage_userclass('add', $attrib['name'], $attrib['description']);
+						}
 
-							//If upgrading, removing any inactive pref
+						//If upgrading, removing any inactive userclass
-							if($function == 'upgrade' && isset($attrib['active']) && $attrib['active'] == 'false')
+						if($function == 'upgrade' && isset($attrib['active']) && $attrib['active'] == 'false')
-							{
+						{
-								echo "Removing userclass ".$attrib['name']."<br />";
+							echo "Removing userclass ".$attrib['name']."<br />";
-//								manage_userclass('remove', $attrib['name'], $attrib['description']);
+							//	manage_userclass('remove', $attrib['name'], $attrib['description']);
-							}
+						}
-							break;
+						break;
 
 						//If uninstalling, remove all userclasses (active or inactive)
 						case 'uninstall':
-							echo "Removing prefs ".$attrib['name']."<br />";
+						echo "Removing prefs ".$attrib['name']."<br />";
-//							manage_userclass('remove', $attrib['name'], $attrib['description']);
+						// manage_userclass('remove', $attrib['name'], $attrib['description']);
-							break;
+						break;
 					}
 				}
 			}
@@ -810,7 +830,6 @@ class e107plugin
 
 	}
 
-
 	function install_plugin_php($id)
 	{
 		global $sql;
@@ -820,7 +839,7 @@ class e107plugin
 
 		$plug['plug_action'] = 'install';
 
-		//		$plug_vars = $this->parse_plugin_php($path);
+		//	$plug_vars = $this->parse_plugin_php($path);
 		include_once($path.'plugin.php');
 
 		$func = $eplug_folder.'_install';
@@ -941,7 +960,7 @@ class e107plugin
 			$_path = e_PLUGIN.$plug['plugin_path'].'/';
 			if(file_exists($_path.'plugin.xml'))
 			{
-				$text = $this->manage_plugin_xml($_path, 'install');
+				$text = $this->manage_plugin_xml($id, 'install');
 			}
 			elseif(file_exists($_path.'plugin.php'))
 			{
@@ -1199,9 +1218,10 @@ class e107plugin
 		include_lan($path.'languages/admin/'.e_LANGUAGE.'.php');
 		require_once(e_HANDLER.'xml_class.php');
 		$xml = new xmlClass;
-		$xml->loadXMLfile($path.'plugin.xml', true, true);
+		$this->plug_vars = $xml->loadXMLfile($path.'plugin.xml', true, true);
-		$xml->xmlFileContents = $tp->replaceConstants($xml->xmlFileContents, '', true);
+//		$xml->loadXMLfile($path.'plugin.xml', true, true);
-		$this->plug_vars = $xml->parseXml();
+//		$xml->xmlFileContents = $tp->replaceConstants($xml->xmlFileContents, '', true);
+//		$this->plug_vars = $xml->parseXml();
 		return true;
 	}
 

e107_handlers/xml_class.php

@@ -11,9 +11,9 @@
 |     GNU General Public License (http://gnu.org).
 |
 |     $Source: /cvs_backup/e107_0.8/e107_handlers/xml_class.php,v $
-|     $Revision: 1.4 $
+|     $Revision: 1.5 $
-|     $Date: 2008-01-30 20:33:16 $
+|     $Date: 2008-02-01 14:11:27 $
-|     $Author: secretr $
+|     $Author: mcfly_e107 $
 +----------------------------------------------------------------------------+
 */
 
@@ -120,7 +120,7 @@ class xmlClass
 		return $xml;
 	}
 
-	function loadXMLfile($fname='', $parse = false)
+	function loadXMLfile($fname='', $parse = false, $replace_constants = false)
 	{
 
 		if($fname == '')
@@ -142,6 +142,16 @@ class xmlClass
 		}
 		if($this->xmlFileContents)
 		{
+			if($replace_constants == true)
+			{
+				global $tp;
+				if(!is_object($tp))
+				{
+					require_once('e_parse_class.php');
+					$tp = new e_parse;
+				}
+				$this->xmlFileContents = $tp->replaceConstants($this->xmlFileContents, '', true);
+			}
 			if($parse == true)
 			{
 				return $this->parseXML();

e107_plugins/forum/plugin.xml

@@ -10,11 +10,9 @@
 	<folder>forum</folder>
 	<administration>
 		<configFile>forum_admin.php</configFile>
-		<icon>forum/images/forums_32.png</icon>
+		<icon>images/forums_32.png</icon>
-		<iconSmall>forum/images/images/forums_16.png</iconSmall>
+		<iconSmall>images/images/forums_16.png</iconSmall>
 		<caption>Configure Forum</caption>
-		<latest>true</latest>
-		<status>true</status>
 		<installDone>Your forum is now installed</installDone>
 	</administration>
 	<menuLink name="Forum" url="forum/forum.php" perm='everyone'/>
@@ -35,7 +33,6 @@
 	<userclasses>
 		<userclass name="forum_moderator" description="Moderator of all forums" />
 	</userclasses>
-	<sqlFile>forum_sql.php</sqlFile>
 	<management>
 		<install type="classFunction" file="forum_management.php" class="forum_management" function="forum_install" />
 		<uninstall type="classFunction" file="forum_management.php" class="forum_management" function="forum_uninstall" />

e107_plugins/pm/plugin.xml

@@ -10,11 +10,10 @@
 	<folder>pm</folder>
 	<administration>
 		<configFile>pm_conf.php</configFile>
-		<icon>pm/images/pvt_message_32.png</icon>
+		<icon>images/pvt_message_32.png</icon>
-		<iconSmall>pm/images/pvt_message_16.png</iconSmall>
+		<iconSmall>images/pvt_message_16.png</iconSmall>
 		<caption>{ADLAN_PM_2}</caption>
 	</administration>
-	<sqlFile>pm_sql.php</sqlFile>
 	<management>
 		<install type="fileFunction" file="pm_install.php" function="forum_install" />
 		<uninstall type="fileFunction" file="pm_uninstall.php" function="forum_uninstall" />