WCF-Abhängigkeit von Abstract(XML)PackageInstallationPlugin reduzieren

Matthias Schmidt · Feb 14th 2011

Hallo,

ja ich beschäftige mich momentan viel mit PackageInstallationPlugins ;).

Folgende Ausgangssituation sei gegeben: Ich entwickele eine Endanwendung (ASA), in der ein Menü vorkommt, dass ich analog zum UserProfileMenu abstrahieren möchte. Die entsprechende Datenbank-Tabelle bekommt natürlich dann auch das Präfix asa1_1. Leider unterstützen die Methoden AbstractPackageInstallationPlugin::hasUninstall(), AbstractPackageInstallationPlugin::uninstall() und AbstractXMLPackageInstallationPlugin::getShowOrder() nur wcf-Tabellen, so dass ich in meinem eigenen PackageInstallationPlugin all diese Methoden überschreiben müsste, der Inhalt der Methoden aber immer fast der gleiche zur eigentlichen Methode wäre, ich nur die entsprechenden Datenbankabfragen anpassen müsste.

Schön wäre es deshalb, wenn die entsprechenden Daten (Paketabkürzung und -nummer) in Attribute ausgelagert würden, die man dann nur noch (dynamisch) setzen müsste, um die Methoden wiederverwenden zu können.

Matthias Schmidt · Feb 19th 2011

Hallo,

ich habe die von mir vorgeschlagene Änderungen mal ausprobiert, indem ich ein WCF installiert habe, die entsprechenden Dateien wie folgt angepasst habe:

Diff

--- /wcf/lib/acp/package/plugin/AbstractPackageInstallationPlugin.class.php	2011-02-15 12:07:02.000000000 +0100
+++ /wcf/lib/acp/package/plugin/AbstractPackageInstallationPlugin.class.php	2011-02-19 17:02:42.000000000 +0100
@@ -30,6 +30,20 @@
 	public $tableName = '';

 	/**
+	 * Database table number.
+	 *
+	 * @var string
+	 */
+	public $tableNumber = WCF_N;
+	
+	/**
+	 * Database table prefix.
+	 *
+	 * @var string
+	 */
+	public $tablePrefix = 'wcf';
+	
+	/**
 	 * Active instance of PackageInstallationQueue
 	 *
 	 * @var PackageInstallationQueue
@@ -94,7 +108,7 @@
 		EventHandler::fireAction($this, 'hasUninstall');

 		$sql = "SELECT	COUNT(*) AS count
-			FROM	wcf".WCF_N."_".$this->tableName."
+			FROM	".$this->tablePrefix.$this->tableNumber."_".$this->tableName."
 			WHERE	packageID = ".$this->installation->getPackageID();
 		$installationCount = WCF::getDB()->getFirstRow($sql);
 		return $installationCount['count'];
@@ -107,7 +121,7 @@
 		// call uninstall event
 		EventHandler::fireAction($this, 'uninstall');

-		$sql = "DELETE FROM	wcf".WCF_N."_".$this->tableName."
+		$sql = "DELETE FROM	".$this->tablePrefix.$this->tableNumber."_".$this->tableName."
 			WHERE		packageID = ".$this->installation->getPackageID();
 		WCF::getDB()->sendQuery($sql);
 	}

Display More

Diff

--- wcf/lib/acp/package/plugin/AbstractXMLPackageInstallationPlugin.class.php	2011-02-15 12:07:02.000000000 +0100
+++ wcf/lib/acp/package/plugin/AbstractXMLPackageInstallationPlugin.class.php	2011-02-19 16:41:18.000000000 +0100
@@ -74,7 +74,7 @@
 		if ($showOrder === null) {
 	        	 // get greatest showOrder value
 	          	$sql = "SELECT	MAX(showOrder) AS showOrder
-			  	FROM	wcf".WCF_N."_".$this->tableName.$tableNameExtension." 
+			  	FROM	".$this->tablePrefix.$this->tableNumber."_".$this->tableName.$tableNameExtension." 
 				".($columnName !== null ? "WHERE ".$columnName." = '".escapeString($parentName)."'" : "");
 			$maxShowOrder = WCF::getDB()->getFirstRow($sql);
 			if (is_array($maxShowOrder) && isset($maxShowOrder['showOrder'])) {
@@ -86,7 +86,7 @@
 	       	}
 	       	else {
 			// increase all showOrder values which are >= $showOrder
-			$sql = "UPDATE	wcf".WCF_N."_".$this->tableName.$tableNameExtension."
+			$sql = "UPDATE	".$this->tablePrefix.$this->tableNumber."_".$this->tableName.$tableNameExtension."
 				SET	showOrder = showOrder+1
 				WHERE	showOrder >= ".$showOrder." 
 				".($columnName !== null ? "AND ".$columnName." = '".escapeString($parentName)."'" : "");

Display More

und dann die ganzen LGPL-WCF-Pakete installiert habe und abschließend noch eine Woltlab Burning Board Lite 2.1-Instanz, alles ohne Probleme (wobei durch die Änderungen eigentlich auch keine auftreten sollten).

WCF-Abhängigkeit von Abstract(XML)PackageInstallationPlugin reduzieren

Participate now!

Share