phergie / phergie Goto Github PK

See http://svn2.assembla.com/svn/phergie/trunk/Phergie/Plugin/Weather.php for the 1.x version of this plugin.

In addition to the port, support could be added for sources other than weather.com as the original plugin used. This could be implemented using a driver-based approach where the driver to use could be modified via configuration.

Here are a few options for drivers:
http://svn.itrebal.com/thefuckingweather/
http://wiki.wunderground.com/index.php/API_-_XML
http://code.assembla.com/iweather/subversion/nodes/trunk/third_party/google_weather/google_weather_api.php

The Twitter plugin depends on Phergie_Plugin_Helper_Time, which the plugin does not install when installing via PEAR. Triggering the Twitter plugin will cause Phergie to crash due to a missing include.

See http://svn2.assembla.com/svn/phergie/trunk/Phergie/Plugin/Lart.php for the 1.x version of this plugin. In addition to porting, add support for regular expressions in triggers and responses.

New jokes can be found here: http://www.codesqueeze.com/the-ultimate-top-25-chuck-norris-the-programmer-jokes/

installed Phergie_Plugin_Php (v2.0.0/beta) via pear to a local install of pear (ie not site wide as I don't have root access on that server). Amended the config file for phergie etc.
However, the bot's having problems:
13:36 < radagast> php stripos
13:36 < Phergie17> Search for function stripos returned no results.

http://svn2.assembla.com/svn/phergie/trunk/Phergie/Plugin/ServerInfo.php needs to be ported. May change the name to a more sensible one.

Problem:
In testing with the Ping plugin, which is called onTick, I was not seeing that PING messages were being sent to the server. With further debugging, I saw that the events were enqueued within the event handler, but were not being executed. This was because when there was no event received from the driver (ie: $driver->getEvent() returns null), the loop would cycle to the next connection and not process events. Meanwhile, after each cycle through the connections, onTick() was called. In Ping::onTick() when a certain threshold is met, a self-ping is initiated, which queues it on to the stack. This was then sitting in the event queue until the next event was received from the server.

Regardless of whether we have received an event or not, the event queue needs to be processed to handle events that may be injected during non-event driven callbacks.

The Settings.php.dist file that is used as the baseline shows how to enable plugins, but does not give any information as to what might be a good plugin to enable for basic bot operation, such as Ping, Pong, Ctcp, Quit, etc. The only way a new user would find out about these plugins would be by scanning the Plugin directory to see what's available.

Both the documentation and Settings.php.dist should be updated to provide some suggestions as to which plugins should be enabled for a decent experience with Phergie.

See http://svn2.assembla.com/svn/phergie/trunk/Phergie/Plugin/FeedTicker.php for the 1.x version of this plugin. It should use the new feed parsing, cron, and HTTP plugins once they're created.

This plugin would function in the same way as previous drink plugins, but would throw the user a cookie instead. Here's a potential data source for this plugin: http://www.geocities.com/webcipes/cookie.html. Note that it may be pertinent to exclude brownies and possibly put those into a separate plugin.

See http://svn2.assembla.com/svn/phergie/trunk/Phergie/Plugin/Abstract/Cron.php for the 1.x version of this plugin. This should be modified to be a service plugin similar Command where, rather than being extended, other plugins can call a method of this plugin to register callbacks to be executed on a timed interval.

When running a manual install of Phergie that isn't on the path, the filename appears in $argv, which leads Phergie to try and include it as a config file.

Bittarman: convert 0K to C
barwench: Bittarman: 0 kelvin = -273.15 degrees Celsius
NoiseEee: that shit is COLD
ajt_: convert 77F to C
barwench: ajt_: 77 degrees Fahrenheit = 25 degrees Celsius
DASPRiD: convert -100K to C
barwench:DASPRiD: (-100) kelvin = -373.15 degrees Celsius

below 0 kelvin is impossible, its absolute 0.

See http://trac2.assembla.com/phergie/browser/trunk/Phergie/Plugin/Logging.php for the 1.x version of this plugin.

The ported version should be restricted to only logging data to a local database and providing methods for querying that database (including a filter by channel) to other plugins. (i.e. It shouldn't include functionality from the previous Seen, Heard, Willsee, or Search plugins.)

It must include an option to disable logging of message content (such as the bodies of PRIVMSG or NOTICE events) for privacy purposes. Though not necessary, another handy option would be a way to restrict which messages are logged by event type.

[17:03:52] <Obsidian> http://phergie.org/users/
[17:03:54] <Phergie2> [ http://is.gd/cVLB1 ] Users &mdash; Phergie

Not sure why, but it seems that there is an html entity that is not being accounted for when trimming.

The only way to set the driver that Phergie uses by modifying the script to make a call to setDriver on Phergie_Bot. There is no method available in the configuration to specify the driver of a connection.

See http://svn2.assembla.com/svn/phergie/trunk/Phergie/Plugin/Karma.php for the 1.x version of this plugin.

When querying last.fm or libre.fm, Phergie just says ", was listening to by", even when the user specified is currently listening to something and it should actually say that.

In the manual install, the bot enters the $argc > 0 configuration section. It then pops the first item ('phergie.php') off the stack. A new Phergie_Config is created, and, since no other items were put on the argument list, $bot->setConfig($config) is putting an empty Phergie_Config into the bot. When $bot->getConfig() is called, a Phergie_Config already exists, so it doesn't try to create the new Phergie_Config there and read 'Settings.php'.

Therefore running 'php ./phergie.php' ends up running Phergie with an empty config.

See http://svn2.assembla.com/svn/phergie/trunk/Phergie/Plugin/Spellcheck.php for the 1.x version of this plugin.

In addition to the port, it would be ideal if support for the Enchant extension could be added. See http://php.net/enchant. This could be implemented using a driver-based approach where either a pspell or enchant driver would be selected via configuration or automatically detected based on PHP's configuration if no value was specified.

When a user sends a message containing a URL to a channel (use the Url plugin to detect this), this plugin should use the Delicious API to post that URL to a communal Delicious account, either an account specific to that channel or a general account (in which case it could tag the bookmark for the channel that received the original message).

This plugin could also include a command allowing users to specify a) a URL or the nick of a user; and b) a tag. The specified URL or the last URL posted by the specified user should have the specified tag added to it on Delicious. This would allow people to tag bookmarks for themselves. Bookmarks should by default be tagged with the originating channel and nick.

This should be doable with streams: http://www.php.net/manual/en/wrappers.http.php. For HTTP authentication, use the header context option to send an Authorization header with this value...
'Basic ' . base64_encode($user . ':' . $password)
... where $user and $password are the username and password on the Delicious account. Those should be created as options in the configuration file for the user to specify.

PHP Notice: Undefined index: sec in /home/dmtrsslvdr/project/origami/phergie/Phergie/Process/Async.php on line 73
PHP Stack trace:
PHP 1. {main}() /home/dmtrsslvdr/project/origami/phergie/phergie.php:0
PHP 2. Phergie_Bot->run() /home/dmtrsslvdr/project/origami/phergie/phergie.php:54
PHP 3. Phergie_Bot->getProcessor() /home/dmtrsslvdr/project/origami/phergie/Phergie/Bot.php:379
PHP 4. Phergie_Process_Async->__construct() /home/dmtrsslvdr/project/origami/phergie/Phergie/Bot.php:293
PHP Notice: Undefined index: sec in /home/dmtrsslvdr/project/origami/phergie/Phergie/Process/Async.php on line 78
PHP Stack trace:
PHP 1. {main}() /home/dmtrsslvdr/project/origami/phergie/phergie.php:0
PHP 2. Phergie_Bot->run() /home/dmtrsslvdr/project/origami/phergie/phergie.php:54
PHP 3. Phergie_Bot->getProcessor() /home/dmtrsslvdr/project/origami/phergie/Phergie/Bot.php:379
PHP 4. Phergie_Process_Async->__construct() /home/dmtrsslvdr/project/origami/phergie/Phergie/Bot.php:293
PHP Notice: Undefined property: Phergie_Process_Async::$wait in /home/dmtrsslvdr/project/origami/phergie/Phergie/Process/Async.php on line 146
PHP Stack trace:
PHP 1. {main}() /home/dmtrsslvdr/project/origami/phergie/phergie.php:0
PHP 2. Phergie_Bot->run() /home/dmtrsslvdr/project/origami/phergie/phergie.php:54
PHP 3. Phergie_Process_Async->handleEvents() /home/dmtrsslvdr/project/origami/phergie/Phergie/Bot.php:383

If PEAR_Config is present on the include path, then when Phergie attempts to use its autoloader to load Phergie_Config it instead loads PEAR_Config. This seems to be because Phergie will remove Phergie_ from the class name when determining the path and therefore look for Config.php on the include path. Phergie adds its path to the end of the include path, which means that the autoloader will pick up PEAR_Config before Phergie_Config. To fix the issue, Phergie should prepend it's path to the include_path.

Several plugins; such as TerryChay, BeerScore, Url, and Twitter; use file_get_contents directly as an HTTP client. This should be moved into a separate plugin, mainly to implement better error handling by exposing 4xx and 5xx responses as exceptions that plugins can catch and implementing a more friendly API than streams offers for HTTP.

If you have two netwrok interfaces you should be able to set which one phergie will use. I believe you can do this with the stream context option bindto.

Minor issue, but this error shows up:
PHP Notice: Undefined variable: lower in ... /Phergie/Plugin/Handler.php on line 236

Source code for the 1.x version: http://svn2.assembla.com/svn/phergie/trunk/Phergie/Plugin/Drink.php

1. The Drink plugin should be split up into individual plugins per drink type.
2. Coffee drinks should be filtered from the Coke database into their own.
3. Each Drink plugin should be modified to include URLs to drinks where applicable.
4. Illegal and incomplete multibyte characters are present in cocktail data; proper handling should be added to avoid related notices being generated.
5. Rather than scraping data for drinks on the bot's first execution, data should be scraped into databases that are bundled and updated with each package release. Scripts for this should be retained in the git repository, but need not be bundled in the package release.
6. Some drinks (mainly cocktails) have obscene names, so this will need to integrate with a censorship plugin once they're both ported in order to return drinks with non-obscene names to channels with censorship enabled.

An additional potential data source: http://www.programmableweb.com/api/liquor-control-board-of-ontario

See http://svn2.assembla.com/svn/phergie/trunk/Phergie/Plugin/Remind.php for the old 1.x version of this plugin.

In addition to porting, there may be two outstanding issues to be fixed. (i.e. They may have already been fixed, we're not sure.)

1. When the plugin senses that a user for which a message was previously left has spoken, it outputs the message with the sender's nick included, but that nick has been converted to lowercase. Thanks to Remi Woler for reporting this.
2. Messages aren't delivered consistently; in fact, they often are never delivered at all. Some of the developers in the #phergie channel discussed the issue at one point before and may have found the cause, but a patch was never implemented to solve it.

See http://svn2.assembla.com/svn/phergie/trunk/Phergie/Plugin/Tld.php for the 1.x version of this plugin. The Url plugin should be modified to use this once it's ported.

See http://svn2.assembla.com/svn/phergie/trunk/Phergie/Plugin/UrbanDictionary.php for the 1.x version of this plugin. It should be modified to use the new Command plugin.

Currently, there are none. The 1.x branch never had them and I'd like to change that. It may require a few changes to the core components, such as allowing the stream resource in Phergie_Driver_Streams to be injected. Focus on the core first, then plugins. PHPUnit must be used for these. Ideally, it should be easy to include (or not) code coverage results in its final output assuming Xdebug is enabled.

The number of coding standard violations is growing and needs to be addressed. Below is a current log obtained by running phpcs --standard=PEAR . from the Phergie directory where phpcs can be obtained by installing PHP_CodeSniffer (http://pear.php.net/package/PHP_CodeSniffer).

FILE: /home/matt/Documents/projects/phergie/Phergie/Driver/Abstract.php
--------------------------------------------------------------------------------
FOUND 2 ERROR(S) AND 0 WARNING(S) AFFECTING 2 LINE(S)
--------------------------------------------------------------------------------
 264 | ERROR | The comments for parameters $nick (1) and $version (2) do not
     |       | align
 286 | ERROR | The comments for parameters $nick (1) and $finger (2) do not
     |       | align
--------------------------------------------------------------------------------


FILE: /home/matt/Documents/projects/phergie/Phergie/Event/Request.php
--------------------------------------------------------------------------------
FOUND 1 ERROR(S) AND 0 WARNING(S) AFFECTING 1 LINE(S)
--------------------------------------------------------------------------------
 30 | ERROR | @link tag comment indented incorrectly. Expected 5 spaces but
    |       | found 6.
--------------------------------------------------------------------------------


FILE: /home/matt/Documents/projects/phergie/Phergie/Event/Response.php
--------------------------------------------------------------------------------
FOUND 1 ERROR(S) AND 0 WARNING(S) AFFECTING 1 LINE(S)
--------------------------------------------------------------------------------
 31 | ERROR | @link tag comment indented incorrectly. Expected 5 spaces but
    |       | found 6.
--------------------------------------------------------------------------------


FILE: /home/matt/Documents/projects/phergie/Phergie/Hostmask.php
--------------------------------------------------------------------------------
FOUND 1 ERROR(S) AND 0 WARNING(S) AFFECTING 1 LINE(S)
--------------------------------------------------------------------------------
 201 | ERROR | The comments for parameters $pattern (1) and $hostmask (2) do
     |       | not align
--------------------------------------------------------------------------------


FILE: /home/matt/Documents/projects/phergie/Phergie/Plugin/Acl.php
--------------------------------------------------------------------------------
FOUND 1 ERROR(S) AND 0 WARNING(S) AFFECTING 1 LINE(S)
--------------------------------------------------------------------------------
 42 | ERROR | Closing parenthesis of a multi-line IF statement must be on a new
    |       | line
--------------------------------------------------------------------------------


FILE: /home/matt/Documents/projects/phergie/Phergie/Plugin/Google.php
--------------------------------------------------------------------------------
FOUND 0 ERROR(S) AND 3 WARNING(S) AFFECTING 3 LINE(S)
--------------------------------------------------------------------------------
 141 | WARNING | Line exceeds 85 characters; contains 94 characters
 222 | WARNING | Line exceeds 85 characters; contains 88 characters
 286 | WARNING | Line exceeds 85 characters; contains 110 characters
--------------------------------------------------------------------------------


FILE: /home/matt/Documents/projects/phergie/Phergie/Plugin/Http.php
--------------------------------------------------------------------------------
FOUND 24 ERROR(S) AND 1 WARNING(S) AFFECTING 16 LINE(S)
--------------------------------------------------------------------------------
  59 | WARNING | Line exceeds 85 characters; contains 86 characters
  79 | ERROR   | The variable names for parameters $type (1) and $callback (2)
     |         | do not align
  79 | ERROR   | The comments for parameters $type (1) and $callback (2) do not
     |         | align
 123 | ERROR   | The variable names for parameters $errno (1) and $errstr (2)
     |         | do not align
 123 | ERROR   | The comments for parameters $errno (1) and $errstr (2) do not
     |         | align
 124 | ERROR   | The comments for parameters $errstr (2) and $errfile (3) do
     |         | not align
 146 | ERROR   | The variable names for parameters $url (1) and $context (2) do
     |         | not align
 146 | ERROR   | The comments for parameters $url (1) and $context (2) do not
     |         | align
 197 | ERROR   | The variable names for parameters $url (1) and $query (2) do
     |         | not align
 197 | ERROR   | The comments for parameters $url (1) and $query (2) do not
     |         | align
 199 | ERROR   | The comments for parameters $query (2) and $context (3) do not
     |         | align
 205 | ERROR   | Multi-line function declaration not indented correctly;
     |         | expected 4 spaces but found 8
 205 | ERROR   | There must be a single space between the closing parenthesis
     |         | and the opening brace of a multi-line function declaration;
     |         | found newline
 220 | ERROR   | The variable names for parameters $url (1) and $query (2) do
     |         | not align
 220 | ERROR   | The comments for parameters $url (1) and $query (2) do not
     |         | align
 222 | ERROR   | The comments for parameters $query (2) and $context (3) do not
     |         | align
 228 | ERROR   | Multi-line function declaration not indented correctly;
     |         | expected 4 spaces but found 8
 228 | ERROR   | There must be a single space between the closing parenthesis
     |         | and the opening brace of a multi-line function declaration;
     |         | found newline
 243 | ERROR   | The variable names for parameters $url (1) and $query (2) do
     |         | not align
 243 | ERROR   | The comments for parameters $url (1) and $query (2) do not
     |         | align
 245 | ERROR   | The comments for parameters $query (2) and $post (3) do not
     |         | align
 248 | ERROR   | The comments for parameters $post (3) and $context (4) do not
     |         | align
 254 | ERROR   | Multi-line function declaration not indented correctly;
     |         | expected 4 spaces but found 8
 254 | ERROR   | There must be a single space between the closing parenthesis
     |         | and the opening brace of a multi-line function declaration;
     |         | found newline
 264 | ERROR   | Closing parenthesis of a multi-line IF statement must be on a
     |         | new line
--------------------------------------------------------------------------------


FILE: /home/matt/Documents/projects/phergie/Phergie/Plugin/Http/Response.php
--------------------------------------------------------------------------------
FOUND 5 ERROR(S) AND 0 WARNING(S) AFFECTING 5 LINE(S)
--------------------------------------------------------------------------------
 100 | ERROR | Line indented incorrectly; expected 8 spaces, found 12
 101 | ERROR | Line indented incorrectly; expected 8 spaces, found 12
 102 | ERROR | Line indented incorrectly; expected 8 spaces, found 12
 104 | ERROR | Line indented incorrectly; expected 8 spaces, found 12
 135 | ERROR | Doc comment for var $body does not match actual variable name
     |       | $content at position 1
--------------------------------------------------------------------------------


FILE: /home/matt/Documents/projects/phergie/Phergie/Plugin/Invisible.php
--------------------------------------------------------------------------------
FOUND 1 ERROR(S) AND 0 WARNING(S) AFFECTING 1 LINE(S)
--------------------------------------------------------------------------------
 30 | ERROR | @link tag comment indented incorrectly. Expected 5 spaces but
    |       | found 6.
--------------------------------------------------------------------------------


FILE: /home/matt/Documents/projects/phergie/Phergie/Plugin/Php/Source.php
--------------------------------------------------------------------------------
FOUND 11 ERROR(S) AND 1 WARNING(S) AFFECTING 2 LINE(S)
--------------------------------------------------------------------------------
 4 | WARNING | PHP version not specified
 4 | ERROR   | Missing @category tag in file comment
 4 | ERROR   | Missing @package tag in file comment
 4 | ERROR   | Missing @author tag in file comment
 4 | ERROR   | Missing @license tag in file comment
 4 | ERROR   | Missing @link tag in file comment
 4 | ERROR   | Missing @category tag in class comment
 4 | ERROR   | Missing @package tag in class comment
 4 | ERROR   | Missing @author tag in class comment
 4 | ERROR   | Missing @license tag in class comment
 4 | ERROR   | Missing @link tag in class comment
 7 | ERROR   | Missing function doc comment
--------------------------------------------------------------------------------


FILE: ...ome/matt/Documents/projects/phergie/Phergie/Plugin/Php/Source/Local.php
--------------------------------------------------------------------------------
FOUND 15 ERROR(S) AND 9 WARNING(S) AFFECTING 23 LINE(S)
--------------------------------------------------------------------------------
  51 | WARNING | Line exceeds 85 characters; contains 90 characters
  65 | ERROR   | Line indented incorrectly; expected at least 12 spaces, found
     |         | 8
  66 | ERROR   | Line indented incorrectly; expected at least 12 spaces, found
     |         | 8
  67 | ERROR   | Closing brace must be on a line by itself
  73 | ERROR   | Last parameter comment requires a blank newline after it
  85 | WARNING | Line exceeds 85 characters; contains 118 characters
  89 | ERROR   | Expected "if (...) {\n"; found "if(...) {\n"
 107 | ERROR   | Last parameter comment requires a blank newline after it
 114 | WARNING | Line exceeds 85 characters; contains 106 characters
 117 | ERROR   | Expected "if (...) {\n"; found "if(...) {\n"
 118 | WARNING | Line exceeds 85 characters; contains 104 characters
 119 | WARNING | Line exceeds 85 characters; contains 98 characters
 123 | ERROR   | Expected "if (...) {\n"; found "if(...) {\n"
 127 | WARNING | Line exceeds 85 characters; contains 89 characters
 129 | ERROR   | Expected "if (...) {\n"; found "if(...) {\n"
 137 | ERROR   | Expected "foreach (...) {\n"; found "foreach(...) {\n"
 158 | ERROR   | Expected "} else {\n"; found "}\n                // ... it's
     |         | the last part of the complete function description\n          
     |         |      else {\n"
 161 | ERROR   | Expected "if (...) {\n"; found "if(...) {\n"
 161 | WARNING | Line exceeds 85 characters; contains 128 characters
 172 | ERROR   | Expected "if (...) {\n"; found "if(...) {\n"
 177 | WARNING | Line exceeds 85 characters; contains 129 characters
 181 | ERROR   | Expected "foreach (...) {\n"; found "foreach(...) {\n"
 184 | ERROR   | Expected "if (...) {\n"; found "if(...) {\n"
 189 | WARNING | Line exceeds 85 characters; contains 89 characters
--------------------------------------------------------------------------------


FILE: /home/matt/Documents/projects/phergie/Phergie/Plugin/Ping.php
--------------------------------------------------------------------------------
FOUND 5 ERROR(S) AND 0 WARNING(S) AFFECTING 3 LINE(S)
--------------------------------------------------------------------------------
  90 | ERROR | Expected "if (...) {\n"; found "if(...) {\n"
 116 | ERROR | Last parameter comment requires a blank newline after it
 116 | ERROR | Missing comment for param "$ping" at position 1
 146 | ERROR | Last parameter comment requires a blank newline after it
 146 | ERROR | Missing comment for param "$event" at position 1
--------------------------------------------------------------------------------


FILE: ...me/matt/Documents/projects/phergie/Phergie/Plugin/TheFuckingWeather.php
--------------------------------------------------------------------------------
FOUND 15 ERROR(S) AND 1 WARNING(S) AFFECTING 11 LINE(S)
--------------------------------------------------------------------------------
  89 | ERROR   | Doc comment for "$location" missing
 108 | ERROR   | Opening brace should be on a new line
 113 | ERROR   | Opening parenthesis of a multi-line function call must be the
     |         | last content on the line
 114 | ERROR   | Multi-line function call not indented correctly; expected 8
     |         | spaces but found 24
 114 | ERROR   | Closing parenthesis of a multi-line function call must be on a
     |         | line by itself
 118 | ERROR   | Opening parenthesis of a multi-line function call must be the
     |         | last content on the line
 119 | ERROR   | Multi-line function call not indented correctly; expected 12
     |         | spaces but found 28
 119 | ERROR   | Closing parenthesis of a multi-line function call must be on a
     |         | line by itself
 123 | ERROR   | Opening parenthesis of a multi-line function call must be the
     |         | last content on the line
 124 | ERROR   | Multi-line function call not indented correctly; expected 12
     |         | spaces but found 28
 124 | ERROR   | Closing parenthesis of a multi-line function call must be on a
     |         | line by itself
 127 | ERROR   | Opening parenthesis of a multi-line function call must be the
     |         | last content on the line
 127 | WARNING | Line exceeds 85 characters; contains 87 characters
 128 | ERROR   | Multi-line function call not indented correctly; expected 12
     |         | spaces but found 28
 128 | ERROR   | Closing parenthesis of a multi-line function call must be on a
     |         | line by itself
 132 | ERROR   | No space found after comma in function call
--------------------------------------------------------------------------------


FILE: ...tt/Documents/projects/phergie/Phergie/Plugin/Twitter/laconica.class.php
--------------------------------------------------------------------------------
FOUND 14 ERROR(S) AND 1 WARNING(S) AFFECTING 10 LINE(S)
--------------------------------------------------------------------------------
  2 | ERROR   | Missing file doc comment
 13 | ERROR   | Missing @category tag in class comment
 13 | ERROR   | Missing @package tag in class comment
 13 | ERROR   | Missing @author tag in class comment
 13 | ERROR   | Missing @license tag in class comment
 13 | ERROR   | Missing @link tag in class comment
 14 | ERROR   | Opening brace of a class must be on the line after the
    |         | definition
 21 | ERROR   | The comments for parameters $pass (2) and $baseUrl (3) do not
    |         | align
 23 | ERROR   | Opening brace should be on a new line
 23 | WARNING | Line exceeds 85 characters; contains 89 characters
 30 | ERROR   | Missing @return tag in function comment
 31 | ERROR   | Opening brace should be on a new line
 35 | ERROR   | Doc comment for "$tweet" missing
 37 | ERROR   | Missing @return tag in function comment
 38 | ERROR   | Opening brace should be on a new line
--------------------------------------------------------------------------------


FILE: ...att/Documents/projects/phergie/Phergie/Plugin/Twitter/twitter.class.php
--------------------------------------------------------------------------------
FOUND 64 ERROR(S) AND 8 WARNING(S) AFFECTING 58 LINE(S)
--------------------------------------------------------------------------------
   2 | ERROR   | Missing file doc comment
  16 | ERROR   | Missing @category tag in class comment
  16 | ERROR   | Missing @package tag in class comment
  16 | ERROR   | Missing @author tag in class comment
  16 | ERROR   | Missing @license tag in class comment
  16 | ERROR   | Missing @link tag in class comment
  17 | ERROR   | Opening brace of a class must be on the line after the
     |         | definition
  49 | ERROR   | Opening brace should be on a new line
  74 | ERROR   | Last parameter comment requires a blank newline after it
  77 | ERROR   | Opening brace should be on a new line
  89 | ERROR   | Last parameter comment requires a blank newline after it
  89 | ERROR   | The variable names for parameters $tweeter (1) and $num (2) do
     |         | not align
  89 | ERROR   | The comments for parameters $tweeter (1) and $num (2) do not
     |         | align
  94 | WARNING | Line exceeds 85 characters; contains 86 characters
 105 | ERROR   | Doc comment for "$sinceId" missing
 105 | ERROR   | Doc comment for "$count" missing
 107 | ERROR   | Missing @return tag in function comment
 108 | ERROR   | Opening brace should be on a new line
 109 | WARNING | Line exceeds 85 characters; contains 87 characters
 112 | ERROR   | Doc comment for "$cursor" missing
 114 | ERROR   | Missing @return tag in function comment
 115 | ERROR   | Opening brace should be on a new line
 119 | ERROR   | Doc comment for "$userId" missing
 121 | ERROR   | Missing @return tag in function comment
 122 | ERROR   | Opening brace should be on a new line
 143 | ERROR   | Doc comment for "$sinceId" missing
 143 | ERROR   | Doc comment for "$count" missing
 143 | ERROR   | Doc comment for "$page" missing
 145 | ERROR   | Missing @return tag in function comment
 146 | ERROR   | Opening brace should be on a new line
 147 | WARNING | Line exceeds 85 characters; contains 89 characters
 150 | ERROR   | Doc comment for "$screenName" missing
 150 | ERROR   | Doc comment for "$text" missing
 152 | ERROR   | Missing @return tag in function comment
 153 | ERROR   | Opening brace should be on a new line
 178 | ERROR   | Last parameter comment requires a blank newline after it
 178 | ERROR   | Doc comment for "$limit" missing
 181 | ERROR   | Opening brace should be on a new line
 208 | ERROR   | Missing @return tag in function comment
 209 | ERROR   | Opening brace should be on a new line
 217 | ERROR   | Missing @return tag in function comment
 218 | ERROR   | Opening brace should be on a new line
 222 | ERROR   | Doc comment for "$user" missing
 224 | ERROR   | Missing @return tag in function comment
 225 | ERROR   | Opening brace should be on a new line
 226 | WARNING | Line exceeds 85 characters; contains 89 characters
 231 | ERROR   | Missing @return tag in function comment
 232 | ERROR   | Opening brace should be on a new line
 236 | ERROR   | Doc comment for "$tweet" missing
 238 | ERROR   | Missing @return tag in function comment
 239 | ERROR   | Opening brace should be on a new line
 240 | WARNING | Line exceeds 85 characters; contains 108 characters
 243 | ERROR   | Doc comment for "$sinceId" missing
 243 | ERROR   | Doc comment for "$count" missing
 245 | ERROR   | Missing @return tag in function comment
 246 | ERROR   | Opening brace should be on a new line
 247 | WARNING | Line exceeds 85 characters; contains 88 characters
 254 | ERROR   | Doc comment for "$cursor" missing
 256 | ERROR   | Missing @return tag in function comment
 257 | ERROR   | Opening brace should be on a new line
 258 | WARNING | Line exceeds 85 characters; contains 87 characters
 261 | ERROR   | Doc comment for "$userid" missing
 263 | ERROR   | Missing @return tag in function comment
 264 | ERROR   | Opening brace should be on a new line
 265 | WARNING | Line exceeds 85 characters; contains 86 characters
 268 | ERROR   | Doc comment for "$sinceId" missing
 268 | ERROR   | Doc comment for "$count" missing
 268 | ERROR   | Doc comment for "$page" missing
 270 | ERROR   | Missing @return tag in function comment
 271 | ERROR   | Opening brace should be on a new line
 283 | ERROR   | Missing @return tag in function comment
 284 | ERROR   | Opening brace should be on a new line
--------------------------------------------------------------------------------


FILE: /home/matt/Documents/projects/phergie/Phergie/Plugin/Url.php
--------------------------------------------------------------------------------
FOUND 206 ERROR(S) AND 41 WARNING(S) AFFECTING 144 LINE(S)
--------------------------------------------------------------------------------
 190 | WARNING | Line exceeds 85 characters; contains 91 characters
 204 | WARNING | Line exceeds 85 characters; contains 1712 characters
 240 | ERROR   | Opening parenthesis of a multi-line function call must be the
     |         | last content on the line
 240 | WARNING | Line exceeds 85 characters; contains 121 characters
 241 | ERROR   | Multi-line IF statement not indented correctly; expected 12
     |         | spaces but found 0
 241 | ERROR   | Each line in a multi-line IF statement must begin with a
     |         | boolean operator
 241 | WARNING | Line exceeds 85 characters; contains 147 characters
 242 | ERROR   | Closing parenthesis of a multi-line IF statement must be on a
     |         | new line
 242 | ERROR   | Multi-line IF statement not indented correctly; expected 12
     |         | spaces but found 28
 242 | ERROR   | Multi-line function call not indented correctly; expected 8
     |         | spaces but found 28
 242 | ERROR   | Closing parenthesis of a multi-line function call must be on a
     |         | line by itself
 242 | WARNING | Line exceeds 85 characters; contains 91 characters
 244 | ERROR   | Expected "foreach (...) {\n"; found "foreach(...) {\n"
 247 | WARNING | Line exceeds 85 characters; contains 89 characters
 249 | WARNING | Line exceeds 85 characters; contains 108 characters
 255 | WARNING | Line exceeds 85 characters; contains 89 characters
 272 | WARNING | Line exceeds 85 characters; contains 105 characters
 278 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 278 | ERROR   | Line indented incorrectly; expected at least 20 spaces, found
     |         | 17
 279 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 279 | ERROR   | Line indented incorrectly; expected at least 20 spaces, found
     |         | 17
 280 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 280 | ERROR   | Line indented incorrectly; expected at least 20 spaces, found
     |         | 17
 280 | WARNING | Line exceeds 85 characters; contains 92 characters
 282 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 282 | ERROR   | Line indented incorrectly; expected at least 20 spaces, found
     |         | 17
 283 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 283 | ERROR   | Line indented incorrectly; expected 20 spaces, found 17
 283 | WARNING | Line exceeds 85 characters; contains 105 characters
 284 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 284 | ERROR   | Line indented incorrectly; expected at least 24 spaces, found
     |         | 21
 284 | WARNING | Line exceeds 85 characters; contains 112 characters
 285 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 285 | ERROR   | Line indented incorrectly; expected at least 24 spaces, found
     |         | 21
 286 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 289 | WARNING | Line exceeds 85 characters; contains 96 characters
 292 | WARNING | Line exceeds 85 characters; contains 122 characters
 300 | WARNING | Line exceeds 85 characters; contains 117 characters
 317 | ERROR   | Opening parenthesis of a multi-line function call must be the
     |         | last content on the line
 325 | ERROR   | Closing parenthesis of a multi-line function call must be on a
     |         | line by itself
 333 | WARNING | Line exceeds 85 characters; contains 91 characters
 338 | ERROR   | Opening parenthesis of a multi-line function call must be the
     |         | last content on the line
 338 | ERROR   | Opening parenthesis of a multi-line function call must be the
     |         | last content on the line
 344 | ERROR   | Closing parenthesis of a multi-line function call must be on a
     |         | line by itself
 344 | ERROR   | Closing parenthesis of a multi-line function call must be on a
     |         | line by itself
 346 | ERROR   | Expected "foreach (...) {\n"; found "foreach(...) {\n"
 347 | ERROR   | Opening parenthesis of a multi-line function call must be the
     |         | last content on the line
 347 | ERROR   | Opening parenthesis of a multi-line function call must be the
     |         | last content on the line
 353 | ERROR   | Closing parenthesis of a multi-line function call must be on a
     |         | line by itself
 353 | ERROR   | Closing parenthesis of a multi-line function call must be on a
     |         | line by itself
 365 | ERROR   | Last parameter comment requires a blank newline after it
 365 | ERROR   | The comments for parameters $url (1) and $shortenedUrl (2) do
     |         | not align
 373 | WARNING | Line exceeds 85 characters; contains 97 characters
 380 | WARNING | Line exceeds 85 characters; contains 99 characters
 381 | WARNING | Line exceeds 85 characters; contains 126 characters
 387 | WARNING | Line exceeds 85 characters; contains 97 characters
 390 | WARNING | Line exceeds 85 characters; contains 112 characters
 391 | ERROR   | Closing parenthesis of a multi-line IF statement must be on a
     |         | new line
 404 | ERROR   | Last parameter comment requires a blank newline after it
 404 | ERROR   | The comments for parameters $url (1) and $shortenedUrl (2) do
     |         | not align
 412 | WARNING | Line exceeds 85 characters; contains 97 characters
 419 | WARNING | Line exceeds 85 characters; contains 88 characters
 426 | WARNING | Line exceeds 85 characters; contains 94 characters
 441 | ERROR   | Last parameter comment requires a blank newline after it
 441 | ERROR   | The variable names for parameters $str (1) and $trim (2) do
     |         | not align
 441 | ERROR   | The comments for parameters $str (1) and $trim (2) do not
     |         | align
 453 | ERROR   | Doc comment for "$errno" missing
 453 | ERROR   | Doc comment for "$errstr" missing
 453 | ERROR   | Doc comment for "$errfile" missing
 453 | ERROR   | Doc comment for "$errline" missing
 455 | ERROR   | Missing @return tag in function comment
 462 | WARNING | Line exceeds 85 characters; contains 107 characters
 463 | WARNING | Line exceeds 85 characters; contains 101 characters
 465 | ERROR   | Line indented incorrectly; expected at least 16 spaces, found
     |         | 12
 467 | ERROR   | Multi-line IF statement not indented correctly; expected 16
     |         | spaces but found 23
 467 | ERROR   | Each line in a multi-line IF statement must begin with a
     |         | boolean operator
 468 | ERROR   | Multi-line IF statement not indented correctly; expected 16
     |         | spaces but found 23
 468 | ERROR   | Each line in a multi-line IF statement must begin with a
     |         | boolean operator
 469 | ERROR   | Multi-line IF statement not indented correctly; expected 16
     |         | spaces but found 23
 469 | ERROR   | Each line in a multi-line IF statement must begin with a
     |         | boolean operator
 469 | WARNING | Line exceeds 85 characters; contains 122 characters
 470 | ERROR   | Multi-line IF statement not indented correctly; expected 16
     |         | spaces but found 23
 470 | ERROR   | Each line in a multi-line IF statement must begin with a
     |         | boolean operator
 471 | ERROR   | Multi-line IF statement not indented correctly; expected 16
     |         | spaces but found 23
 471 | ERROR   | Each line in a multi-line IF statement must begin with a
     |         | boolean operator
 471 | WARNING | Line exceeds 85 characters; contains 135 characters
 472 | ERROR   | Multi-line IF statement not indented correctly; expected 16
     |         | spaces but found 23
 472 | ERROR   | Each line in a multi-line IF statement must begin with a
     |         | boolean operator
 472 | WARNING | Line exceeds 85 characters; contains 86 characters
 473 | ERROR   | Closing parenthesis of a multi-line IF statement must be on a
     |         | new line
 473 | ERROR   | Multi-line IF statement not indented correctly; expected 16
     |         | spaces but found 23
 475 | WARNING | Line exceeds 85 characters; contains 101 characters
 482 | ERROR   | Doc comment for "$url" missing
 485 | ERROR   | Missing @return tag in function comment
 497 | ERROR   | You must use "/**" style comments for a function comment
 499 | WARNING | Inline control structures are discouraged
 516 | WARNING | Line exceeds 85 characters; contains 90 characters
 530 | ERROR   | You must use "/**" style comments for a function comment
 541 | WARNING | Line exceeds 85 characters; contains 88 characters
 543 | WARNING | Line exceeds 85 characters; contains 91 characters
 553 | ERROR   | Each line in a multi-line IF statement must begin with a
     |         | boolean operator
 554 | ERROR   | Closing parenthesis of a multi-line IF statement must be on a
     |         | new line
 558 | ERROR   | Expected "if (...) {\n"; found "if(...)\n            {\n"
 558 | WARNING | Line exceeds 85 characters; contains 86 characters
 560 | WARNING | Line exceeds 85 characters; contains 107 characters
 564 | WARNING | Line exceeds 85 characters; contains 87 characters
 573 | ERROR   | You must use "/**" style comments for a function comment
 573 | ERROR   | Opening brace should be on a new line
 578 | ERROR   | Line indented incorrectly; expected at least 4 spaces, found 3
 579 | ERROR   | Line indented incorrectly; expected at least 4 spaces, found 3
 580 | ERROR   | Line indented incorrectly; expected at least 4 spaces, found 3
 580 | ERROR   | Last parameter comment requires a blank newline after it
 581 | ERROR   | Line indented incorrectly; expected at least 4 spaces, found 3
 582 | ERROR   | Line indented incorrectly; expected at least 4 spaces, found 3
 585 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 585 | ERROR   | Line indented incorrectly; expected at least 8 spaces, found 2
 586 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 586 | ERROR   | Line indented incorrectly; expected at least 8 spaces, found 3
 587 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 587 | ERROR   | Line indented incorrectly; expected at least 8 spaces, found 4
 588 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 588 | ERROR   | Line indented incorrectly; expected at least 8 spaces, found 4
 589 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 589 | ERROR   | Line indented incorrectly; expected at least 8 spaces, found 4
 589 | WARNING | Line exceeds 85 characters; contains 114 characters
 590 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 590 | ERROR   | Line indented incorrectly; expected at least 8 spaces, found 3
 591 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 591 | ERROR   | Line indented incorrectly; expected at least 8 spaces, found 2
 592 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 592 | ERROR   | Line indented incorrectly; expected at least 8 spaces, found 2
 594 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 594 | ERROR   | Line indented incorrectly; expected 8 spaces, found 2
 595 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 595 | ERROR   | Line indented incorrectly; expected at least 12 spaces, found
     |         | 3
 596 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 596 | ERROR   | Line indented incorrectly; expected at least 12 spaces, found
     |         | 3
 597 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 597 | ERROR   | Expected "foreach (...) {\n"; found "foreach(...) {\n"
 597 | ERROR   | Line indented incorrectly; expected 12 spaces, found 3
 598 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 598 | ERROR   | Line indented incorrectly; expected 16 spaces, found 4
 599 | ERROR   | Closing parenthesis of a multi-line IF statement must be on a
     |         | new line
 599 | ERROR   | Multi-line IF statement not indented correctly; expected 8
     |         | spaces but found 5
 599 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 600 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 600 | ERROR   | Line indented incorrectly; expected at least 20 spaces, found
     |         | 5
 601 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 602 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 603 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 603 | ERROR   | Line indented incorrectly; expected 12 spaces, found 3
 604 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 604 | ERROR   | Line indented incorrectly; expected at least 16 spaces, found
     |         | 4
 605 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 605 | ERROR   | Line indented incorrectly; expected at least 16 spaces, found
     |         | 4
 607 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 607 | ERROR   | Line indented incorrectly; expected 16 spaces, found 4
 608 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 608 | ERROR   | Line indented incorrectly; expected at least 20 spaces, found
     |         | 5
 609 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 609 | ERROR   | Line indented incorrectly; expected 20 spaces, found 5
 610 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 610 | ERROR   | Line indented incorrectly; expected at least 24 spaces, found
     |         | 6
 611 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 611 | ERROR   | Line indented incorrectly; expected at least 24 spaces, found
     |         | 6
 612 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 612 | ERROR   | Line indented incorrectly; expected at least 24 spaces, found
     |         | 6
 613 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 614 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 614 | ERROR   | Line indented incorrectly; expected at least 20 spaces, found
     |         | 5
 615 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 615 | ERROR   | Line indented incorrectly; expected at least 20 spaces, found
     |         | 5
 616 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 616 | WARNING | Inline control structures are discouraged
 617 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 617 | ERROR   | Line indented incorrectly; expected at least 20 spaces, found
     |         | 5
 618 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 618 | ERROR   | Line indented incorrectly; expected 20 spaces, found 5
 619 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 619 | WARNING | Line exceeds 85 characters; contains 97 characters
 619 | ERROR   | Line indented incorrectly; expected at least 24 spaces, found
     |         | 6
 620 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 620 | ERROR   | Line indented incorrectly; expected at least 24 spaces, found
     |         | 6
 621 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 621 | ERROR   | Line indented incorrectly; expected at least 24 spaces, found
     |         | 6
 622 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 622 | ERROR   | Line indented incorrectly; expected 24 spaces, found 6
 623 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 623 | ERROR   | Line indented incorrectly; expected at least 28 spaces, found
     |         | 7
 624 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 624 | ERROR   | Line indented incorrectly; expected at least 28 spaces, found
     |         | 7
 625 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 625 | WARNING | Inline control structures are discouraged
 626 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 627 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 627 | ERROR   | Line indented incorrectly; expected at least 24 spaces, found
     |         | 6
 628 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 628 | ERROR   | Line indented incorrectly; expected at least 24 spaces, found
     |         | 6
 629 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 629 | ERROR   | Line indented incorrectly; expected at least 24 spaces, found
     |         | 6
 630 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 630 | ERROR   | Line indented incorrectly; expected at least 24 spaces, found
     |         | 6
 631 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 632 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 632 | ERROR   | Line indented incorrectly; expected at least 20 spaces, found
     |         | 5
 633 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 633 | ERROR   | Line indented incorrectly; expected 20 spaces, found 5
 634 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 634 | ERROR   | Line indented incorrectly; expected at least 24 spaces, found
     |         | 6
 635 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 636 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 637 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 638 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 638 | ERROR   | Line indented incorrectly; expected at least 12 spaces, found
     |         | 3
 639 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 639 | ERROR   | Line indented incorrectly; expected 8 spaces, found 2
 640 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 640 | ERROR   | Line indented incorrectly; expected at least 12 spaces, found
     |         | 3
 641 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 641 | ERROR   | Line indented incorrectly; expected at least 8 spaces, found 2
 643 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 643 | ERROR   | Line indented incorrectly; expected 8 spaces, found 2
 644 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 644 | ERROR   | Line indented incorrectly; expected 12 spaces, found 3
 645 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 645 | ERROR   | Line indented incorrectly; expected 16 spaces, found 4
 646 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 646 | ERROR   | Line indented incorrectly; expected at least 20 spaces, found
     |         | 5
 647 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 648 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 648 | ERROR   | Line indented incorrectly; expected at least 16 spaces, found
     |         | 4
 649 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 649 | ERROR   | Line indented incorrectly; expected at least 16 spaces, found
     |         | 4
 650 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 650 | ERROR   | Line indented incorrectly; expected at least 16 spaces, found
     |         | 4
 651 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 651 | ERROR   | Line indented incorrectly; expected 12 spaces, found 3
 652 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 652 | ERROR   | Line indented incorrectly; expected at least 16 spaces, found
     |         | 4
 653 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 654 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 656 | ERROR   | Spaces must be used to indent lines; tabs are not allowed
 656 | ERROR   | Line indented incorrectly; expected at least 8 spaces, found 2
 659 | ERROR   | You must use "/**" style comments for a function comment
 664 | ERROR   | Missing function doc comment
 670 | ERROR   | You must use "/**" style comments for a function comment
 670 | ERROR   | Opening brace should be on a new line
--------------------------------------------------------------------------------


FILE: ...matt/Documents/projects/phergie/Phergie/Plugin/Url/Shorten/Abstract.php
--------------------------------------------------------------------------------
FOUND 7 ERROR(S) AND 0 WARNING(S) AFFECTING 3 LINE(S)
--------------------------------------------------------------------------------
 2 | ERROR | Missing file doc comment
 5 | ERROR | Missing @category tag in class comment
 5 | ERROR | Missing @package tag in class comment
 5 | ERROR | Missing @author tag in class comment
 5 | ERROR | Missing @license tag in class comment
 5 | ERROR | Missing @link tag in class comment
 8 | ERROR | Missing function doc comment
--------------------------------------------------------------------------------


FILE: ...ome/matt/Documents/projects/phergie/Phergie/Plugin/Url/Shorten/Trim.php
--------------------------------------------------------------------------------
FOUND 7 ERROR(S) AND 1 WARNING(S) AFFECTING 4 LINE(S)
--------------------------------------------------------------------------------
  2 | ERROR   | Missing file doc comment
  5 | ERROR   | Missing @category tag in class comment
  5 | ERROR   | Missing @package tag in class comment
  5 | ERROR   | Missing @author tag in class comment
  5 | ERROR   | Missing @license tag in class comment
  5 | ERROR   | Missing @link tag in class comment
  8 | ERROR   | Missing function doc comment
 10 | WARNING | Line exceeds 85 characters; contains 94 characters
--------------------------------------------------------------------------------


FILE: /home/matt/Documents/projects/phergie/Phergie/Process/Async.php
--------------------------------------------------------------------------------
FOUND 8 ERROR(S) AND 0 WARNING(S) AFFECTING 5 LINE(S)
--------------------------------------------------------------------------------
 80 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 80 | ERROR | Line indented incorrectly; expected 12 spaces, found 5
 81 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 81 | ERROR | Expected "if (...) {\n"; found "if(...) {\n"
 81 | ERROR | Line indented incorrectly; expected 16 spaces, found 2
 83 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 85 | ERROR | Closing brace indented incorrectly; expected 2 spaces, found 16
 87 | ERROR | Closing brace indented incorrectly; expected 5 spaces, found 12
--------------------------------------------------------------------------------


FILE: /home/matt/Documents/projects/phergie/Phergie/Tools/LogViewer/index.php
--------------------------------------------------------------------------------
FOUND 50 ERROR(S) AND 12 WARNING(S) AFFECTING 48 LINE(S)
--------------------------------------------------------------------------------
   2 | WARNING | Line exceeds 85 characters; contains 93 characters
   2 | ERROR   | You must use "/**" style comments for a file comment
   3 | WARNING | Line exceeds 85 characters; contains 97 characters
   4 | WARNING | Line exceeds 85 characters; contains 99 characters
   5 | WARNING | Line exceeds 85 characters; contains 100 characters
  21 | WARNING | Line exceeds 85 characters; contains 93 characters
  22 | WARNING | Line exceeds 85 characters; contains 101 characters
  32 | ERROR   | Line indented incorrectly; expected 0 spaces, found 4
  35 | ERROR   | Line indented incorrectly; expected 0 spaces, found 4
  38 | ERROR   | Line indented incorrectly; expected 0 spaces, found 4
  42 | WARNING | Line exceeds 85 characters; contains 90 characters
  53 | ERROR   | Last parameter comment requires a blank newline after it
  53 | ERROR   | Doc comment for var A does not match actual variable name $db
     |         | at position 1
  57 | ERROR   | Function name "show_channels" is prefixed with a package name
     |         | but does not begin with a capital letter
  57 | ERROR   | Opening brace should be on a new line
  62 | WARNING | Line exceeds 85 characters; contains 88 characters
  65 | ERROR   | Opening parenthesis of a multi-line function call must be the
     |         | last content on the line
  69 | ERROR   | Closing parenthesis of a multi-line function call must be on a
     |         | line by itself
  90 | ERROR   | Last parameter comment requires a blank newline after it
  90 | ERROR   | Doc comment for var A does not match actual variable name $db
     |         | at position 1
  94 | ERROR   | Function name "show_days" is prefixed with a package name but
     |         | does not begin with a capital letter
  94 | ERROR   | Opening brace should be on a new line
 104 | ERROR   | Opening parenthesis of a multi-line function call must be the
     |         | last content on the line
 108 | ERROR   | Closing parenthesis of a multi-line function call must be on a
     |         | line by itself
 131 | WARNING | Line exceeds 85 characters; contains 97 characters
 148 | WARNING | Line exceeds 85 characters; contains 90 characters
 173 | ERROR   | Last parameter comment requires a blank newline after it
 173 | ERROR   | Doc comment for var A does not match actual variable name $db
     |         | at position 1
 177 | ERROR   | Function name "show_log" is prefixed with a package name but
     |         | does not begin with a capital letter
 177 | ERROR   | Opening brace should be on a new line
 184 | ERROR   | Opening parenthesis of a multi-line function call must be the
     |         | last content on the line
 185 | ERROR   | Multi-line function call not indented correctly; expected 4
     |         | spaces but found 8
 185 | ERROR   | Closing parenthesis of a multi-line function call must be on a
     |         | line by itself
 188 | ERROR   | Opening parenthesis of a multi-line function call must be the
     |         | last content on the line
 193 | ERROR   | Closing parenthesis of a multi-line function call must be on a
     |         | line by itself
 194 | ERROR   | Opening parenthesis of a multi-line function call must be the
     |         | last content on the line
 197 | ERROR   | Closing parenthesis of a multi-line function call must be on a
     |         | line by itself
 210 | ERROR   | Line indented incorrectly; expected 8 spaces, found 12
 211 | WARNING | Line exceeds 85 characters; contains 102 characters
 213 | ERROR   | Line indented incorrectly; expected 8 spaces, found 12
 214 | WARNING | Line exceeds 85 characters; contains 95 characters
 216 | ERROR   | Line indented incorrectly; expected 8 spaces, found 12
 219 | ERROR   | Line indented incorrectly; expected 8 spaces, found 12
 222 | ERROR   | Line indented incorrectly; expected 8 spaces, found 12
 225 | ERROR   | Line indented incorrectly; expected 8 spaces, found 12
 228 | ERROR   | Line indented incorrectly; expected 8 spaces, found 12
 231 | ERROR   | Line indented incorrectly; expected 8 spaces, found 12
 233 | ERROR   | Line indented incorrectly; expected 8 spaces, found 12
 243 | ERROR   | Doc comment for "$user" missing
 254 | ERROR   | Function name "nick_color" is prefixed with a package name but
     |         | does not begin with a capital letter
 254 | ERROR   | Opening brace should be on a new line
 269 | ERROR   | Last parameter comment requires a blank newline after it
 269 | ERROR   | Doc comment for var The does not match actual variable name
     |         | $string at position 1
 273 | ERROR   | Opening brace should be on a new line
 285 | ERROR   | Last parameter comment requires a blank newline after it
 285 | ERROR   | Doc comment for var string does not match actual variable name
     |         | $title at position 1
 289 | ERROR   | Function name "template_header" is prefixed with a package
     |         | name but does not begin with a capital letter
 289 | ERROR   | Opening brace should be on a new line
 313 | ERROR   | Function name "template_footer" is prefixed with a package
     |         | name but does not begin with a capital letter
 313 | ERROR   | Opening brace should be on a new line
 328 | ERROR   | Function name "template_css" is prefixed with a package name
     |         | but does not begin with a capital letter
 328 | ERROR   | Opening brace should be on a new line
--------------------------------------------------------------------------------

This command: convert 8500000 dollars in euro
yields this odd result: 8? 500? 000 U.S. dollars = 6.12480184 million Euros

Currently, the FeedTicker plugin contains logic for parsing items from a feed. This logic should be moved into a separate plugin and FeedTicker should be integrate with it. This would expose the functionality that FeedTicker already possesses to other plugins that don't necessarily need to fetch the feed synchronously.

Currently, the Command and Url plugins both internally establish data caches. A central caching plugin would be useful to remove duplication of logic in these areas. Its duties would be as follows:

1. Accept data to cache. Optionally, a lifetime for it could be specified where this plugin would automatically handle checking for expiration and indicating misses by returning null. Another handy option would be to support the specification of a callback to determine whether or not a specific cached datum should be expired.

2. Provide access to cached data.

Issue:
When the command.prefix option is set in Phergie's settings, she doesn't respect the prefix.

Problem:
This is due to how the Phergie_Plugin_Command class is instantiated. By itself, it doesn't do anything, so it's not explicitly required in the plugins section to autoload. However, other plugins can require it to function. When Command is instantiated via the plugins autoload, Phergie_Bot::loadPlugins() calls setConfig on the plugin when it's instantiated. However, if the plugin is requested by Phergie_Plugin_Handler::getPlugin('Command'), setEvent() is not called. When Phergie_Plugin_Command::onPrivMsg() is received, there is no configuration for it to look up the prefix and commands are therefore not processed correctly.

Proposed Solution:
Plugin loading should be consolidated so that no matter where a plugin is finally instantiated and put into the Handler, all of the correct objects should be passed in to the plugin and the onLoad() method should be called.

Temporary solution:
Putting 'Command' into the plugin autoload will allow Command to load and get the required Phergie_Config object passed in to it to allow it to work.

See http://svn2.assembla.com/svn/phergie/trunk/Phergie/Plugin/Daddy.php for the 1.x version of this plugin.

See http://svn2.assembla.com/svn/phergie/trunk/Phergie/Plugin/Convert.php for the 1.x version of this plugin. This should be modified to use the new transliteration and HTTP plugins once they're created, in the case of the former to correct a long-standing encoding issue.

This plugin should offer the following features:

1. Configuring options that facilitate either masking of censored words (i.e. using asterisks) or discarding of events that contain them entirely; this may or may not allow per-plugin configuration
2. Public methods to be used by other plugins to detect whether event data contains words that would be censored so they can adjust their behavior accordingly
3. An easily modifiable data source, such as a flat text file or SQLite database, that allows censored words to be tweaked, but contains a relatively comprehensive initial set of data if a web service is not used

Integration with this web service might work: http://wiki.cdyne.com/wiki/index.php?title=Profanity_Filter

Test the core and plugins for integration with PHAR so it can be easily distributed in 5.3 environments and add this as an option to build.xml. See http://phing.info/docs/guide/stable/chapters/appendixes/AppendixC-OptionalTasks.html#PharPackageTask for more details on how to do this.

Received:
irc.freenode.net <- :frigg!~frigg@freenode/utility-bot/frigg PRIVMSG botEvan :VERSION

causes:
PHP Fatal error: Undefined class constant 'VERSION' in /data/phergie/Phergie/Plugin/Ctcp.php on line 55

when i try to: Gmap greece(or whateva)
PHP Notice: Undefined property: Phergie_Plugin_Google::$_config in /home/dmtrs/phergie/Phergie/Plugin/Google.php on line 235
PHP Fatal error: Cannot use object of type stdClass as array in /home/dmtrs/phergie/Phergie/Plugin/Google.php on line 262

See decodeTranslit() in Phergie_Plugin_Abstract_Base from the old 1.x branch for inspiration on this:
http://svn2.assembla.com/svn/phergie/trunk/Phergie/Plugin/Abstract/Base.php

The responsibilities of this plugin would include handling any or all of encoding, decoding, and transliteration of string data. It could be designed in either or both of these manners:

1. A service plugin to be used by other plugins
2. A passive plugin that handles its duties across all outgoing messages using the preDispatch event hook

Phergie does not currently support following channel forwards and fails to join channels as a result. #atlphp -> #atlantaphp on Freenode is an example case of this. This should be addressed in a ChannelForward plugin that intercepts the server response indicating a channel forward and processes it accordingly. This relates to functionality provided by the AutoJoin and Join plugins.

installed Phergie_Plugin_Url 2.0.0 via pear, edited config file etc.
The girl's not shortening any urls (or reacting to a pasted url in any way)

13:22 < radagast> anyone making use of namespaces in PHP yet? http://www.phparch.com/2010/03/29/namespaces-in-php/
13:28 -!- barryo__ [[email protected]] has joined #phpug

1. Database dependencies need to be moved to Phergie_Plugin_Source_Php_Local.
2. Error and exception handling should be improved in Phergie_Plugin_Source_Php_Local (see @todo items in the source)
3. A configuration setting should be added and Phergie_Plugin_Php modified to allow a different source class to be used.
4. Support should be added for class methods to supplement the existing function lookup functionality.

http://github.com/elazar/phergie/blob/master/Phergie/Driver/Streams.php#L634

$finer should be $finger

If the socket connection is unexpectedly terminated, fwrite will fail, and there is no attempt to handle nor detect this currently in the streams driver.

http://php.net/manual/en/function.fwrite.php

Could just strict-check for false being returned from fwrite, and handle it as you please.

This plugin must use the Command plugin. It must provide a registry and a method by which to add command descriptions to that registry such that other plugins are able to register the commands they provide. Further, it should behave as follows:

1. If a user issues a 'help' command with no parameters, the plugin must return a list of names for all commands made available by currently loaded plugins to the source that initiated the command.

2. If a user issues a 'help' command with a single parameter, a lookup by command name is performed on that parameter value and either the corresponding command description or a notice indicating that no such command exists must be returned to the source that initiated the command.

This would allow Phergie to be started, stopped, restarted, etc. as a service daemon on *NIX systems. This would require the command to continue running even after the initiating users logs out of the system (see http://en.wikipedia.org/wiki/Nohup). See http://pear.php.net/package/System_Daemon and http://phpadvent.org/2009/daemonize-your-php-by-sean-coates for possible solutions. This should be accounted for in the Phergie package.xml file.