From 01e74be8cbeb125d8489317ba84acd890962c88f Mon Sep 17 00:00:00 2001
From: Dan Wells <dbw2@calvin.edu>
Date: Thu, 13 Feb 2014 11:56:19 -0500
Subject: [PATCH] LP#1526543 Truthiness check for reset_password is flawed

We expect boolean options in config.tt2 to be 'true' or 'false',
but the code was simple asking 'IF reset_password', which is true
for either value.

Instead, match on the string (as we do in other places).

To test:
- Load the stock catalog login page
- Notice the "Forgot your password?" link, despite the default config
  being 'false'
- Apply patch, leaving 'reset_password' value in config.tt2 as 'false'
- RESULT: Password reset link doesn't show up

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Remington Steed <rjs7@calvin.edu>
---
 Open-ILS/src/templates/opac/parts/login/form.tt2 | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Open-ILS/src/templates/opac/parts/login/form.tt2 b/Open-ILS/src/templates/opac/parts/login/form.tt2
index 9b13f587a2..88d42c91eb 100644
--- a/Open-ILS/src/templates/opac/parts/login/form.tt2
+++ b/Open-ILS/src/templates/opac/parts/login/form.tt2
@@ -31,7 +31,7 @@
                 <input id="password_field" name="password" type="password" />
             </div>
             [% INCLUDE "opac/parts/login/password_hint.tt2" %]
-            [% IF reset_password %]
+            [% IF reset_password == 'true' %]
             <a style="font-size: 80%" href='[% mkurl(ctx.opac_root _ '/password_reset', {}, 1) %]'>[% l('Forgot your password?') %]</a>
             [% END %]
         </div>
-- 
2.11.0