show accesskey also on actionlink title and add accesskey parameter only if a key is defined (FS#2226)

The login wasn't able to modify the session as it was already closed
earlier.

This patch also executes the correct event when logins via XMLRPC are
done.

patch by Chris Green

If the user is already logged in, a 403 is sent instead now.

This is far from perfect but should solve most issues in the recommended
configuration where only authorized users have access. Sending proper
status codes should be implemented when the API implementation
refactoring is done.

unset does not reset the Array's internal pointer which messes up later
calls. array_pop is still much faster than array_splice

Looks like a for loop is faster than array_splice

The file contained a few DOS lineendings and trailing spaces

Turned out a test wasn't really testing what it should have been testing
and thus did hide a bug. Still puzzles me why it still worked some
times.
This patch also sets the default iteration count for bmd5 and pmd5 to 8.

no problems found

When an overlong edit summary was given for an edit, the resulting
changelog line could be longer than the chunk that is handled in the
changelog reader (8192 bytes) causing the reader to abort the operation.

This meant that old revisions where no longer accessible. This patch
fixes the reader (it continues reading chunks until a full line is
found).

However, limiting the summary makes sense and will be added in another
patch.