From baddb381775ebb915ec6a5b2834781b8433aedc5 Mon Sep 17 00:00:00 2001
From: Sophie Meynieux
Date: Mon, 10 Mar 2014 17:07:07 +0100
Subject: [PATCH] Bug 11647 : (follow-up) Test if user is logged in before
proposing to subscribe or unscribe from serial routing list
To test :
1) Search for a subscription with patron notification active
2) If you're not logged in, a message warns you that you must log in to subscribe to notification on new issues
3) If you log in, and have not yet subscribed to this routing list, you've got a link to subscribe
4) once you have subscribed, you've got a ling to unsubscribe
5) if you log out, you've got a message saying you must log in to subscribe
6) if you log in with another user, you've got the link to subscribe or unsubscribe depending on if he has already subscribed.
---
.../opac-tmpl/prog/en/modules/opac-serial-issues.tt | 18 +++++++++++-------
opac/opac-serial-issues.pl | 18 +++++++++++-------
2 files changed, 22 insertions(+), 14 deletions(-)
diff --git a/koha-tmpl/opac-tmpl/prog/en/modules/opac-serial-issues.tt b/koha-tmpl/opac-tmpl/prog/en/modules/opac-serial-issues.tt
index 546bc15..5d590a9 100644
--- a/koha-tmpl/opac-tmpl/prog/en/modules/opac-serial-issues.tt
+++ b/koha-tmpl/opac-tmpl/prog/en/modules/opac-serial-issues.tt
@@ -113,14 +113,18 @@
[% END %]
[% IF ( subscription_LOO.letter ) %]
- [% IF ( subscription_LOO.hasalert ) %]
- You have subscribed to email notification on new issues
- Cancel email notification
-
+ [% IF ( loggedinusername ) %]
+ [% IF ( subscription_LOO.hasalert ) %]
+ You have subscribed to email notification on new issues
+ Cancel email notification
+
+ [% ELSE %]
+
+ Subscribe to email notification on new issues
+
+ [% END %]
[% ELSE %]
-
- Subscribe to email notification on new issues
-
+ You must log in if you want to subscribe to email notification on new issues
[% END %]
[% END %]
diff --git a/opac/opac-serial-issues.pl b/opac/opac-serial-issues.pl
index b534974..edc6248 100755
--- a/opac/opac-serial-issues.pl
+++ b/opac/opac-serial-issues.pl
@@ -62,9 +62,11 @@ if ( $selectview eq "full" ) {
}
# now, check is there is an alert subscription for one of the subscriptions
- foreach (@$subscriptions) {
- if (getalert($loggedinuser,'issue',$_->{subscriptionid})) {
- $_->{hasalert} = 1;
+ if ($loggedinuser) {
+ foreach (@$subscriptions) {
+ if (getalert($loggedinuser,'issue',$_->{subscriptionid})) {
+ $_->{hasalert} = 1;
+ }
}
}
@@ -100,10 +102,12 @@ else {
my $subscriptions = GetSubscriptionsFromBiblionumber($biblionumber);
# now, check is there is an alert subscription for one of the subscriptions
- foreach (@$subscriptions) {
- my $subscription = getalert($loggedinuser,'issue',$_->{subscriptionid});
- if (@$subscription[0]) {
- $_->{hasalert} = 1;
+ if ($loggedinuser){
+ foreach (@$subscriptions) {
+ my $subscription = getalert($loggedinuser,'issue',$_->{subscriptionid});
+ if (@$subscription[0]) {
+ $_->{hasalert} = 1;
+ }
}
}
--
1.7.10.4