Bug 21706: Make authority framework subfield links work (TT filter)
This patch removes URL building from the controller and does it on the templates, using the right 'uri' filter for the query parameters. To test: - Go to Home > Administration > Athority types - On any of the listed frameworks, pick the 'MARC structure' action in the dropdown - On any subfield, hover on the actions dropdown => FAIL: The URLs are wrongly formed - Click on the links => FAIL: You get errors because of the bad URL - Apply this patch - Reload the 'MARC structure' page - On any subfield, hover on the actions dropdown => SUCCESS: LInks look correct - Click on any of them => SUCCESS: They work! - Sign off :-D Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> Signed-off-by: Josef Moravec <josef.moravec@gmail.com> Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
This commit is contained in:
parent
2fc599c089
commit
bea8edd972
|
|
@ -213,9 +213,9 @@
|
|||
<a class="btn btn-default btn-xs dropdown-toggle" id="authtagactions[% loo.tagfield | html %]" role="button" data-toggle="dropdown" href="#">
|
||||
Actions <b class="caret"></b></a>
|
||||
<ul class="dropdown-menu pull-right" role="menu" aria-labelledby="authtagactions[% loo.tagfield | html %]">
|
||||
<li><a href="[% loo.subfield_link | url %]"><i class="fa fa-eye"></i> Subfields</a></li>
|
||||
<li><a href="[% loo.edit | url %]"><i class="fa fa-pencil"></i> Edit</a></li>
|
||||
<li><a href="[% loo.delete | url %]"><i class="fa fa-trash"></i> Delete</a></li>
|
||||
<li><a href="auth_subfields_structure.pl?tagfield=[% loo.tagfield | uri %]&authtypecode=[% authtypecode | uri %]"><i class="fa fa-eye"></i> Subfields</a></li>
|
||||
<li><a href="?op=add_form&searchfield=[% loo.tagfield | uri %]&authtypecode=[% authtypecode | uri %]"><i class="fa fa-pencil"></i> Edit</a></li>
|
||||
<li><a href="?op=delete_confirm&searchfield=[% loo.tagfield | uri %]&authtypecode=[% authtypecode | uri %]"><i class="fa fa-trash"></i> Delete</a></li>
|
||||
</ul>
|
||||
</div>
|
||||
</td>
|
||||
|
|
|
|||
Loading…
Reference in New Issue