Bug 15167 - Remove unused script serials/acqui-search-result.pl
Summary: Remove unused script serials/acqui-search-result.pl
Status: CLOSED INVALID
Alias: None
Product: Koha
Classification: Unclassified
Component: Architecture, internals, and plumbing (show other bugs)
Version: Main
Hardware: All All
: P5 - low enhancement (vote)
Assignee: Jonathan Druart
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-11-11 08:42 UTC by Jonathan Druart
Modified: 2016-06-21 21:36 UTC (History)
1 user (show)

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


Attachments
Bug 15167; Remove unused script serials/acqui-search-result.pl (5.05 KB, patch)
2015-11-11 08:44 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 15167; Remove unused script serials/acqui-search-result.pl (5.24 KB, patch)
2015-11-11 10:26 UTC, Marc Véron
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Jonathan Druart 2015-11-11 08:42:57 UTC
This script does not seem to be used. It has not been updated usefully since 2006.
Comment 1 Jonathan Druart 2015-11-11 08:44:24 UTC Comment hidden (obsolete)
Comment 2 Marc Véron 2015-11-11 10:26:33 UTC
Created attachment 44732 [details] [review]
Bug 15167; Remove unused script serials/acqui-search-result.pl

This script does not seem to be used. It has not been updated usefully
since 2006.

Test plan:
  git grep acqui-search-result
should not return any result.

Searched for file acqui-search-result*.*, vanished.
Signed-off-by: Marc Véron <veron@veron.ch>
Comment 3 Marc Véron 2015-11-11 15:48:30 UTC
Oh, it is used...
Sorry, I missed that while signing-off.

To verify:
- Home > Serials > New subscription
- Click on "Search for a vendor"
- You get a pop up window (...cgi-bin/koha/serials/acqui-search.pl)
- Enter a search term, click OK
=> result 404 page (...cgi-bin/koha/serials/acqui-search-result.pl)

It is used in ...serials/acqui-search.tt
Comment 4 Jonathan Druart 2015-11-11 16:28:05 UTC
How can I missed that??