Owen Leonard
3156950e0a
There are some issues with the template changes introduced by the EDI
feature in Bug 7736. This patch makes the following changes:
- Use Font Awesome icon on the Acquisitions basket page instead of a
Glyphicon.
- Use Font Awesome icon on the Acquisitions basket group page instead of
a Glyphicon.
- For consistency's sake, use the phrase "Library EANs" instead of "EDI
EANs" in menus and the Library EANs page.
- On the EDI accounts page:
- Correct the markup of the title tag.
- Improve the breadcrumbs to eliminate redundancy.
- Use Font Awesome icon instead of Glyphicon for the "New account"
button.
- Change or remove "maxlength" attribute of form fields to match table
structure.
- Move the contents of some title attributes into a visible hint.
- Use 'checked="checked"' instead of 'checked'. Koha templates use
XTHML-style attributes.
- Correct template variable in account deletion confirmation dialog.
Before this patch the vendor name was not correctly displayed.
- Show a message if there are no accounts defined rather than an empty
table.
- Remove use of "highlight" class from table rows (Bug 15927).
- Correct parameter name in link to vendor details page (should be
booksellerid instead of supplierid).
- Correct the markup of the deletion confirmation dialog (Bug 15785).
- Use Bootstrap-style buttons for "edit" and "delete," with Font
Awesome icons.
- On the Library EANs page:
- Correct the markup of the title tag.
- Make capitalization of "EAN" consistent.
- Use Font Awesome icon instead of Glyphicon for the "New EAN" button.
- Show a message if there are no EANs defined rather than an empty
table.
- Change or remove "maxlength" attribute of form fields to match table
structure.
- Remove use of "highlight" class from table rows (Bug 15927).
- Correct the markup of the deletion confirmation dialog (Bug 15785).
- Use Bootstrap-style buttons for "edit" and "delete," with Font
Awesome icons.
To test, apply the patch and review the affected templates. Confirm that
pages look correct and work correctly.
Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com>
All seems to work and looks well,
add/edit/delete EDI accounts and Library EANs works Ok
No errors
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Brendan Gallagher <bredan@bywatersolutions.com>
172 lines
5.9 KiB
Text
172 lines
5.9 KiB
Text
[% INCLUDE 'doc-head-open.inc' %]
|
|
<title>Koha › Administration › Library EANs
|
|
[% IF ean_form %]
|
|
[% IF ean %]
|
|
› Modify library EAN
|
|
[% ELSE %]
|
|
› Add new library EAN
|
|
[% END %]
|
|
[% END %]
|
|
[% IF delete_confirm %]
|
|
› Confirm deletion of EAN
|
|
[% END %]
|
|
</title>
|
|
[% INCLUDE 'doc-head-close.inc' %]
|
|
</head>
|
|
<body id="admin_edi_ean" class="admin">
|
|
[% INCLUDE 'header.inc' %]
|
|
[% INCLUDE 'cat-search.inc' %]
|
|
|
|
<div id="breadcrumbs">
|
|
<a href="/cgi-bin/koha/mainpage.pl">Home</a>
|
|
› <a href="/cgi-bin/koha/admin/admin-home.pl">Administration</a>
|
|
[% IF ean_form %]
|
|
[% IF ean %]
|
|
› <a href="/cgi-bin/koha/admin/edi_ean_accounts.pl">Library EANs</a> › Modify library EAN
|
|
[% ELSE %]
|
|
› <a href="/cgi-bin/koha/admin/edi_ean_accounts.pl">Library EANs</a> › Add new library EAN
|
|
[% END %]
|
|
[% ELSIF delete_confirm %]
|
|
› <a href="/cgi-bin/koha/admin/edi_ean_accounts.pl">Library EANs</a> › Confirm deletion of EAN
|
|
[% ELSE %]
|
|
› Library EANs
|
|
[% END %]
|
|
</div>
|
|
|
|
<div id="doc3" class="yui-t2">
|
|
|
|
<div id="bd">
|
|
<div id="yui-main">
|
|
<div class="yui-b">
|
|
[% IF display %]
|
|
<div id="toolbar" class="btn-toolbar">
|
|
<a class="btn btn-small" id="newediean" href="/cgi-bin/koha/admin/edi_ean_accounts.pl?op=ean_form">
|
|
<i class="fa fa-plus"></i>
|
|
New EAN
|
|
</a>
|
|
</div>
|
|
[% END %]
|
|
|
|
[% IF ean_form %]
|
|
<form action="/cgi-bin/koha/admin/edi_ean_accounts.pl" name="Eanform" method="post">
|
|
<input type="hidden" name="op" value="save" />
|
|
[% IF ean %]
|
|
<input type="hidden" name="oldbranchcode" value="[% ean.branch.branchcode %]" />
|
|
<input type="hidden" name="oldean" value="[% ean.ean %]" />
|
|
[% END %]
|
|
|
|
<fieldset class="rows">
|
|
<legend>
|
|
[% IF ean %]
|
|
Modify EAN
|
|
[% ELSE %]
|
|
New EAN
|
|
[% END %]
|
|
</legend>
|
|
|
|
<ol>
|
|
<li>
|
|
<label for="branchcode">Library: </label>
|
|
<select name="branchcode" id="branchcode">
|
|
[% FOREACH branch IN branches %]
|
|
[% IF branch.branchcode == ean.branch.branchcode %]
|
|
<option value="[% branch.branchcode %]" selected="selected">[% branch.branchname %]</option>
|
|
[% ELSE %]
|
|
<option value="[% branch.branchcode %]">[% branch.branchname %]</option>
|
|
[% END %]
|
|
[% END %]
|
|
</select>
|
|
</li>
|
|
<li>
|
|
<label for="description">Description: </label>
|
|
<input type="text" name="description" id="description" size="30" maxlength="128" value="[% ean.description %]" />
|
|
</li>
|
|
<li>
|
|
<label for="ean">EAN: </label>
|
|
<input type="text" name="ean" id="ean" size="20" maxlength="15" value="[% ean.ean %]" />
|
|
</li>
|
|
<li>
|
|
<label for="id_code_qualifier">Qualifier: </label>
|
|
<select name="id_code_qualifier" id="id_code_qualifier">
|
|
[% FOREACH qualifier IN code_qualifiers %]
|
|
[% IF qualifier.code == ean.id_code_qualifier %]
|
|
<option value="[% qualifier.code %]" selected="selected">
|
|
[% qualifier.description %] ([% qualifier.code %])
|
|
</option>
|
|
[% ELSE %]
|
|
<option value="[% qualifier.code %]">
|
|
[% qualifier.description %] ([% qualifier.code %])
|
|
</option>
|
|
[% END %]
|
|
[% END %]
|
|
</select>
|
|
</li>
|
|
</ol>
|
|
</fieldset>
|
|
|
|
<fieldset class="action">
|
|
<input type="submit" value="Submit"/>
|
|
<a href="/cgi-bin/koha/admin/edi_ean_accounts.pl" class="cancel">Cancel</a>
|
|
</fieldset>
|
|
</form>
|
|
[% END %]
|
|
|
|
[% IF delete_confirm %]
|
|
<div class="dialog alert">
|
|
<h3>Delete EAN [% ean.ean %] for [% ean.branch.branchname %]?</h3>
|
|
<form action="/cgi-bin/koha/admin/edi_ean_accounts.pl" method="post">
|
|
<input type="hidden" name="op" value="delete_confirmed" />
|
|
<input type="hidden" name="branchcode" value="[% ean.branch.branchcode %]" />
|
|
<input type="hidden" name="ean" value="[% ean.ean %]" />
|
|
<button type="submit" class="approve"><i class="fa fa-fw fa-check"></i> Yes, delete</button>
|
|
</form>
|
|
<form action="/cgi-bin/koha/admin/edi_ean_accounts.pl" method="get">
|
|
<button type="submit" class="deny"><i class="fa fa-fw fa-remove"></i> No, do not delete</button>
|
|
</form>
|
|
</div>
|
|
[% END %]
|
|
|
|
[% IF display %]
|
|
[% IF ( eans ) %]
|
|
<h2>Library EANs</h2>
|
|
<table>
|
|
<tr>
|
|
<th>Library</th>
|
|
<th>Description</th>
|
|
<th>EAN</th>
|
|
<th>Qualifier</th>
|
|
<th>Actions</th>
|
|
</tr>
|
|
[% FOREACH ean IN eans %]
|
|
<tr>
|
|
<td>[% ean.branch.branchname %]</td>
|
|
<td>[% ean.description %]</td>
|
|
<td>[% ean.ean %]</td>
|
|
<td>
|
|
[% FOREACH qualifier IN code_qualifiers %]
|
|
[% IF qualifier.code == ean.id_code_qualifier %]
|
|
[% qualifier.description %] ([% qualifier.code %])
|
|
[% END %]
|
|
[% END %]
|
|
</td>
|
|
<td class="actions">
|
|
<a class="btn btn-mini" href="/cgi-bin/koha/admin/edi_ean_accounts.pl?op=ean_form&branchcode=[% ean.branch.branchcode %]&ean=[% ean.ean %]"><i class="fa fa-pencil"></i> Edit</a>
|
|
<a class="btn btn-mini" href="/cgi-bin/koha/admin/edi_ean_accounts.pl?op=delete_confirm&branchcode=[% ean.branch.branchcode %]&ean=[% ean.ean %]"><i class="fa fa-trash"></i> Delete</a>
|
|
</td>
|
|
</tr>
|
|
[% END %]
|
|
</table>
|
|
[% ELSE %]
|
|
<div class="dialog message">
|
|
There are no library EANs.
|
|
</div>
|
|
[% END %]
|
|
[% END %]
|
|
|
|
</div>
|
|
</div>
|
|
<div class="yui-b">
|
|
[% INCLUDE 'admin-menu.inc' %]
|
|
</div>
|
|
</div>
|
|
[% INCLUDE 'intranet-bottom.inc' %]
|