KPAC Login Redirect Fix

Fixes the redirect from the KPAC login to return back to the
home page of the KPAC from the /opac/login screen, rather
than back to the /opac/home screen, which is the current
behavior, and confuses users of the KPAC when they end
up back in the OPAC after having logged in.

Modified KPAC login code to redirect to KPAC after logging in

Signed-off-by: Adam Bowling <abowling@emeralddata.net>
Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>
Conflicts:

	Open-ILS/src/templates/kpac/parts/header.tt2

diff --git a/Open-ILS/src/templates/kpac/parts/header.tt2 b/Open-ILS/src/templates/kpac/parts/header.tt2
index 8f75dce..dc26069 100644 (file)
--- a/Open-ILS/src/templates/kpac/parts/header.tt2
+++ b/Open-ILS/src/templates/kpac/parts/header.tt2
@@ -29,7 +29,8 @@
                 {redirect_to => redir.replace('^https:', 'http:')}, 1) %]">[% 
                     l('Logout ([_1] [_2])', ctx.user.first_given_name, ctx.user.family_name) %]</a>
         [% ELSE %]
-            <a href="[% mkurl(ctx.opac_root _ '/login').replace('^http:', 'https:') %]">[% l('My Account') %]</a>
+                   [% redir = CGI.url('-base' => 1) _ ctx.kpac_root _ '/home' %]
+            <a href="[% mkurl(ctx.opac_root _ '/login?redirect_to=' _ redir).replace('^http:', 'https:') %]">[% l('Login') %]</a>
         [% END %]
     </div>
     

diff --git a/Open-ILS/src/templates/opac/parts/login/form.tt2 b/Open-ILS/src/templates/opac/parts/login/form.tt2
index 9a1d702..928013b 100644 (file)
--- a/Open-ILS/src/templates/opac/parts/login/form.tt2
+++ b/Open-ILS/src/templates/opac/parts/login/form.tt2
@@ -40,6 +40,11 @@
         [%
             redirect = CGI.param('redirect_to');
             # Don't use referer unless we got here from elsewhere within the TPAC
+           IF ctx.kpac_root;
+               IF !redirect AND ctx.referer.match('^https?://' _ ctx.hostname _ ctx.kpac_root);
+                   redirect = ctx.referer;
+               END;
+           END;
             IF !redirect AND ctx.referer.match('^https?://' _ ctx.hostname _ ctx.opac_root);
                 redirect = ctx.referer;
             END;