Jonathan Druart
5c7ff786d5
This patch removes 3 subroutines from C4::Letters: - getalert - addalert - delalert And add 3 methods to Koha::Subscription: - subscribers - add_subscriber - remove_subscriber It makes the code cleaner for future cleanup. TODO - we should remove alert.alertid and alert.type, and rename alert.externalid with alert.subscriptionid That way alert will be renamed borrowers_subscriptions (or similar) and will become a simple join table between borrowers and subscriptions. We will need to deal with FK that could not be satisfied. Let's do that after this patch is pushed. Test plan: Subscribe and unsubscribe to email notifications sent when a new issues is available. Make sure everything works as before and you receive the emails. Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
56 lines
3.6 KiB
Text
56 lines
3.6 KiB
Text
[% USE Koha %]
|
|
[% INCLUDE 'doc-head-open.inc' %]
|
|
<title>[% IF ( LibraryNameTitle ) %][% LibraryNameTitle %][% ELSE %]Koha online[% END %] catalog › [% IF ( typeissue ) %]Subscribe to a subscription alert [% ELSIF ( typeissuecancel ) %] Unsubscribe from a subscription alert [% END %]</title>
|
|
[% INCLUDE 'doc-head-close.inc' %]
|
|
[% BLOCK cssinclude %][% END %]
|
|
</head>
|
|
[% INCLUDE 'bodytag.inc' bodyid='opac-alert-subscribe' %]
|
|
[% INCLUDE 'masthead.inc' %]
|
|
|
|
<div class="main">
|
|
<ul class="breadcrumb">
|
|
<li><a href="/cgi-bin/koha/opac-main.pl">Home</a> <span class="divider">›</span></li>
|
|
<li><a href="/cgi-bin/koha/opac-detail.pl?biblionumber=[% biblionumber | html %]">Details for [% bibliotitle %]</a> <span class="divider">›</span></li>
|
|
<li><a href="#">[% IF ( typeissue ) %]Subscribe to a subscription alert [% ELSIF ( typeissuecancel ) %] Unsubscribe from a subscription alert [% END %]</a></li>
|
|
</ul>
|
|
|
|
<div class="container-fluid">
|
|
<div class="row-fluid">
|
|
<div class="span12">
|
|
<div id="useralertsubscribe">
|
|
[% IF ( typeissue ) %]
|
|
<h1>Subscribe to a subscription alert</h1>
|
|
<form action="opac-alert-subscribe.pl" method="post">
|
|
<p>Do you want to receive an email when a new issue for this subscription arrives?</p>
|
|
<h4>[% bibliotitle %]</h4>
|
|
[% IF ( notes ) %]<p>[% notes %]</p>[% END %]
|
|
<input type="hidden" name="subscriptionid" value="[% subscriptionid %]">
|
|
<input type="hidden" name="referer" value="[% referer %]">
|
|
<input type="hidden" name="biblionumber" value="[% biblionumber | html %]">
|
|
<input type="hidden" name="op" value="alert_confirmed">
|
|
<input type="submit" class="btn" value="Yes">
|
|
<a class="cancel" href="opac-serial-issues.pl?biblionumber=[% biblionumber | html %]" >No</a>
|
|
</form>
|
|
[% END %]
|
|
[% IF ( typeissuecancel ) %]
|
|
<h1>Unsubscribe from a subscription alert</h1>
|
|
<form action="opac-alert-subscribe.pl" method="post">
|
|
<p>Please confirm that you do not want to receive email when a new issue arrives for this subscription.</p>
|
|
<h4>[% bibliotitle %]</h4>
|
|
[% IF ( notes ) %]<p>[% notes %]</p>[% END %]
|
|
<input type="hidden" name="subscriptionid" value="[% subscriptionid %]">
|
|
<input type="hidden" name="referer" value="[% referer %]">
|
|
<input type="hidden" name="biblionumber" value="[% biblionumber | html %]">
|
|
<input type="hidden" name="op" value="cancel_confirmed">
|
|
<input type="submit" value="Yes" class="btn">
|
|
<a href="opac-serial-issues.pl?biblionumber=[% biblionumber | html %]" class="cancel">No</a>
|
|
</form>
|
|
[% END %]
|
|
</div> <!-- / #useralertsubscribe -->
|
|
</div> <!-- / .span10/12 -->
|
|
</div> <!-- / .row-fluid -->
|
|
</div> <!-- / .container-fluid -->
|
|
</div> <!-- / .main -->
|
|
|
|
[% INCLUDE 'opac-bottom.inc' %]
|
|
[% BLOCK jsinclude %][% END %]
|