TokyoCabinet pseudo-environment

TokyoCabinet does not have an environment feature to automatically
manage databases, and in particular there is no simple way to close all
open registries on exit. While we do close main registries for all
plugins, this is not usually done for sub-registries. Rather than
shifting the chore of their management to each plugin, we set up a list
of all opened registries, updating it manually and releasing it on
shutdown.

Ruby 1.9 cleanup: variables warnings

Fix most ruby 1.9 warnings about shadowed variables (still one remaining in
keywords.rb). The only significant changes are in the quiz game plugin.

Also fix an issue in dictclient where the block parameter of a method
was not correctly isolated from the previous parameter.

Limit Comparable inclusion to Array

Including Comparable at the top level causes infinite recursions in Ruby
1.9.2

Load the 1.9 compatibility code earlier

The compat19 code touches the monitor mixins which are used by the
logger which is started as soon as ircbot.rb is loaded. To make sure the
logger uses the compat code we must load compat19 before ircbot.

rss: obviously, the precending patch wasn't enough

Proper support for a 'modified' element actually requires its model
to be defined and added to the RSS Atom namespace.

rss: updated can be nil

An Atom item can reply to updated even though its value is nil, so
check for it actually being valued instead. Also, some malformed Atom
feeds use the nonstandard 'modified' element instead, so check for that
too.

fake message: set replied on reply

Although the reply method for fake messages was actually relying on the
original, we still need to set 'replied' to prevent the fake message
from being wrongly delegated to 'unreplied'.

The most obvious bug this commit fixes is the potentially infinite loop
triggered by defining a command reaction triggered by the command name
itself (e.g. reply to ping with cmd:ping).

+ Array#delete_if_at

Allow deletion of an element from the array iff it's at a given position (or in
a given range of positions).

small updates for italian markov

time plugin: check if argument is a nick earlier

When !time <somenick> was being used and the user <somenick> hadn't set
his or her location, the bot would wrongly assume the argument was some
timezone.

Fix by checking against local nicks first.

geoip: blogama is dead, long live ipinfodb

Sadly, the new ipinfodb API requires an API key, and I don't know yet if
I'm allowed to redistributed it (I'm betting on no, but in case I can
it will added with an appropriate patch).

UNO!: when the game hasn't started, it's nobody's turn

UNO!: prevent joins after a challengeable card

If someone joins after a challengeable W+4, he would be inserted between
the last and first player, and if there is a challenge _his_ cards would
be exposed. So prevent joining in such a situation.

UNO!: reset must_play when changing turn

After a successful challenge, the challenged player can choose to
pick&pass instead of playing one of its allowed cards. In such a case,
the must_play array would not get reset, preventing the next player to
be able to play any of its card.

Fix by resetting must_play on next_turn rather than on set_discard.

UNO!: early exit when checking if W+4 was legal

Exit as soon as we find a card that reveals that the W+4 was not a valid
move.

auth module: update for messagemapper changes

Since commit a993d1c358e24ab85621568c10411c5496e2dea8, the message
mappers have a more rational way of reporting failures, but the auth
module was not using it, resulting in the wrong permissions being
granted/revoked whenever allow/deny was being used.

More gettext frozen string woes

The mIRC color for white is 0, not 16

Default mIRC colors are 0-15, and some clients don't react too well to a
color > 15, while most wrap or default. Konversation, for example, will
stop the color command parsing and interpret it as color 1, followed by
the literal digit 6.

So use the 'official' (mIRC) value of 0 for white.

Greet plugin: bot should not greet himself

poll: missing choice value in reply

poll: cope with gettext frozen string

Some gettext versions freeze the string they return, so dup them before
modifying them.

rss: watch handle case during rename

We allow rss handles to be of any case on creation, even though matching
is case-insentivie. However, when renaming an rss using 'rss change
handle' and only changing the case, two things prevented this from
working correctly:
* since the new downcased handle was equal to the old downcased handle,
  the bot would prevent the renaming due to the existence of the new
  handle
* the new handle was forcefully downcased, preventing the user
  from renaming handle 'case' to 'CaSe'.

Fix by checking for this case explicitly, and handling it separately.

remind: fix presence of 'in', 'every' or 'at' in message

if 'in' was present in the message but not part of the time
specification (e.g. because the specification was with 'every' or with
'at'), the parser would fail at the first attempt (look for time after
'in') and fail unnecessarily.

Fix by trying all possibilities and only giving an error if none
succeeds.

rss plugin: don't fail when feed.last_success is nil

The first_run check comparing the time delta from last success failed
when feed.last_success was nil. Prevent this from happening by checking
if we are on the first run before the rest of the checks.

imdb: fix title regexp for missing second date

UNO!: fix g9r9 fix

Commit a306b2c68eb58c1c88057ff7ee78895401adb6a9 introduced a fix to
prevent a player from playing something like g9r9, but the fix actually
prevented any single-card play. Fix by making the \1 optional.

urban: fix regexp

uno plugin: don't allow g9r9 card specification

The regexp should make sure that the second card is equal to the first
specified card, instead of matching the card expression one or two
times. Otherwise, a play like 'pl g9r9' would be accepted when the
player had two r9 cards and could play them.

This is only a aesthetical change because the plays would still be
valid, albeit (1) surprising and (2) mismatching what the user actually
wrote.

translator: don't show provider

imdb: restore 'character of' functionality

imdb: update to latest html

search: gcalc fix

Google changed their HTML again, breaking gcalc. Fix by using the WAP
search and get the first result if it contains an equal sign.

safe_exec fixes

Some plugins (ri, fortune) wrap safe_exec in begin/rescue blocks that
did not get triggered because the exceptions from the execution were
caught in the popen block (and would never get exposed anyway). Fix this
by raising if $? does not indicate success.

dictclient: monkeypatch DICTError

The Ruby/DICT library is so tightly coupled with the rdict client that
its errors exit on initialize, instead of letting the lib user do that.
This makes those errors untrappable from the code.

Fix by monkeypatching DICTError to just act like the other exceptions in
Ruby.

youtube: fix detection of video pages

TokyoCabinet #fwmkeys() expects a string

Convert the (optional) first argument to #each*() methods to a string to
ensure the calls work as expected. This fixes a problem with
@registry.each not working as expected, and although it would have been
sufficient to convert the default for the first argument from nil to the
empty string, the approach implemented by this patch is more robust
overall.

TokyoCabinet: upgrade from BDB using case-insensitive tree

DBFatal at load time deserve special treatment

Don't trap them, rerise and let the bot handle them

time plugin: properly reset TZ in on_timezone

poll: suppress warning

compat19: fix condition monitor waiting

When timing out during a wait, return false instead of raising. This
fixes a problem with TimeoutError being raised during restarts on
ruby 1.9.x

Revert "markov: removed unnecessary mutexes"

This reverts commit 06485aeb187dde5e81204b06c8e956e7e035c323. The mutex
is necessary because of the concurrent learning and database conversion.
Some other approach should be used instead (e.g. skipping the mutex if
not running a conversion).

IRC framework: Server#to_s must always return a String

Having Server#to_s alias the hostname would cause problems when the
hostname was nil (e.g. after a disconnect). Fix by making sure that to_s
always returns a string instead.

* more BDB leaking fixed

twitter: fix status count

* wrap BDB::Fatal classes for abstract trapping

When using a DB backend different from BDB, the BDB constant may not be
defined, causing a NameError during error trapping in the main loop. Fix
this by defining our own DBFatal error that maps to BDB::Fatal in the
BDB case and is defined as an (unused) Exception for TokyoCabinet.

time: support POSIX time

Check BDB version only if it's loaded

TokyoCabinet: more BDB checks

TokyoCabinet: only warn about skipped upgrade if the old reg was found

TokyoCabinet: fix extension in 0.9.9 upgrade path

TokyoCabinet: 0.9.5-to-0.9.9 must use BDB only

When upgrading from 0.9.5 data, write the 0.9.9-style regisitry using
BDB still rather than trying to use TokyoCabinet already. This makes
sure that the next step (0.9.9 to modern times) works correctly.

(Moreover, the existing BDB-to-TC attempt wouldn't have worked anyway
due to wrong var names. I doubt anybody will ever hit this path and
notice though.)

translator: stricter url mapping

time: be consistent with <Continent>/<City> notation

translator: url translation through google

time: use "time zone" consistently in replies

lastfm: fix set user suggestion reply

time: update help

tumblr: typo in var name

tumblr: support tags

Tags are specified on the IRC line by putting them between square
brakets.

tumblr: embed Vimeo videos

tumblr: adjust TODO list

tumblr: support reblogging and fix HTML issues

Detect tumblr posts and use the reblogging API to post them. Also
produce HTML-escaped lines for video and photo captions because
the format=markdown specification doesn't seem to apply to them.

twitter: warn about failed authorization

Fly-by whitespace cleanups too.

twitter: report missing key/secret configuration

Instead of failing (silently) when the OAuth process cannot complete due
to a missing key/secret pair, report the issue to the user.

twitter: let user know when no status was found

time: calibrate to user's timezone when parsing time input

time: add timestamp parser as a fallback to the default functionality

twitter: don't fail on missing oauth

When OAuth is missing, some barebone functionality in the twitter plugin
is still available (namely, status reading from public timelines), so
don't fail loading and just warn the users and admins about the reduced
availability.

twitter: bugfix for invalid OAuth PIN Entry

There was a bug in "twitter pin" functionality where it didn't
check to verify that the PIN entered by the user was valid. As
a result, if the user entered an invalid PIN, the bot would
not respond as to whether or not twitter account binding was
successful. I replaced it with an error message if the PIN
is invalid. I also changed the error message for someone
who tries to enter a PIN without first using "twitter
authorize" to be more clear.

twitter: add OAuth support

Module now requires twitter.key and twitter.secret config parameters for
write access and private users' statuses. Added twitter authorize,
twitter deauthorize, and twitter pin features, removed twitter identify
feature, as basic authentication is no longer supported.

rss plugin: support atom categories

Atom categories don't have a content attribute, and obviously they don't
have an empty? method. Squash them to their label rather than bombing
out with error during the blank2nil processing.

search: google calc can be in non-h2 headers

digg: cleanup whitespace in title

digg: new URL

digg: fix message when news could not be retrieved

hangman plugin: don't fail silently on !hangman

If the random generation site failed for any reason (eg. by returning a
non-2xx page), starting a hangman game would fail silently.

Catch this case and announce the failure.

auth: minimal help about user import/export

Plugin rake task: specify the path the plugin was written to

Add a plugin skeleton generator

`rake plugin`

Fix typo

lastfm: require spotify plugin to be enabled for spotify link info

search: skip non-search-results link

This prevents the regexp scan to grab the Google Privacy link when no
results were found.

translator: support both pre-1.0 and post-1.0 Mechanize namespaces

UNO!: replace_player() returns whether it worked or not

UNO!: don't replace with nonexisting players

UNO!: prevent a challenge after a pick after a W+4

hangman: fix playing with a private message

The target was being of the wrong class (String vs Irc::Channel),
busting all the hash lookups.

hangman: IRCify HTML in definitions

hangman: ensure 'hangman define' always answers

imdb: fix plot detection

imdb: fix grab_info()

geoip: massage a few messages

Solve a few gettext issue, and re-introduce the commas around the
region.

hangman plugin: update a couple of messages

It's 'hangman define' now, and we should only use one prefix in the
example command

factoids plugin: more help

UNO! plugin: clean join_timer when game starts

This prevents spurious messages about the game join timer being
cancelled when a game is ended by all players but one dropping out.

load-gettext: cope with ruby gettext 2.1.0

freshmeat plugin: use XML news if possible

Freshmeat now provides the news in XML form under /index.xml, which
provides more information, and in a more computer-friendly way. Use that
if possible (correctly-configured API token), fall back to RSS parsing
otherwise.

HTTP: only set cookies for the correct domain

When a redirect has a Set-Cookie: header, check if the cookie domain is
valid for the host we are redirected to. If not, don't set the cookie
in the new request.

HTTP: support servers that forget to escape the redirect location