http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7458
--- Comment #18 from Brendan Gallagher <[email protected]> 2012-02-07 18:14:37 UTC --- (In reply to comment #17) > The current patch implements both the plugin (which is in the scope of this > bug > report) AND a database change to add an index on items.itemcallnumber (out of > scope). I do not object to either change; even if the new plugin isn't used > by > anyone, it can safely sit in the plugins directory without harming anything. > The index on items.itemcallnumber may well provide useful, as I'm sure there > are other places in Koha that perform SQL WHERE clauses on itemcallnumber > (C4::Search::SearchAcquisitions() for example). > > What I object to is the two enhancements being bundled together. This new > plugin should exist entirely on it's own, in it's own patch. The new > itemcallnumber index should be it's own change, on it's own bug report, tested > independently. If this plugin does indeed dependent on this index, a > dependency between the two bug reports should be added. > > Moving to "In Discussion" to solicit further opinions on this. Great Makes Sense... -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are watching all bug changes. _______________________________________________ Koha-bugs mailing list [email protected] http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
