[open-ils-commits] ***SPAM*** [GIT] Evergreen ILS branch rel_2_7 updated. 2d83e10b5ab37d995c8503e8e2b631ac43cad5c7

[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, rel_2_7 has been updated
       via  2d83e10b5ab37d995c8503e8e2b631ac43cad5c7 (commit)
       via  0fa2055705b17b0ac8959e57aec2ead91e939718 (commit)
      from  a946eb97e45c6fde2898fb5390bce21e47d41688 (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 2d83e10b5ab37d995c8503e8e2b631ac43cad5c7
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>

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 0fa2055705b17b0ac8959e57aec2ead91e939718
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>

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