[open-ils-commits] ***SPAM*** [GIT] Evergreen ILS branch tags/rel_2_7_1 updated. d4b2ab2fe147be289bcdd5869dd664e013306818

[Evergreen Git]

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "Evergreen ILS".

The branch, tags/rel_2_7_1 has been updated
       via  d4b2ab2fe147be289bcdd5869dd664e013306818 (commit)
       via  b9f7003c64b2964a6839bc3642b60920d94878d0 (commit)
      from  e9667172a9b03165837df776862ca7ed83f5ab69 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit d4b2ab2fe147be289bcdd5869dd664e013306818
Author: Mike Rylander <mrylander at gmail.com>
Date:   Thu Nov 6 16:03:36 2014 -0500

    LP#1390225: Fail to care about errors from auth.session.delete
    
    We're just tossing this call over the wall and moving on.  We don't
    care if the session wasn't there to delete or even if we didn't pass
    a session to the server.
    
    Signed-off-by: Mike Rylander <mrylander at gmail.com>
    Signed-off-by: Chris Sharp <csharp at georgialibraries.org>
    Signed-off-by: Jason Stephenson <jstephenson at mvlc.org>
    Signed-off-by: Ben Shum <bshum at biblio.org>

diff --git a/Open-ILS/src/perlmods/lib/OpenILS/WWW/EGCatLoader.pm b/Open-ILS/src/perlmods/lib/OpenILS/WWW/EGCatLoader.pm
index 868907d..3d717ad 100644
--- a/Open-ILS/src/perlmods/lib/OpenILS/WWW/EGCatLoader.pm
+++ b/Open-ILS/src/perlmods/lib/OpenILS/WWW/EGCatLoader.pm
@@ -483,11 +483,13 @@ sub load_logout {
     # while logged in, go ahead and clear it out.
     $self->clear_anon_cache;
 
-    $U->simplereq(
-        'open-ils.auth',
-        'open-ils.auth.session.delete',
-        $self->cgi->cookie(COOKIE_SES)
-    );
+    try { # a missing auth token will cause an ugly explosion
+        $U->simplereq(
+            'open-ils.auth',
+            'open-ils.auth.session.delete',
+            $self->cgi->cookie(COOKIE_SES)
+        );
+    } catch Error with {};
 
     return $self->generic_redirect(
         $redirect_to || $self->ctx->{home_page},

commit b9f7003c64b2964a6839bc3642b60920d94878d0
Author: Mike Rylander <mrylander at gmail.com>
Date:   Thu Nov 6 15:36:16 2014 -0500

    LP#1390225: redirect to ctx.home_page instead of through ctx.logout_page
    
    This way, we skip the auth.session.delete call that will error out because
    we no longer have the authtoken cookie.
    
    Signed-off-by: Mike Rylander <mrylander at gmail.com>
    Signed-off-by: Chris Sharp <csharp at georgialibraries.org>
    Signed-off-by: Jason Stephenson <jstephenson at mvlc.org>
    Signed-off-by: Ben Shum <bshum at biblio.org>

diff --git a/Open-ILS/src/templates/opac/parts/base.tt2 b/Open-ILS/src/templates/opac/parts/base.tt2
index 2513973..03e950d 100644
--- a/Open-ILS/src/templates/opac/parts/base.tt2
+++ b/Open-ILS/src/templates/opac/parts/base.tt2
@@ -6,7 +6,7 @@
         [% IF ctx.refresh %]
         <meta http-equiv="refresh" content="[% ctx.refresh %]" />
         [% ELSIF ctx.authtime AND !ctx.is_staff %]
-        <meta http-equiv="refresh" content="[% ctx.authtime %]; url=[% ctx.logout_page %]" />
+        <meta http-equiv="refresh" content="[% ctx.authtime %]; url=[% ctx.home_page %]" />
         [% END %]
         <meta name = "viewport" content = "initial-scale = 1.0">
         <link rel="stylesheet" type="text/css" href="[% ctx.media_prefix %]/css/skin/default/opac/semiauto.css" />

-----------------------------------------------------------------------

Summary of changes:
 .../src/perlmods/lib/OpenILS/WWW/EGCatLoader.pm    |   12 +++++++-----
 Open-ILS/src/templates/opac/parts/base.tt2         |    2 +-
 2 files changed, 8 insertions(+), 6 deletions(-)


hooks/post-receive
-- 
Evergreen ILS