[aur-dev] [PATCH] Drop the message when no changes were made to an account
Lukas Fleischer
lfleischer at archlinux.org
Sat Jun 27 15:10:26 UTC 2015
On Sat, 27 Jun 2015 at 16:54:04, Lukas Fleischer wrote:
> On Sat, 27 Jun 2015 at 16:26:07, Johannes Löthberg wrote:
> > When no changes has been made to the account there's no need for a
> > message, and it is no longer applicable since the setting of SSH keys is
> > done in a separate function now.
> >
> > Reported-by: Alexis Chotard <alexis.horgix.chotard at gmail.com>
> > Signed-off-by: Johannes Löthberg <johannes at kyriasis.com>
> > ---
> > web/lib/acctfuncs.inc.php | 5 +----
> > 1 file changed, 1 insertion(+), 4 deletions(-)
> > [...]
>
> Applied, thanks!
Just noticed that this doesn't seem to work. PDO::exec() returns false
if an error occurs and the number of modified rows otherwise. So this
results in an empty page (with just a heading) when no changes are made.
Maybe we should just replace "!$result" with "$result === false"
instead?
Regards,
Lukas
More information about the aur-dev
mailing list