[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 Martin Renvoize changed: What|Removed |Added Blocks||21835 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21835 [Bug 21835] Request ID is displayed as NaN -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 Martin Renvoize changed: What|Removed |Added Resolution|--- |FIXED Status|Pushed to Master|RESOLVED --- Comment #78 from Martin Renvoize --- Enhancement, will not be backported to 18.05.x series. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 --- Comment #77 from Martin Renvoize --- Odd.. I didn't get that warning at all when I tested the patch. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 --- Comment #76 from Nick Clemens --- Last two patches pushed to master for 18.11 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 --- Comment #75 from Jonathan Druart --- Created attachment 81998 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=81998=edit Bug 20996: Remove warning 'Un-mocked method' t/db_dependent/api/v1/illrequests.t .. Un-mocked method 'status_graph()' called at /home/vagrant/kohaclone/Koha/Illrequest.pm line 439. Signed-off-by: Jonathan Druart -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 Jonathan Druart changed: What|Removed |Added Attachment #81996|0 |1 is obsolete|| --- Comment #74 from Jonathan Druart --- Created attachment 81997 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=81997=edit Bug 20996: (RM follow-up) Fix the api tests Signed-off-by: Jonathan Druart -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 --- Comment #73 from Jonathan Druart --- I get a warning with the last patch: t/db_dependent/api/v1/illrequests.t .. Un-mocked method 'status_graph()' called at /home/vagrant/kohaclone/Koha/Illrequest.pm line 439. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 Martin Renvoize changed: What|Removed |Added Attachment #81994|0 |1 is obsolete|| --- Comment #72 from Martin Renvoize --- Created attachment 81996 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=81996=edit Bug 20996: (RM follow-up) Fix the api tests https://bugs.koha-community.org/show_bug.cgi?id=21766 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 --- Comment #71 from Martin Renvoize --- Added a follow-up for the RM that should fix the last remaining failing test. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 --- Comment #70 from Martin Renvoize --- Created attachment 81994 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=81994=edit Bug 20996: (RM follow-up) Fix the api tests -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 --- Comment #69 from Nick Clemens --- Awesome work all! Pushed to master for 18.11(In reply to Tomás Cohen Arazi from comment #68) > Created attachment 81967 [details] [review] > Bug 20996: (follow-up) Fix test > > - Fixes the t/db_dependent/Illrequest/Config.t > - Adds tests for Koha::Illrequest::Config->has_branch > > 0) Do not apply the path > 1) Run test t/db_dependent/Illrequest/Config.t > 2) It fails - it is because the Koha/Illrequest/Config.pm prefixes > handling changed > 3) Apply the patch > 4) Run test again, it should pass now > 5) Read through the test code and confirm it makes sense > > Signed-off-by: Tomas Cohen Arazi Followup pushed to master for 18.11 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 Tomás Cohen Arazi changed: What|Removed |Added Attachment #81928|0 |1 is obsolete|| --- Comment #68 from Tomás Cohen Arazi --- Created attachment 81967 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=81967=edit Bug 20996: (follow-up) Fix test - Fixes the t/db_dependent/Illrequest/Config.t - Adds tests for Koha::Illrequest::Config->has_branch 0) Do not apply the path 1) Run test t/db_dependent/Illrequest/Config.t 2) It fails - it is because the Koha/Illrequest/Config.pm prefixes handling changed 3) Apply the patch 4) Run test again, it should pass now 5) Read through the test code and confirm it makes sense Signed-off-by: Tomas Cohen Arazi -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 --- Comment #67 from Josef Moravec --- (In reply to Jonathan Druart from comment #63) > t/db_dependent/Illrequest/Config.t is failing. > > New code added to Koha/Illrequest/Config.pm but no tests? I added a follow-up patch with test changes -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 --- Comment #66 from Josef Moravec --- Created attachment 81928 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=81928=edit Bug 20996: (follow-up) Fix test - Fixes the t/db_dependent/Illrequest/Config.t - Adds tests for Koha::Illrequest::Config->has_branch 0) Do not apply the path 1) Run test t/db_dependent/Illrequest/Config.t 2) It fails - it is because the Koha/Illrequest/Config.pm prefixes handling changed 3) Apply the patch 4) Run test again, it should pass now 5) Read through the test code and confirm it makes sense -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 Martin Renvoize changed: What|Removed |Added Attachment #81908|0 |1 is obsolete|| -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 Martin Renvoize changed: What|Removed |Added Attachment #81909|0 |1 is obsolete|| -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 --- Comment #65 from Martin Renvoize --- Created attachment 81909 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=81909=edit Bug 20966: (follow-up) Add template variable filter; Fix table export This patch makes to corrections: 1. The "$raw" filter has been added to the column configuration 2. The columns configuration now specifies which columns are exported in Copy, Excel, etc. operations. To test, apply the patch and test the various table export buttons on a both a closed and open basket. Confirm that the 'Modify' and 'Cancel' columns are not exported. Signed-off-by: Katrin Fischer Signed-off-by: Martin Renvoize -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 --- Comment #64 from Martin Renvoize --- Created attachment 81908 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=81908=edit Bug 20966: Add column configuration to table of orders in a basket This patch adds column configuration to the table of orders in a basket. It replaces the "Show all details" checkbox which previously was used to show the hidden "tax included" columns. Now those columns are hidden by default in the columns configuration file. To test, apply the patch and view a basket with multiple orders. Test all DataTables functionality (paging, sorting, filtering) including showing and hiding columns. Test with both an open and a closed basket. Signed-off-by: Claire Gravely Signed-off-by: Katrin Fischer Signed-off-by: Martin Renvoize -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 Jonathan Druart changed: What|Removed |Added CC||jonathan.dru...@bugs.koha-c ||ommunity.org --- Comment #63 from Jonathan Druart --- t/db_dependent/Illrequest/Config.t is failing. New code added to Koha/Illrequest/Config.pm but no tests? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 Nick Clemens changed: What|Removed |Added CC||n...@bywatersolutions.com Status|Passed QA |Pushed to Master --- Comment #62 from Nick Clemens --- Awesome work all! Pushed to master for 18.11 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 Martin Renvoize changed: What|Removed |Added Depends on|7317| Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7317 [Bug 7317] Add an Interlibrary Loan Module to Circulation and OPAC -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 Josef Moravec changed: What|Removed |Added QA Contact||josef.mora...@gmail.com -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 --- Comment #58 from Josef Moravec --- Created attachment 81758 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=81758=edit Bug 20996: (follow-up) Add to conf tpl Since we now require the block, we should add it to the config templates Signed-off-by: Magnus Enger Signed-off-by: Josef Moravec -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 --- Comment #60 from Josef Moravec --- Created attachment 81760 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=81760=edit Bug 20996: (follow-up) Fix merge problems Merge conflicts resolved Signed-off-by: Josef Moravec -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 Josef Moravec changed: What|Removed |Added Attachment #81747|0 |1 is obsolete|| Attachment #81748|0 |1 is obsolete|| Attachment #81749|0 |1 is obsolete|| Attachment #81750|0 |1 is obsolete|| Attachment #81751|0 |1 is obsolete|| Attachment #81752|0 |1 is obsolete|| Attachment #81753|0 |1 is obsolete|| --- Comment #55 from Josef Moravec --- Created attachment 81755 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=81755=edit Bug 20996: Remove prefix use of borrower category This patch removes the potential use of borrower category as a ILL request ID prefix. It makes no sense. We provide the ability for a site to define a request prefix based on branch, there is no use case for using the borrower category. Add to this that the borrower for every request was being retrieved in order to get the category, it's a huge performance hit also. We also now require the block in the block and complain if it's not present. The request prefix should be defined in this block. This patch also improves the performance of the API request that returns all requests, optionally including additional data. It also deprecates the overloaded TO_JSON method and moves the request augmentation code into the API route's controller. It may be that we want to shift it out of there at some point, but it is fine where it is for now. Signed-off-by: Magnus Enger Signed-off-by: Josef Moravec -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 --- Comment #57 from Josef Moravec --- Created attachment 81757 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=81757=edit Bug 20996: Further enhance performance We were immediately unblessing all request objects after retrieval then, at a later stage, re-instantiating each one so we could call methods on that. This was a very bad idea. We now preserve the objects for as long as we can. Signed-off-by: Magnus Enger Signed-off-by: Josef Moravec -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 --- Comment #56 from Josef Moravec --- Created attachment 81756 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=81756=edit Bug 20996: Fix unit tests - Remove unit tests relating to TO_JSON - Add tests for new explicit embedding - Modify tests relating to now deprecated brw_cat Signed-off-by: Magnus Enger Signed-off-by: Josef Moravec -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 --- Comment #59 from Josef Moravec --- Created attachment 81759 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=81759=edit Bug 20996: (follow-up) Fix column name expectation This bug changes how request metadata is obtained. Previously the appropriate backend's metadata function would return it. For at least the FreeForm backend, the metadata property names we title cased. The datatable JS was expecting this and breaks with the new metadata getting which is performed in the API controller, which just returns the property names as they appear in the DB. This patch changes the expectation of the JS to match what's coming back from the API Signed-off-by: Magnus Enger Tested by repeatedly loading the API-call http://intranet/api/v1/illrequests?embed=metadata,patron,capabilities,library in Chromium and watching the reported time to load the URL. There is a siginificant decrease in the time with the patches compared to the time it takes without the patches. I had some concerns along the way, but they have all been fixed. Signed-off-by: Magnus Enger Tested by repeatedly loading the API-call http://intranet/api/v1/illrequests?embed=metadata,patron,capabilities,library in Chromium and watching the reported time to load the URL. There is a siginificant decrease in the time with the patches compared to the time it takes without the patches. I had some concerns along the way, but they have all been fixed. Signed-off-by: Josef Moravec -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 --- Comment #61 from Josef Moravec --- Created attachment 81761 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=81761=edit Bug 20996: (QA follow-up) Fix pod Signed-off-by: Josef Moravec -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 Josef Moravec changed: What|Removed |Added Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 --- Comment #54 from Andrew Isherwood --- Rebased, fixed merge problems (including the "comments" embed in the API response) and added Josef's POD fix. Ready for QA again -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 Andrew Isherwood changed: What|Removed |Added Attachment #79592|0 |1 is obsolete|| --- Comment #51 from Andrew Isherwood --- Created attachment 81751 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=81751=edit Bug 20996: (follow-up) Fix column name expectation This bug changes how request metadata is obtained. Previously the appropriate backend's metadata function would return it. For at least the FreeForm backend, the metadata property names we title cased. The datatable JS was expecting this and breaks with the new metadata getting which is performed in the API controller, which just returns the property names as they appear in the DB. This patch changes the expectation of the JS to match what's coming back from the API Signed-off-by: Magnus Enger Tested by repeatedly loading the API-call http://intranet/api/v1/illrequests?embed=metadata,patron,capabilities,library in Chromium and watching the reported time to load the URL. There is a siginificant decrease in the time with the patches compared to the time it takes without the patches. I had some concerns along the way, but they have all been fixed. Signed-off-by: Magnus Enger Tested by repeatedly loading the API-call http://intranet/api/v1/illrequests?embed=metadata,patron,capabilities,library in Chromium and watching the reported time to load the URL. There is a siginificant decrease in the time with the patches compared to the time it takes without the patches. I had some concerns along the way, but they have all been fixed. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 --- Comment #52 from Andrew Isherwood --- Created attachment 81752 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=81752=edit Bug 20996: (follow-up) Fix merge problems Merge conflicts resolved -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 --- Comment #53 from Andrew Isherwood --- Created attachment 81753 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=81753=edit Bug 20996: (QA follow-up) Fix pod -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 Andrew Isherwood changed: What|Removed |Added Attachment #79590|0 |1 is obsolete|| --- Comment #49 from Andrew Isherwood --- Created attachment 81749 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=81749=edit Bug 20996: Further enhance performance We were immediately unblessing all request objects after retrieval then, at a later stage, re-instantiating each one so we could call methods on that. This was a very bad idea. We now preserve the objects for as long as we can. Signed-off-by: Magnus Enger -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 Andrew Isherwood changed: What|Removed |Added Attachment #79591|0 |1 is obsolete|| --- Comment #50 from Andrew Isherwood --- Created attachment 81750 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=81750=edit Bug 20996: (follow-up) Add to conf tpl Since we now require the block, we should add it to the config templates Signed-off-by: Magnus Enger -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 Andrew Isherwood changed: What|Removed |Added Attachment #79589|0 |1 is obsolete|| --- Comment #48 from Andrew Isherwood --- Created attachment 81748 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=81748=edit Bug 20996: Fix unit tests - Remove unit tests relating to TO_JSON - Add tests for new explicit embedding - Modify tests relating to now deprecated brw_cat Signed-off-by: Magnus Enger -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 Andrew Isherwood changed: What|Removed |Added Attachment #79588|0 |1 is obsolete|| --- Comment #47 from Andrew Isherwood --- Created attachment 81747 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=81747=edit Bug 20996: Remove prefix use of borrower category This patch removes the potential use of borrower category as a ILL request ID prefix. It makes no sense. We provide the ability for a site to define a request prefix based on branch, there is no use case for using the borrower category. Add to this that the borrower for every request was being retrieved in order to get the category, it's a huge performance hit also. We also now require the block in the block and complain if it's not present. The request prefix should be defined in this block. This patch also improves the performance of the API request that returns all requests, optionally including additional data. It also deprecates the overloaded TO_JSON method and moves the request augmentation code into the API route's controller. It may be that we want to shift it out of there at some point, but it is fine where it is for now. Signed-off-by: Magnus Enger -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 Andrew Isherwood changed: What|Removed |Added Attachment #79592|1 |0 is obsolete|| -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 Andrew Isherwood changed: What|Removed |Added Attachment #79590|1 |0 is obsolete|| -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 Andrew Isherwood changed: What|Removed |Added Attachment #79588|1 |0 is obsolete|| -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 Andrew Isherwood changed: What|Removed |Added Attachment #79589|1 |0 is obsolete|| -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 Andrew Isherwood changed: What|Removed |Added Attachment #79591|1 |0 is obsolete|| -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 Andrew Isherwood changed: What|Removed |Added Attachment #81720|0 |1 is obsolete|| --- Comment #46 from Andrew Isherwood --- Comment on attachment 81720 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=81720 Bug 20996: (QA follow-up) Fix pod One of these patches in the rebased set is failing with "error: sha1 information is lacking or useless (Koha/Illrequest.pm)", so I'm obsoleting them all, un-obsoleting their predecessors, then going to rebase and re-attach patches (including josef's POD fix) -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 Andrew Isherwood changed: What|Removed |Added Attachment #81716|0 |1 is obsolete|| --- Comment #42 from Andrew Isherwood --- Comment on attachment 81716 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=81716 Bug 20996: Fix unit tests One of these patches in the rebased set is failing with "error: sha1 information is lacking or useless (Koha/Illrequest.pm)", so I'm obsoleting them all, un-obsoleting their predecessors, then going to rebase and re-attach patches (including josef's POD fix) -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 Andrew Isherwood changed: What|Removed |Added Attachment #81719|0 |1 is obsolete|| --- Comment #45 from Andrew Isherwood --- Comment on attachment 81719 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=81719 Bug 20996: (follow-up) Fix column name expectation One of these patches in the rebased set is failing with "error: sha1 information is lacking or useless (Koha/Illrequest.pm)", so I'm obsoleting them all, un-obsoleting their predecessors, then going to rebase and re-attach patches (including josef's POD fix) -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 Andrew Isherwood changed: What|Removed |Added Attachment #81717|0 |1 is obsolete|| --- Comment #43 from Andrew Isherwood --- Comment on attachment 81717 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=81717 Bug 20996: Further enhance performance One of these patches in the rebased set is failing with "error: sha1 information is lacking or useless (Koha/Illrequest.pm)", so I'm obsoleting them all, un-obsoleting their predecessors, then going to rebase and re-attach patches (including josef's POD fix) -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 Andrew Isherwood changed: What|Removed |Added Attachment #81718|0 |1 is obsolete|| --- Comment #44 from Andrew Isherwood --- Comment on attachment 81718 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=81718 Bug 20996: (follow-up) Add to conf tpl One of these patches in the rebased set is failing with "error: sha1 information is lacking or useless (Koha/Illrequest.pm)", so I'm obsoleting them all, un-obsoleting their predecessors, then going to rebase and re-attach patches (including josef's POD fix) -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 Andrew Isherwood changed: What|Removed |Added Attachment #81715|0 |1 is obsolete|| --- Comment #41 from Andrew Isherwood --- Comment on attachment 81715 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=81715 Bug 20996: Remove prefix use of borrower category One of these patches in the rebased set is failing with "error: sha1 information is lacking or useless (Koha/Illrequest.pm)", so I'm obsoleting them all, un-obsoleting their predecessors, then going to rebase and re-attach patches (including josef's POD fix) -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 --- Comment #40 from Josef Moravec --- Andrew is going to add comments to embed as it was done in bug 18591 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 --- Comment #39 from Josef Moravec --- Rebased on master and add small follow-up -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 --- Comment #38 from Josef Moravec --- Created attachment 81720 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=81720=edit Bug 20996: (QA follow-up) Fix pod -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 --- Comment #35 from Josef Moravec --- Created attachment 81717 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=81717=edit Bug 20996: Further enhance performance We were immediately unblessing all request objects after retrieval then, at a later stage, re-instantiating each one so we could call methods on that. This was a very bad idea. We now preserve the objects for as long as we can. Signed-off-by: Magnus Enger -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 --- Comment #37 from Josef Moravec --- Created attachment 81719 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=81719=edit Bug 20996: (follow-up) Fix column name expectation This bug changes how request metadata is obtained. Previously the appropriate backend's metadata function would return it. For at least the FreeForm backend, the metadata property names we title cased. The datatable JS was expecting this and breaks with the new metadata getting which is performed in the API controller, which just returns the property names as they appear in the DB. This patch changes the expectation of the JS to match what's coming back from the API Signed-off-by: Magnus Enger Tested by repeatedly loading the API-call http://intranet/api/v1/illrequests?embed=metadata,patron,capabilities,library in Chromium and watching the reported time to load the URL. There is a siginificant decrease in the time with the patches compared to the time it takes without the patches. I had some concerns along the way, but they have all been fixed. Signed-off-by: Magnus Enger Tested by repeatedly loading the API-call http://intranet/api/v1/illrequests?embed=metadata,patron,capabilities,library in Chromium and watching the reported time to load the URL. There is a siginificant decrease in the time with the patches compared to the time it takes without the patches. I had some concerns along the way, but they have all been fixed. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 --- Comment #34 from Josef Moravec --- Created attachment 81716 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=81716=edit Bug 20996: Fix unit tests - Remove unit tests relating to TO_JSON - Add tests for new explicit embedding - Modify tests relating to now deprecated brw_cat Signed-off-by: Magnus Enger -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 --- Comment #36 from Josef Moravec --- Created attachment 81718 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=81718=edit Bug 20996: (follow-up) Add to conf tpl Since we now require the block, we should add it to the config templates Signed-off-by: Magnus Enger -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 Josef Moravec changed: What|Removed |Added Attachment #79588|0 |1 is obsolete|| Attachment #79589|0 |1 is obsolete|| Attachment #79590|0 |1 is obsolete|| Attachment #79591|0 |1 is obsolete|| Attachment #79592|0 |1 is obsolete|| --- Comment #33 from Josef Moravec --- Created attachment 81715 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=81715=edit Bug 20996: Remove prefix use of borrower category This patch removes the potential use of borrower category as a ILL request ID prefix. It makes no sense. We provide the ability for a site to define a request prefix based on branch, there is no use case for using the borrower category. Add to this that the borrower for every request was being retrieved in order to get the category, it's a huge performance hit also. We also now require the block in the block and complain if it's not present. The request prefix should be defined in this block. This patch also improves the performance of the API request that returns all requests, optionally including additional data. It also deprecates the overloaded TO_JSON method and moves the request augmentation code into the API route's controller. It may be that we want to shift it out of there at some point, but it is fine where it is for now. Signed-off-by: Magnus Enger -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 Bug 20996 depends on bug 20548, which changed state. Bug 20548 Summary: Remove copyright clearance workflow from staff created ILL requests https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20548 What|Removed |Added Status|Pushed to Stable|RESOLVED Resolution|--- |FIXED -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 Martin Renvoize changed: What|Removed |Added CC||martin.renvoize@ptfs-europe ||.com -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 Andrew Isherwood changed: What|Removed |Added Blocks|18589 | Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18589 [Bug 18589] Show ILLs as part of patron profile -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 Andrew Isherwood changed: What|Removed |Added Depends on||20548 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20548 [Bug 20548] Remove copyright clearance workflow from staff created ILL requests -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 Andrew Isherwood changed: What|Removed |Added Text to go in the||This patch makes the release notes|| configuration ||section compulsory in the || ||section of your Koha ||configuration file. The || section allows you ||define per-branch ||Interlibrary loan options ||for each branch. In it's ||most basic form, the branch ||section can be: ||code_for_your ||_branch -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 Andrew Isherwood changed: What|Removed |Added Blocks||21079 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21079 [Bug 21079] Unify metadata schema across backends -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 Magnus Enger changed: What|Removed |Added Attachment #79586|0 |1 is obsolete|| --- Comment #31 from Magnus Enger --- Created attachment 79591 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=79591=edit Bug 20996: (follow-up) Add to conf tpl Since we now require the block, we should add it to the config templates Signed-off-by: Magnus Enger -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 Magnus Enger changed: What|Removed |Added Attachment #79587|0 |1 is obsolete|| --- Comment #32 from Magnus Enger --- Created attachment 79592 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=79592=edit Bug 20996: (follow-up) Fix column name expectation This bug changes how request metadata is obtained. Previously the appropriate backend's metadata function would return it. For at least the FreeForm backend, the metadata property names we title cased. The datatable JS was expecting this and breaks with the new metadata getting which is performed in the API controller, which just returns the property names as they appear in the DB. This patch changes the expectation of the JS to match what's coming back from the API Signed-off-by: Magnus Enger Tested by repeatedly loading the API-call http://intranet/api/v1/illrequests?embed=metadata,patron,capabilities,library in Chromium and watching the reported time to load the URL. There is a siginificant decrease in the time with the patches compared to the time it takes without the patches. I had some concerns along the way, but they have all been fixed. Signed-off-by: Magnus Enger Tested by repeatedly loading the API-call http://intranet/api/v1/illrequests?embed=metadata,patron,capabilities,library in Chromium and watching the reported time to load the URL. There is a siginificant decrease in the time with the patches compared to the time it takes without the patches. I had some concerns along the way, but they have all been fixed. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 Magnus Enger changed: What|Removed |Added Attachment #79379|0 |1 is obsolete|| --- Comment #26 from Magnus Enger --- Created attachment 79586 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=79586=edit Bug 20996: (follow-up) Add to conf tpl Since we now require the block, we should add it to the config templates Signed-off-by: Magnus Enger https://bugs.koha-community.org/show_bug.cgi?id=20797 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 Magnus Enger changed: What|Removed |Added Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 Magnus Enger changed: What|Removed |Added Attachment #76688|0 |1 is obsolete|| --- Comment #24 from Magnus Enger --- Created attachment 79584 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=79584=edit Bug 20996: Fix unit tests - Remove unit tests relating to TO_JSON - Add tests for new explicit embedding - Modify tests relating to now deprecated brw_cat Signed-off-by: Magnus Enger https://bugs.koha-community.org/show_bug.cgi?id=20797 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 Magnus Enger changed: What|Removed |Added Attachment #76705|0 |1 is obsolete|| --- Comment #25 from Magnus Enger --- Created attachment 79585 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=79585=edit Bug 20996: Further enhance performance We were immediately unblessing all request objects after retrieval then, at a later stage, re-instantiating each one so we could call methods on that. This was a very bad idea. We now preserve the objects for as long as we can. Signed-off-by: Magnus Enger https://bugs.koha-community.org/show_bug.cgi?id=20797 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 Magnus Enger changed: What|Removed |Added Attachment #79576|0 |1 is obsolete|| --- Comment #27 from Magnus Enger --- Created attachment 79587 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=79587=edit Bug 20996: (follow-up) Fix column name expectation This bug changes how request metadata is obtained. Previously the appropriate backend's metadata function would return it. For at least the FreeForm backend, the metadata property names we title cased. The datatable JS was expecting this and breaks with the new metadata getting which is performed in the API controller, which just returns the property names as they appear in the DB. This patch changes the expectation of the JS to match what's coming back from the API Signed-off-by: Magnus Enger Tested by repeatedly loading the API-call http://intranet/api/v1/illrequests?embed=metadata,patron,capabilities,library in Chromium and watching the reported time to load the URL. There is a siginificant decrease in the time with the patches compared to the time it takes without the patches. I had some concerns along the way, but they have all been fixed. https://bugs.koha-community.org/show_bug.cgi?id=20797 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 Magnus Enger changed: What|Removed |Added Attachment #76687|0 |1 is obsolete|| --- Comment #23 from Magnus Enger --- Created attachment 79583 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=79583=edit Bug 20996: Remove prefix use of borrower category This patch removes the potential use of borrower category as a ILL request ID prefix. It makes no sense. We provide the ability for a site to define a request prefix based on branch, there is no use case for using the borrower category. Add to this that the borrower for every request was being retrieved in order to get the category, it's a huge performance hit also. We also now require the block in the block and complain if it's not present. The request prefix should be defined in this block. This patch also improves the performance of the API request that returns all requests, optionally including additional data. It also deprecates the overloaded TO_JSON method and moves the request augmentation code into the API route's controller. It may be that we want to shift it out of there at some point, but it is fine where it is for now. Signed-off-by: Magnus Enger https://bugs.koha-community.org/show_bug.cgi?id=20797 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 --- Comment #22 from Andrew Isherwood --- Created attachment 79576 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=79576=edit Bug 20996: (follow-up) Fix column name expectation This bug changes how request metadata is obtained. Previously the appropriate backend's metadata function would return it. For at least the FreeForm backend, the metadata property names we title cased. The datatable JS was expecting this and breaks with the new metadata getting which is performed in the API controller, which just returns the property names as they appear in the DB. This patch changes the expectation of the JS to match what's coming back from the API -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 Andrew Isherwood changed: What|Removed |Added Status|Failed QA |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 --- Comment #21 from Andrew Isherwood --- OK, I know what it is. In the FreeForm backend's metadata method (and, I presume, your backend's metadata method), there is this line: https://github.com/PTFS-Europe/koha-ill-freeform/blob/master/Base.pm#L124 Which, as you can see, title cases the name of the property (Author etc), which breaks the JS when we use the alternative method of getting our metadata, which doesn't title case it. I've no idea why the properties are title cased, but it seems to have been the case since day 1. I'm going to fix it by changing what the JS expects from "metadata_Author" to "metadata_author", same for title, and leave it at that. This whole issue is made irrelevant by bug 21063 which changes how the datatable JS works anyway and has no expectation of column names from the API. I'll attach a patch momentarily. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 --- Comment #20 from Andrew Isherwood --- No, it's not backend dependent. I looked into it yesterday, prior to these patches "title" and "author" metadata were returned by the API as "metadata_Title" & "metadata_Author", which is what the JS expects, hence no error. With the patches, they're returned as "metadata_title" & "metadata_author". I didn't get to the bottom of why, but now I write it out, I think I might know why. I suspect the fix is going to be to change what the JS expects, since the API is just returning illrequestattribute names as they appear in the database, which is ultimately correct. I'm going to fix this now, I'm not going to rest until it's done damn it! :) -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 Magnus Enger changed: What|Removed |Added Status|Needs Signoff |Failed QA --- Comment #19 from Magnus Enger --- Not quite that simple, I'm afraid... When I am on the master branch everything looks fine. Title and Author are displayed for the requests that have them. When I apply the patches from this bug title and author are not displayed, and this warning pops up, both on initial rendering of the "View ILL requests" page and whenever I use the Next and Previous links in the datatable to see more requests. DataTables warning: table id=ill-requests - Requested unknown parameter 'metadata_Author' for row 367, column 0. For more information about this error, please see http://datatables.net/tn/4 Could this be because we are only getting the full requestattributes, and not the data from "sub metadata"? I am testing with my in-progress backend for Libris, so I guess this might be an issue with that backend. Especially if things work as they should for you. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 Andrew Isherwood changed: What|Removed |Added Status|Failed QA |Needs Signoff --- Comment #18 from Andrew Isherwood --- I've dug into this some more. The additional weight appears to be due to exactly what you observed. This is due to us now returning all metadata associated with a request rather than just a subset defined by the relevant backend. For example, prior to these patches, the metadata was populated via (for example, with the BLDSS backend): https://github.com/PTFS-Europe/koha-ill-bldss/blob/master/Base.pm#L244-L293 With this bug, it's populated by: https://github.com/PTFS-Europe/koha/blob/bug_20996/Koha/REST/V1/Illrequests.pm#L126-L136 which gets *all* metadata regardless of the backend. Whether this is a good idea or not is up for debate I guess. Any calls on a request object's 'metadata' method will still use the backend's metadata method to return the curated metadata from the backend. Dunno what the best thing to do is really.. Anyway, putting the response weight aside, this all now appears to be working as designed, so I'm going to change the status back to "Needs Signoff" -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 Andrew Isherwood changed: What|Removed |Added Blocks||18589 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18589 [Bug 18589] Show ILLs as part of patron profile -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 --- Comment #17 from Andrew Isherwood --- Oooh, good investigating, I completely failed to find any difference in responses. I clearly didn't have sufficiently well populated requests! What you've found sounds promising, I'll do some digging and see what I can find. Thanks! -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 Magnus Enger changed: What|Removed |Added Status|Needs Signoff |Failed QA --- Comment #16 from Magnus Enger --- No bug 21079 applied, no. At least I think I found the difference in the JSON delivered, thanks to the "Response preview" in the Chromium developer tools. Without the patches applied, each request described in the JSON data has a data element "metadata" which contains just the properties that are set up by "sub metadata" in Base.pm. With the patches applied, this same data element contains *all* the request attributes, but not the data from "sub metadata" (this results in Title and Author not being populated in the table, as well as the errors/warnings I mentioned earlier). I have not looked into what is causing this difference. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 --- Comment #15 from Andrew Isherwood --- Hi Magnus. I can't account for the weight increase, I've not been able to replicate it. You've not also got bug 21079 applied have you? That runs a database update that creates additional illrequestmetadata values for request from the FreeForm backend. That's the only thing I can thing that might have increased the response size. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 --- Comment #14 from Andrew Isherwood --- Created attachment 79379 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=79379=edit Bug 20996: (follow-up) Add to conf tpl Since we now require the block, we should add it to the config templates -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 Andrew Isherwood changed: What|Removed |Added Status|Failed QA |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 --- Comment #13 from Andrew Isherwood --- It's not related to this bug, it's due to an item not having an author, I've seen it before. I'll create a bug if I've not already created one. Thanks for mentioning it! -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 --- Comment #12 from Magnus Enger --- Also, with the patch applied, when I try to view /cgi-bin/koha/ill/ill-requests.pl I get two JS popup warnings: DataTables warning: table id=ill-requests - Requested unknown parameter 'metadata_Author' for row 0, column 0. For more information about this error, please see http://datatables.net/tn/4 DataTables warning: table id=ill-requests - Requested unknown parameter 'metadata_Author' for row 358, column 0. For more information about this error, please see http://datatables.net/tn/4 Author and Title is empty in the table. Not sure if this is a data problem, but it did not happen before I applied the patches. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 --- Comment #11 from Andrew Isherwood --- Interesting, I'd not noticed the weight increase. I'll investigate! -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 --- Comment #10 from Magnus Enger --- Some observations from a DB with 359 illrequests and 34,485 illrequestattributes. Measurements of the standard API call to /api/v1/illrequests?embed=metadata,patron,capabilities,library: Without the patch: Size of data: 1MB Time: ~12 seconds With the patch: Size of data: 1.8MB Time: ~8 seconds The decrease in time is much appreciated! I'm not sure where the increase in weight comes from. Could it be avoided and save us even more time? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 --- Comment #9 from Andrew Isherwood --- Excellent point, thanks Magnus. I'll create a patch to add these -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 Magnus Enger changed: What|Removed |Added CC||mag...@libriotech.no Status|Needs Signoff |Failed QA --- Comment #8 from Magnus Enger --- If is now mandatory inside in koha-conf.xml it should probably be added to these template files? - etc/koha-conf.xml - debian/templates/koha-conf-site.xml.in It looks like the increase in speed is really significant, so I'll try to sign off as soon as I can. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 Andrew Isherwood changed: What|Removed |Added Status|ASSIGNED|Needs Signoff --- Comment #7 from Andrew Isherwood --- Since this is a performance enhancement, it's very hard to test without a substantial number of ILL requests in the database. I have therefore attached the script I created to populate large number of requests (add_ill.pl). To test: - Before applying patches, add a large number of requests, maybe 5000, using attached add_ill.pl - Make a request to: http://your-koha/api/v1/illrequests?embed=metadata,patron,capabilities,library - Time how long the response takes - Apply patches - Make another request to the same URL - Time the response => TEST: The response time should be substantially reduced -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 --- Comment #6 from Andrew Isherwood --- Created attachment 77100 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=77100=edit Script for creating large numbers of ILL requests, for performance testing -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 Bug 20996 depends on bug 7317, which changed state. Bug 7317 Summary: Add an Interlibrary Loan Module to Circulation and OPAC https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7317 What|Removed |Added Status|Pushed to Master|RESOLVED Resolution|--- |FIXED -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 --- Comment #5 from Andrew Isherwood --- Created attachment 76705 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=76705=edit Bug 20996: Further enhance performance We were immediately unblessing all request objects after retrieval then, at a later stage, re-instantiating each one so we could call methods on that. This was a very bad idea. We now preserve the objects for as long as we can. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 --- Comment #3 from Andrew Isherwood --- Created attachment 76687 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=76687=edit Bug 20996: Remove prefix use of borrower category This patch removes the potential use of borrower category as a ILL request ID prefix. It makes no sense. We provide the ability for a site to define a request prefix based on branch, there is no use case for using the borrower category. Add to this that the borrower for every request was being retrieved in order to get the category, it's a huge performance hit also. We also now require the block in the block and complain if it's not present. The request prefix should be defined in this block. This patch also improves the performance of the API request that returns all requests, optionally including additional data. It also deprecates the overloaded TO_JSON method and moves the request augmentation code into the API route's controller. It may be that we want to shift it out of there at some point, but it is fine where it is for now. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 --- Comment #4 from Andrew Isherwood --- Created attachment 76688 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=76688=edit Bug 20996: Fix unit tests - Remove unit tests relating to TO_JSON - Add tests for new explicit embedding - Modify tests relating to now deprecated brw_cat -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 --- Comment #2 from Andrew Isherwood --- Hi Benjamin I'm not aware of anywhere where we query that table on type or value, it's only ever the illrequest_id we query via the join. So I'm not sure of the value of adding these indexes. I might be missing something though, so please put me straight if so! -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 Benjamin Rokseth changed: What|Removed |Added CC||benjamin.rokseth@deichman.n ||o --- Comment #1 from Benjamin Rokseth --- Hi, db design at least partly to blame. At least a quick fix would be to create an index on illrequestattributes CREATE INDEX `illrequestattributes_type_value` ON illrequestattributes(type,value(32)); -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 Josef Moravec changed: What|Removed |Added CC||josef.mora...@gmail.com -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 20996] Fix API response time on ILL request endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20996 Andrew Isherwood changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|koha-b...@lists.koha-commun |andrew.isherwood@ptfs-europ |ity.org |e.com Depends on||7317 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7317 [Bug 7317] Add an Interlibrary Loan Module to Circulation and OPAC -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/