From: Simon Perrat <simon.perrat@gmail.com>
Date: Tue, 12 Jun 2012 08:44:40 GMT
Subject: Re: [PATCH v2] add git credential login to remote mediawiki
Message-ID: <CA+hdvHjnVc50rOsxeKUHrwBxRHi4FjcG5sSnnKYWejwXLesKNg@mail.gmail.com>
In-Reply-To: <1339440893-8522-1-git-send-email-Javier.Roucher-Iglesias@enismag.imag.fr>

2012/6/11 <Javier.Roucher-Iglesias@ensimag.imag.fr>

> +       if ($op eq "fill") {
> +               while (<Reader>) {
> +                       my ($key, $value) = /([^=]*)=(.*)/;
> +                       # error if key undef
> +                       if (not defined $key) {
> +                               print STDERR "ERROR reciving reponse git
> credential fill\n";

rec*eiving re*sponse


> +sub ask_login {
> +       run_credential("fill");
> +
> +       if (!$mediawiki->login( {
> +               lgname => $wiki_login,
> +               lgpassword => $wiki_passwd,
> +               lgdomain => $wiki_domain,
> +               } )) {
> +                       print STDERR "Failed to log in mediawiki user
> \"$wiki_login\" on $url\n";

Is there a reason for escaping $wiki_login alone ?

> +                       print STDERR "URL:$wiki_domain $url\n";

Space after :

> +                       print STDERR "(error " .
> +                           $mediawiki->{error}->{code} . ': ' .
> +                           $mediawiki->{error}->{details} . ")\n";

Broken indentation

> +               if (!$wiki_passwd) {
> +                       #user knows, password not.
> +                       ask_login();

know*n (other instances below)
and don't forget space after # ;)


Best regards.