Assembla home | Assembla project page
 

Changeset 278

Show
Ignore:
Timestamp:
08/01/08 21:18:09 (4 months ago)
Author:
Seldaek
Message:

* Fixed a bug in staticPluginLoaded() and updated all calls to it

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved

  • trunk/Phergie/Plugin/Abstract/Base.php

    r275 r278  
    451451     * @return bool 
    452452     */ 
    453     public static function staticPluginLoaded($plugin, $plugins, $client) 
     453    public static function staticPluginLoaded($plugin, $client, array $plugins) 
    454454    { 
    455455        $plugin = trim($plugin); 

  • trunk/Phergie/Plugin/FeedTicker.php

    r275 r278  
    158158            $errors[] = 'SimpleXML php extension is required'; 
    159159        } 
    160         if (self::staticPluginLoaded('TinyUrl', $client, $plugins)) { 
     160        if (!self::staticPluginLoaded('TinyUrl', $client, $plugins)) { 
    161161            $errors[] = 'TinyUrl plugin must be enabled'; 
    162162        } 

  • trunk/Phergie/Plugin/Google.php

    r277 r278  
    1919        $errors = array(); 
    2020 
    21         if (self::staticPluginLoaded('TinyUrl', $client, $plugins)) { 
     21        if (!self::staticPluginLoaded('TinyUrl', $client, $plugins)) { 
    2222            $errors[] = 'TinyUrl plugin must be enabled'; 
    2323        } 
    24         if (self::staticPluginLoaded('Url', $client, $plugins)) { 
     24        if (!self::staticPluginLoaded('Url', $client, $plugins)) { 
    2525            $errors[] = 'Url plugin must be enabled'; 
    2626        } 

  • trunk/Phergie/Plugin/Logging.php

    r251 r278  
    162162     * @return bool TRUE if dependencies are met, FALSE otherwise 
    163163     */ 
    164     public static function checkDependencies(Phergie_Driver_Abstract $client, array$plugins) 
    165     { 
    166         if (self::staticPluginLoaded('ServerInfo', $client, $plugins)) { 
    167             return false; 
    168         } 
    169  
    170         if (!extension_loaded('PDO') || !extension_loaded('pdo_sqlite')) { 
    171             return false; 
    172         } 
    173  
    174         return true; 
     164    public static function checkDependencies(Phergie_Driver_Abstract $client, array $plugins) 
     165    { 
     166        $errors = array(); 
     167 
     168        if (!self::staticPluginLoaded('ServerInfo', $client, $plugins)) { 
     169            $errors[] = 'ServerInfo plugin must be enabled'; 
     170        } 
     171        if (!extension_loaded('PDO')) { 
     172            $errors[] = 'PDO php extension is required'; 
     173        } 
     174        if (!extension_loaded('pdo_sqlite')) { 
     175            $errors[] = 'pdo_sqlite php extension is required'; 
     176        } 
     177 
     178        return empty($errors) ? true : $errors; 
    175179    } 
    176180 

  • trunk/Phergie/Plugin/Php.php

    r275 r278  
    3636    public static function checkDependencies(Phergie_Driver_Abstract $client, array $plugins) 
    3737    { 
    38         if (self::staticPluginLoaded('TinyUrl', $client, $plugins)) { 
     38        if (!self::staticPluginLoaded('TinyUrl', $client, $plugins)) { 
    3939            return 'TinyUrl plugin must be enabled'; 
    4040        } 

  • trunk/Phergie/Plugin/Sed.php

    r273 r278  
    5757    public static function checkDependencies(Phergie_Driver_Abstract $client, array $plugins) 
    5858    { 
    59         if (self::staticPluginLoaded('Logging', $client, $plugins)) { 
     59        if (!self::staticPluginLoaded('Logging', $client, $plugins)) { 
    6060            return 'Logging plugin must be enabled'; 
    6161        } 

  • trunk/Phergie/Plugin/Seen.php

    r273 r278  
    138138    public static function checkDependencies(Phergie_Driver_Abstract $client, array $plugins) 
    139139    { 
    140         if (self::staticPluginLoaded('Logging', $client, $plugins)) { 
     140        if (!self::staticPluginLoaded('Logging', $client, $plugins)) { 
    141141            return 'Logging plugin must be enabled'; 
    142142        } 

  • trunk/Phergie/Plugin/UrbanDictionary.php

    r275 r278  
    1818    public static function checkDependencies(Phergie_Driver_Abstract $client, array $plugins) 
    1919    { 
    20         if (self::staticPluginLoaded('TinyUrl', $client, $plugins)) { 
     20        if (!self::staticPluginLoaded('TinyUrl', $client, $plugins)) { 
    2121            return 'TinyUrl plugin must be enabled'; 
    2222        } 

  • trunk/Phergie/Plugin/Url.php

    r277 r278  
    44 * Monitors incoming messages for instances of URLs and responds with messages 
    55 * containing relevant information about detected URLs. 
    6  *  
     6 * 
    77 * Has an utility method accessible through $this->getPlugin('Url')->getTitle('http://foo..') 
    88 */ 
    …… 
    138138    public static function checkDependencies(Phergie_Driver_Abstract $client, array $plugins) 
    139139    { 
    140         if (self::staticPluginLoaded('TinyUrl', $client, $plugins)) { 
     140        if (!self::staticPluginLoaded('TinyUrl', $client, $plugins)) { 
    141141            return 'TinyUrl plugin must be enabled'; 
    142142        }