February 2020 - Koha-bugs - lists.koha-community.org

[Bug 15522] New interface for revamped circulation rules
by bugzilla-daemon＠bugs.koha-community.org 18 Feb '20

18 Feb '20

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15522 --- Comment #52 from Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> --- Created attachment 99149 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=99149&action=ed… Bug 15522: Add JS build pipeline This is not directly testable, but can be either used through the Makefile: $ make js-build (This will automatically install all dependencies.) Or by manually installing yarn: $ sudo apt install nodejs npm $ sudo npm install -g yarn $ yarn install $ yarn build (There is also `make js-watch` and `yarn watch`, which automatically recompile when files are changed.) JD note: the make rule does not work -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 15522] New interface for revamped circulation rules
by bugzilla-daemon＠bugs.koha-community.org 18 Feb '20

18 Feb '20

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15522 --- Comment #51 from Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> --- Created attachment 99148 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=99148&action=ed… Bug 15522: Add API for circulation rules Test plan: 1) Run t/db_dependent/api/v1/circ_rules.t . -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 12430] Search truncation disables relevance ranking when not using QueryParser
by bugzilla-daemon＠bugs.koha-community.org 18 Feb '20

18 Feb '20

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12430 --- Comment #43 from David Cook <dcook(a)prosentient.com.au> --- Whoa... added unit tests when UseQueryParser is enabled and it's giving unpredictable results. Half the time, the expected result is correct and half the time it is wrong. It looks like it's swapping the order of the operands around. Bizarre. # got: '@or @or @attr 1=1016 @attr 5=1 @attr 4=6 "test" @attr 2=102 @attr 9=34 @attr 5=1 @attr 4=6 "test" @attr 2=102 @attr 9=20 @attr 5=1 @attr 4=6 "test"' # expected: '@or @or @attr 1=1016 @attr 5=1 @attr 4=6 "test" @attr 2=102 @attr 9=20 @attr 5=1 @attr 4=6 "test" @attr 2=102 @attr 9=34 @attr 5=1 @attr 4=6 "test"' I don't know what to do about that... it'll unpredictably bust the tests on master, but it is arguably a problem with the QueryParser or at least a problem of evaluating the query based on string matching instead of parsing the query and analyzing it logically... I think I'll leave out that test for now... especiallys ince we are planning to remove the QueryParser... -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 24213] New: Koha::Object->get_from_storage should return undef if the object has been deleted
by bugzilla-daemon＠bugs.koha-community.org 18 Feb '20

18 Feb '20

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24213 Bug ID: 24213 Summary: Koha::Object->get_from_storage should return undef if the object has been deleted Change sponsored?: --- Product: Koha Version: unspecified Hardware: All OS: All Status: ASSIGNED Severity: normal Priority: P5 - low Component: Architecture, internals, and plumbing Assignee: jonathan.druart(a)bugs.koha-community.org Reporter: jonathan.druart(a)bugs.koha-community.org QA Contact: testopia(a)bugs.koha-community.org It currently explodes with something like: DBIC result _type isn't of the _type Subscription at t/db_dependent/Biblio.t line 627. -- You are receiving this mail because: You are watching all bug changes.

1 21

[Bug 12430] Search truncation disables relevance ranking when not using QueryParser
by bugzilla-daemon＠bugs.koha-community.org 18 Feb '20

18 Feb '20

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12430 --- Comment #42 from David Cook <dcook(a)prosentient.com.au> --- (In reply to David Cook from comment #41) > Created attachment 99142 [details] [review] > First Draft buildQuery unit test This isn't complete yet, but it's a first pass before I have to run out of the office for a few hours. I need to add more permutations to the tests: 1. like a query with no * or QueryAutoTruncate to see if QueryWeightFields still works 2. like a more complex CCL/CCLish query 3. Turn on UseQueryParser (even if I think we should soon remove QueryParser from Koha) 4. Add limits to the unit tests Note that buildQuery spits out a lot of warnings, but I don't think they're relevant for this test. Rather, I think they're just indicative that buildQuery isn't very well written. Of course, these unit tests could be the basis for improving it. Thanks, Jonathan, for poking me on this one. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 12430] Search truncation disables relevance ranking when not using QueryParser
by bugzilla-daemon＠bugs.koha-community.org 18 Feb '20

18 Feb '20

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12430 --- Comment #41 from David Cook <dcook(a)prosentient.com.au> --- Created attachment 99142 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=99142&action=ed… First Draft buildQuery unit test -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 12430] Search truncation disables relevance ranking when not using QueryParser
by bugzilla-daemon＠bugs.koha-community.org 18 Feb '20

18 Feb '20

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12430 --- Comment #40 from David Cook <dcook(a)prosentient.com.au> --- (In reply to Jonathan Druart from comment #36) > Can you put commit title and bug report title in sync please? Ok hopefully this is more in line with what you were thinking. > I guess nobody will have time to provide tests to cover this change? I'm going to try to do this now in the next 20 minutes... -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 12430] Search truncation disables relevance ranking when not using QueryParser
by bugzilla-daemon＠bugs.koha-community.org 18 Feb '20

18 Feb '20

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12430 David Cook <dcook(a)prosentient.com.au> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #98619|0 |1 is obsolete| | --- Comment #39 from David Cook <dcook(a)prosentient.com.au> --- Created attachment 99141 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=99141&action=ed… Bug 12430: Force relevance ranking when not using QueryParser The system preference "QueryWeightFields" is mutually exclusive with the system preference "QueryAutoTruncate" and the * truncation modifier, when not using QueryParser. If you use truncation, relevance won't work anymore. (N.B. Relevance doesn't work probably when using QueryParser, but for a very different reason beyond the current scope of this bug.) This patch adds relevance ranking when using truncation (or basically when QueryWeightFields is disabled). _TEST PLAN_ 1) Turn on "QueryWeightFields" and set "QueryAutoTruncate" to auto 2) Turn off "UseQueryParser" 3) Do a keyword search in the OPAC 4) Note that the results are ordered by biblionumber ascending 5) Apply patch 6) Do the same keyword search in the OPAC 7) Note that the results are never ordered differently (presumably in relevance order :P) If you're a stickler, throw in some warns so that you can see what CCL query is being sent to Zebra. Signed-off-by: Victor Grousset/tuxayo <victor(a)tuxayo.net> Signed-off-by: Katrin Fischer <katrin.fischer.83(a)web.de> Signed-off-by: Katrin Fischer <katrin.fischer.83(a)web.de> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 12430] Search truncation disables relevance ranking when not using QueryParser
by bugzilla-daemon＠bugs.koha-community.org 18 Feb '20

18 Feb '20

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12430 David Cook <dcook(a)prosentient.com.au> changed: What |Removed |Added ---------------------------------------------------------------------------- Summary|Search truncation disables |Search truncation disables |relevance ranking |relevance ranking when not | |using QueryParser --- Comment #38 from David Cook <dcook(a)prosentient.com.au> --- (In reply to Jonathan Druart from comment #36) > Can you put commit title and bug report title in sync please? > Actually, I'm not 100% sure what you mean by this, but I'm going to -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 12430] Search truncation disables relevance ranking
by bugzilla-daemon＠bugs.koha-community.org 18 Feb '20

18 Feb '20

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12430 David Cook <dcook(a)prosentient.com.au> changed: What |Removed |Added ---------------------------------------------------------------------------- Summary|Using QueryAutoTruncate |Search truncation disables |breaks relevance ranking |relevance ranking -- You are receiving this mail because: You are watching all bug changes.

1 0