Jonathan Druart
0ab22e1c7c
In order to simplify and make uniform the code, the controller scripts send a Koha::Patron object to the templates instead of all attributes of a patron. That will make the code much more easier to maintain and will be less error-prone. The variable "patron" sent to the templates is supposed to represent the patron the librarian is editing the detail. In the members module and some scripts of the circulation module, the patron's detail are sent one by one to the template. That leads to frustration from developpers (making sure everything is passed from all scripts) and to regression (we got tone of bugs in the last year because of this way to do). With this patch set it will be easy access patron's detail, passing only 1 variable from the controllers. Test plan: Play with the patron and circulation module and make sur the detail of the patron you are editing/seeing info are correctly displayed. Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> Signed-off-by: Josef Moravec <josef.moravec@gmail.com> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
66 lines
2.7 KiB
Text
66 lines
2.7 KiB
Text
[% SET footerjs = 1 %]
|
|
[% INCLUDE 'doc-head-open.inc' %]
|
|
<title>Koha › Delete patron [% patron.firstname %] [% patron.surname %]</title>
|
|
[% INCLUDE 'doc-head-close.inc' %]
|
|
</head>
|
|
<body id="pat_deletemem" class="pat">
|
|
[% INCLUDE 'header.inc' %]
|
|
[% INCLUDE 'patron-search.inc' %]
|
|
|
|
<div id="breadcrumbs"><a href="/cgi-bin/koha/mainpage.pl">Home</a> › <a href="/cgi-bin/koha/members/members-home.pl">Patrons</a> › Delete patron [% patron.firstname %] [% patron.surname %]</div>
|
|
<div id="doc3" class="yui-t2">
|
|
|
|
<div id="bd">
|
|
<div id="yui-main">
|
|
<div class="yui-b">
|
|
[% INCLUDE 'members-toolbar.inc' %]
|
|
[% IF ( ItemsOnIssues || charges || guarantees ) %]
|
|
<div class="dialog alert">
|
|
<h3>Cannot delete patron</h3>
|
|
<ul>
|
|
[% IF ( ItemsOnIssues ) %]
|
|
<li>Patron has [% ItemsOnIssues %] item(s) checked out.</li>
|
|
[% END %]
|
|
[% IF ( charges ) %]
|
|
<li>Patron has [% charges %] in fines.</li>
|
|
[% END %]
|
|
[% IF ( guarantees ) %]
|
|
<li>Patron's record has guaranteed accounts attached.</li>
|
|
[% END %]
|
|
</ul>
|
|
</div>
|
|
[% ELSIF op == 'delete_confirm' and patron %]
|
|
[%# TODO add "patron does not exist" unless patron %]
|
|
<div class="dialog alert">
|
|
<h3>Are you sure you want to delete the patron [% patron.firstname %] [% patron.surname %]? This cannot be undone.</h3>
|
|
<form action="/cgi-bin/koha/members/deletemem.pl">
|
|
<input type="hidden" name="csrf_token" value="[% csrf_token %]" />
|
|
<input type="hidden" name="member" value="[% patron.borrowernumber %]"/>
|
|
<input type="hidden" name="op" value="delete_confirmed" />
|
|
<button type="submit" class="approve"><i class="fa fa-fw fa-check"></i> Yes, delete</button>
|
|
</form>
|
|
<form action="/cgi-bin/koha/members/moremember.pl">
|
|
<input type="hidden" name="borrowernumber" value="[% patron.borrowernumber %]"/>
|
|
<button type="submit" class="deny"><i class="fa fa-fw fa-remove"></i> No, do not delete</button>
|
|
</form>
|
|
</div>
|
|
[% END %]
|
|
[% IF ( keeplocal ) %]
|
|
<div class="dialog message">
|
|
<h3>Remote record deleted, local record kept</h3>
|
|
<p>Patron was marked for deletion from Norwegian national patron database, but the local record was kept.</p>
|
|
</div>
|
|
[% END %]
|
|
</div>
|
|
</div>
|
|
|
|
<div class="yui-b">
|
|
[% INCLUDE 'circ-menu.inc' %]
|
|
</div>
|
|
</div>
|
|
|
|
[% MACRO jsinclude BLOCK %]
|
|
<script type="text/javascript" src="[% interface %]/[% theme %]/js/members-menu_[% KOHA_VERSION %].js"></script>
|
|
[% END %]
|
|
|
|
[% INCLUDE 'intranet-bottom.inc' %]
|