Bug 5063 - Improve C4::Booksellers interface
Summary: Improve C4::Booksellers interface
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Acquisitions (show other bugs)
Version: rel_3_2
Hardware: All All
: PATCH-Sent (DO NOT USE) enhancement (vote)
Assignee: Henri-Damien LAURENT
QA Contact: Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-07-26 15:50 UTC by Colin Campbell
Modified: 2012-10-25 23:03 UTC (History)
3 users (show)

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


Attachments
Proposed patch (16.06 KB, patch)
2011-03-08 13:59 UTC, Colin Campbell
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Colin Campbell 2010-07-26 15:50:40 UTC
Merge changes to C4::Booksellers from UNfao system. These changes were done as part of enabling acquisitions on a system with a large number of vendors. The only visible interface difference is that wgen a number of vendors are returned they are sorted by name

Enables warnings in Booksellers.pm 
No longer exports all routines by default
When retrieving multiple records sort the resulting array so that user can navigate resulting list
Comment 1 Colin Campbell 2010-07-26 15:58:28 UTC
NB apart from the fact that the result of a vendor search is now sorted. There are no end user visible changes in this commit. It does remove a lot id warnings from the error log though
Comment 2 Nicole C. Engard 2011-03-07 18:38:47 UTC
This is marked as 'needs signoff' but I don't see a patch to sign off on. Where would one find the patch?
Comment 3 Colin Campbell 2011-03-08 13:59:22 UTC
Created attachment 3264 [details] [review]
Proposed patch

A while since this patch was originally posted so have attached it to bug report
Comment 4 Chris Cormack 2011-03-11 19:43:53 UTC
Pushed please test