[Koha-bugs] [Bug 8753] Add forgot password link to OPAC

[bugzilla-daemon at bugs.koha-community.org]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8753

--- Comment #32 from M. Tompsett <mtompset at hotmail.com> ---
Comment on attachment 24846
  --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=24846
Bug 8753 - Add forgot password link to OPAC

Review of attachment 24846:
 --> (http://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html&bug=8753&attachment=24846)
-----------------------------------------------------------------

::: C4/Members.pm
@@ +2266,5 @@
> +
> +sub SendPasswordRecoveryEmail {
> +    my $borrowernumber = shift;
> +    my $email = shift;
> +    my $query = shift;

Perhaps with the code later you were trying to handle:
    my $query = shift // CGI->new();

@@ +2285,5 @@
> +    #define to/from emails
> +    my $kohaEmail = C4::Context->preference( 'KohaAdminEmailAddress' );
> +
> +    #create link
> +    my $https = $query->https() ? "https://" : "http://";

I believe I have seen others call this protocol, but changing this isn't
necessary.

@@ +2290,5 @@
> +    my $uuidLink = $https . C4::Context->preference( 'OPACBaseURL' ) . "/cgi-bin/koha/opac-password-recovery.pl?uniqueKey=$uuid_str";
> +    #warn $uuidLink;
> +
> +    #build email content
> +    my $query = new CGI;

Why do you need this?! You have line 2270.

-- 
You are receiving this mail because:
You are watching all bug changes.