Bug 21628: (follow-up) Indentation and filter corrections
This patch modifies the new waiting_holds.inc to fix the indentation. A template variable filter has been added to quiet QA tool warnings. Buttons are now styled as Bootstrap buttons. To test, apply the patch and view the holds report. Confirm that everything looks correct. Validate the HTML and confirm that there are no errors in the HTML of the body of the page. Followed test plan, everything looked correct and no HTML errors on waitingreserves.tt found by HTML validator Signed-off-by: Alex Buckley <alexbuckley@catalyst.net.nz> Signed-off-by: Josef Moravec <josef.moravec@gmail.com> Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
This commit is contained in:
parent
b23e544bd9
commit
70f7d801b7
1 changed files with 58 additions and 49 deletions
|
@ -1,51 +1,60 @@
|
|||
[% USE ItemTypes %]
|
||||
<table id="[% table_name %]">
|
||||
<thead><tr>
|
||||
<th class="title-string">Waiting since</th>
|
||||
<th class="title-string">Date hold placed</th>
|
||||
<th class="anti-the">Title</th>
|
||||
<th>Patron</th>
|
||||
<th>Home branch</th>
|
||||
<th>Current location</th>
|
||||
<th>Call number</th>
|
||||
<th>Copy number</th>
|
||||
<th>Enumeration</th>
|
||||
<th>Action</th>
|
||||
</tr></thead>
|
||||
<tbody>[% FOREACH reserveloo IN reserveloop %]
|
||||
<tr>
|
||||
<td><span title="[% reserveloo.waitingdate | html %]">[% reserveloo.waitingdate | $KohaDates %]</span></td>
|
||||
<td><span title="[% reserveloo.reservedate | html %]">[% reserveloo.reservedate | $KohaDates %]</span></td>
|
||||
<td>[% INCLUDE 'biblio-default-view.inc' biblionumber = reserveloo.biblionumber %]
|
||||
[% reserveloo.biblio.title | html %] [% FOREACH subtitl IN reserveloo.biblio.subtitles %] [% subtitl.subfield | html %][% END %]
|
||||
</a>
|
||||
[% UNLESS ( item_level_itypes ) %][% IF ( ItemTypes.GetDescription(reserveloo.item.effective_itemtype) ) %] (<b>[% ItemTypes.GetDescription(reserveloo.item.effective_itemtype) | html %]</b>)[% END %][% END %]
|
||||
<br />Barcode: [% reserveloo.item.barcode | html %]
|
||||
</td>
|
||||
<td><a href="/cgi-bin/koha/members/moremember.pl?borrowernumber=[% reserveloo.borrower.borrowernumber | uri %]">[% reserveloo.borrower.surname | html %], [% reserveloo.borrower.firstname | html %]</a>
|
||||
[% IF ( reserveloo.borrower.phone ) %]<br />[% reserveloo.borrower.phone | html %][% END %]
|
||||
[% IF ( reserveloo.borrower.first_valid_email_address ) %]<br /><a href="mailto:[% reserveloo.borrower.first_valid_email_address | uri %]?subject=Hold waiting: [% reserveloo.biblio.title | uri %]">
|
||||
[% reserveloo.borrower.first_valid_email_address | html %]</a>[% END %]
|
||||
</td>
|
||||
<td>[% Branches.GetName( reserveloo.item.homebranch ) | html %]</td>
|
||||
<td>[% Branches.GetName( reserveloo.item.holdingbranch ) | html %]</td>
|
||||
<td>[% reserveloo.item.itemcallnumber | html %]</td>
|
||||
<td>[% reserveloo.item.copynumber | html %]</td>
|
||||
<td>[% reserveloo.item.enumchron | html %]</td>
|
||||
<td>
|
||||
<form name="cancelReserve" action="waitingreserves.pl" method="post">
|
||||
<input type="hidden" name="borrowernumber" value="[% reserveloo.borrower.borrowernumber | html %]" />
|
||||
<input type="hidden" name="itemnumber" value="[% reserveloo.item.itemnumber | html %]" />
|
||||
<input type="hidden" name="fbr" value="[% reserveloo.item.holdingbranch | html %]" />
|
||||
<input type="hidden" name="tbr" value="[% reserveloo.item.homebranch | html %]" />
|
||||
<input type="hidden" name="tab" value="holdswaiting">
|
||||
[% IF ( reserveloo.item.homebranch != reserveloo.item.holdingbranch ) %]
|
||||
<input type="submit" value="Cancel hold and return to : [% Branches.GetName( reserveloo.item.homebranch ) | html %]" />
|
||||
[% ELSE %]
|
||||
<input type="submit" value="Cancel hold" />
|
||||
[% END %]
|
||||
</form>
|
||||
</td>
|
||||
</tr>
|
||||
[% END %]</tbody>
|
||||
<table id="[% table_name | html %]">
|
||||
<thead>
|
||||
<tr>
|
||||
<th class="title-string">Waiting since</th>
|
||||
<th class="title-string">Date hold placed</th>
|
||||
<th class="anti-the">Title</th>
|
||||
<th>Patron</th>
|
||||
<th>Home branch</th>
|
||||
<th>Current location</th>
|
||||
<th>Call number</th>
|
||||
<th>Copy number</th>
|
||||
<th>Enumeration</th>
|
||||
<th>Action</th>
|
||||
</tr>
|
||||
</thead>
|
||||
<tbody>
|
||||
[% FOREACH reserveloo IN reserveloop %]
|
||||
<tr>
|
||||
<td><span title="[% reserveloo.waitingdate | html %]">[% reserveloo.waitingdate | $KohaDates %]</span></td>
|
||||
<td><span title="[% reserveloo.reservedate | html %]">[% reserveloo.reservedate | $KohaDates %]</span></td>
|
||||
<td>[% INCLUDE 'biblio-default-view.inc' biblionumber = reserveloo.biblionumber %]
|
||||
[% reserveloo.biblio.title | html %] [% FOREACH subtitl IN reserveloo.biblio.subtitles %] [% subtitl.subfield | html %][% END %]
|
||||
</a>
|
||||
[% UNLESS ( item_level_itypes ) %]
|
||||
[% IF ( ItemTypes.GetDescription(reserveloo.item.effective_itemtype) ) %] (<b>[% ItemTypes.GetDescription(reserveloo.item.effective_itemtype) | html %]</b>)
|
||||
[% END %]
|
||||
[% END %]
|
||||
<br />Barcode: [% reserveloo.item.barcode | html %]
|
||||
</td>
|
||||
<td><a href="/cgi-bin/koha/members/moremember.pl?borrowernumber=[% reserveloo.borrower.borrowernumber | uri %]">[% reserveloo.borrower.surname | html %], [% reserveloo.borrower.firstname | html %]</a>
|
||||
[% IF ( reserveloo.borrower.phone ) %]<br />[% reserveloo.borrower.phone | html %][% END %]
|
||||
[% IF ( reserveloo.borrower.first_valid_email_address ) %]
|
||||
<br /><a href="mailto:[% reserveloo.borrower.first_valid_email_address | uri %]?subject=[% "Hold waiting: " | uri %][% reserveloo.biblio.title | uri %]">
|
||||
[% reserveloo.borrower.first_valid_email_address | html %]</a>
|
||||
[% END %]
|
||||
</td>
|
||||
<td>[% Branches.GetName( reserveloo.item.homebranch ) | html %]</td>
|
||||
<td>[% Branches.GetName( reserveloo.item.holdingbranch ) | html %]</td>
|
||||
<td>[% reserveloo.item.itemcallnumber | html %]</td>
|
||||
<td>[% reserveloo.item.copynumber | html %]</td>
|
||||
<td>[% reserveloo.item.enumchron | html %]</td>
|
||||
<td>
|
||||
<form name="cancelReserve" action="waitingreserves.pl" method="post">
|
||||
<input type="hidden" name="borrowernumber" value="[% reserveloo.borrower.borrowernumber | html %]" />
|
||||
<input type="hidden" name="itemnumber" value="[% reserveloo.item.itemnumber | html %]" />
|
||||
<input type="hidden" name="fbr" value="[% reserveloo.item.holdingbranch | html %]" />
|
||||
<input type="hidden" name="tbr" value="[% reserveloo.item.homebranch | html %]" />
|
||||
<input type="hidden" name="tab" value="holdswaiting">
|
||||
[% IF ( reserveloo.item.homebranch != reserveloo.item.holdingbranch ) %]
|
||||
<button type="submit" class="btn btn-default btn-xs"><i class="fa fa-remove"></i> Cancel hold and return to: [% Branches.GetName( reserveloo.item.homebranch ) | html %]</button>
|
||||
[% ELSE %]
|
||||
<button type="submit" class="btn btn-default btn-xs"><i class="fa fa-remove"></i> Cancel hold</button>
|
||||
[% END %]
|
||||
</form>
|
||||
</td>
|
||||
</tr>
|
||||
[% END %]
|
||||
</tbody>
|
||||
</table>
|
||||
|
|
Loading…
Reference in a new issue