Fixes errors in Package class
authorMatthias Schmidt <gravatronics@live.com>
Sun, 30 Dec 2012 15:49:39 +0000 (16:49 +0100)
committerMatthias Schmidt <gravatronics@live.com>
Sun, 30 Dec 2012 15:49:39 +0000 (16:49 +0100)
wcfsetup/install/files/lib/data/package/Package.class.php

index 27378dcd945893aea4a6cfa2b9573836b34f2908..c818c725d63465bdb930d9e19a9f3e80a63224e7 100644 (file)
@@ -21,22 +21,6 @@ use wcf\util\StringUtil;
  * @category   Community Framework
  */
 class Package extends DatabaseObject {
-       /**
-        * @see wcf\data\DatabaseObject::$databaseTableName
-        */
-       protected static $databaseTableName = 'package';
-       
-       /**
-        * @see wcf\data\DatabaseObject::$databaseTableIndexName
-        */
-       protected static $databaseTableIndexName = 'packageID';
-       
-       /**
-        * package requirements
-        * @var array<integer>
-        */
-       protected static $requirements = null;
-       
        /**
         * list of packages that this package requires
         * @var array<wcf\data\package\Package>
@@ -55,15 +39,31 @@ class Package extends DatabaseObject {
         */
        protected $requiredPackages = null;
        
+       /**
+        * @see wcf\data\DatabaseObject::$databaseTableName
+        */
+       protected static $databaseTableName = 'package';
+       
+       /**
+        * @see wcf\data\DatabaseObject::$databaseTableIndexName
+        */
+       protected static $databaseTableIndexName = 'packageID';
+       
+       /**
+        * package requirements
+        * @var array<integer>
+        */
+       protected static $requirements = null;
+       
        /**
         * Returns true, if this package is required by other packages.
         * 
         * @return      boolean
         */
        public function isRequired() {
-               self::loadRequiremens();
+               self::loadRequirements();
                
-               return (isset(self::$requirements[$this->packageID]));
+               return isset(self::$requirements[$this->packageID]);
        }
        
        /**
@@ -177,8 +177,8 @@ class Package extends DatabaseObject {
                self::loadRequirementMap();
                
                $packages = array();
-               if (isset(self::$requirementMap[$this->packageID])) {
-                       foreach (self::$requirementMap[$this->packageID] as $packageID) {
+               if (isset(self::$requirements[$this->packageID])) {
+                       foreach (self::$requirements[$this->packageID] as $packageID) {
                                $packages[$packageID] = PackageCache::getInstance()->getPackage($packageID);
                        }
                }
@@ -190,7 +190,7 @@ class Package extends DatabaseObject {
         * Loads package requirements.
         */
        protected static function loadRequirements() {
-               if (self::$requirementMap === null) {
+               if (self::$requirements === null) {
                        $sql = "SELECT  packageID, requirement
                                FROM    wcf".WCF_N."_package_requirement";
                        $statement = WCF::getDB()->prepareStatement($sql);
@@ -343,7 +343,6 @@ class Package extends DatabaseObject {
        /**
         * Returns a list of plugins for currently active application.
         * 
-        * @todo        Care about simple plugins just providing some crap.
         * @return      wcf\data\package\PackageList
         */
        public static function getPluginList() {