Bug 12704 - Remove CGI::scrolling_list from unimarc_field_225a.pl
Summary: Remove CGI::scrolling_list from unimarc_field_225a.pl
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Templates (show other bugs)
Version: master
Hardware: All All
: P5 - low normal (vote)
Assignee: Bernardo Gonzalez Kriegel
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks: 766
  Show dependency treegraph
 
Reported: 2014-08-03 00:39 UTC by Bernardo Gonzalez Kriegel
Modified: 2015-06-04 23:33 UTC (History)
1 user (show)

See Also:
Change sponsored?: ---
Patch complexity: Small patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
Bug 12704: Remove CGI::scrolling_list from unimarc_field_225a.pl (2.97 KB, patch)
2014-08-03 00:58 UTC, Bernardo Gonzalez Kriegel
Details | Diff | Splinter Review
Bug 12704: Remove CGI::scrolling_list from unimarc_field_225a.pl (3.03 KB, patch)
2014-08-08 03:00 UTC, Chris Cormack
Details | Diff | Splinter Review
[PASSED QA] Bug 12704: Remove CGI::scrolling_list from unimarc_field_225a.pl (3.16 KB, patch)
2014-08-17 10:07 UTC, Katrin Fischer
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Bernardo Gonzalez Kriegel 2014-08-03 00:39:39 UTC
One in this file
Comment 1 Bernardo Gonzalez Kriegel 2014-08-03 00:58:49 UTC Comment hidden (obsolete)
Comment 2 Chris Cormack 2014-08-08 03:00:00 UTC Comment hidden (obsolete)
Comment 3 Katrin Fischer 2014-08-17 10:07:44 UTC
Created attachment 30894 [details] [review]
[PASSED QA] Bug 12704: Remove CGI::scrolling_list from unimarc_field_225a.pl

This patch removes the only instance in this file

To test:
1. Apply the patch
2. If you have UNIMARC, search for a record, edit it,
and go to tag 225
If you don't, edit your framework and connect this
pluging to some tag/subtag
3. Clic on the plugin link. If you don't have any record
that correspond to the search, can remove the commented
line to have something to play
4. The collection pulldown was replaced, check that works,
select a value and press Ok

Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Passes tests and QA script, tested according to test plan.
Comment 4 Tomás Cohen Arazi 2014-08-19 15:54:23 UTC
Patch pushed to master.

Thanks Bernardo!