https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23991
Martin Renvoize <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #134922|0 |1 is obsolete| | --- Comment #56 from Martin Renvoize <[email protected]> --- Created attachment 136441 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=136441&action=edit Bug 23991: Move SearchSuggestion to Koha::Suggestions The C4::Suggestions::SearchSuggestion subroutine is badly written and can be replaced by calls to Koha::Suggestions->search. The hard part in this patch is suggestion.pl, the other occurrences have been replaced easily. Test plan: The idea is to test the whole suggestion workflow. 1. Create a suggestion on OPAC 2. Create a suggestion on the staff interface 3. Edit suggestions 4. Filter suggestions (use the different filters and "organize by" values) Signed-off-by: Tomas Cohen Arazi <[email protected]> Signed-off-by: Nick Clemens <[email protected]> Bug 23991: Remove SearchSuggestion tests Signed-off-by: Tomas Cohen Arazi <[email protected]> Signed-off-by: Nick Clemens <[email protected]> Bug 23991: (QA follow-up) Save some DB queries This patch makes the suggestion-related pages rely on array size instead of querying the DB each time they need to. In the case of suggestion/suggestion.pl it goes from 4 COUNT(*) to 1. To test, with KTD: 1. Run on the host machine: $ docker exec -ti koha_db_1 bash $ mysql -ppassword > SET GLOBAL general_log_file='/var/log/mysql/mycustom.log'; > SET GLOBAL log_output = 'FILE'; > SET GLOBAL general_log = 'ON'; > \q $ tail -f /var/log/mysql/mycustom.log | grep suggestions 2. Visit the different pages changed on this bug => SUCCESS: Some queries 3. Apply this patch 4. Repeat 2 => SUCCESS: Less queries! 5. Sign off :-D Signed-off-by: Tomas Cohen Arazi <[email protected]> Signed-off-by: Nick Clemens <[email protected]> Bug 23991: Fix branchcode and budgetid filtering Signed-off-by: Nick Clemens <[email protected]> Bug 23991: Fix conflict with bug 28941 Well, this patchset fixed the security bug... Redoing on top of bug 28941 Signed-off-by: Nick Clemens <[email protected]> Bug 23991: (follow-up) Missing semicolon Signed-off-by: Nick Clemens <[email protected]> Bug 23991: Fix 'all' libraries Signed-off-by: Nick Clemens <[email protected]> Bug 23991: (follow-up) Add value to filter_archived Signed-off-by: Martin Renvoize <[email protected]> -- You are receiving this mail because: You are watching all bug changes. _______________________________________________ Koha-bugs mailing list [email protected] https://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/
