Hi Michael, first of all thanks for contributing to the AUR Development.

Imho, you must try to use quotes correctly, i mean simples and doubles, you can find a good tutorial here:

http://ve2.php.net/types.string

Or you can google 'simple vs doble quotes + php', that will apply to you in many programming languages.

By example that line:

print "" . $login_error . " \n"; --> bad written

Will work, and that should be enough, but no, why doble quotes " then ' and then you concatenate the string, imho that should be something like:

print ''.$login_error." \n"; // I use double quotes for the \n

Or just something like:

print "$login_error \n";

and that applies in many senses by examples:
// why "? use ' for that case when you are not intending to parse some value is better
$foo["var"] = "test";
include "foo/bar.php"
print $_SESSION["foo"];
$bar = array("foo","bar");

etc

Just a recommendation trying to have clean code, i know that AUR isn't complete clean, but the patches should be, then for next releases will be easier to clean and write code correctly.

Just a recommendation, again, thanks for your patches :-)

--
Angel Velásquez
angvp @ irc.freenode.net
Linux Counter: #359909
Arch Linux Trusted User