[Koha-bugs] [Bug 21417] EDI ordering fails when basket and EAN libraries do not match
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21417 Fridolin SOMERS changed: What|Removed |Added CC||fridolin.som...@biblibre.co ||m --- Comment #18 from Fridolin SOMERS --- Pushed to 17.11.x for 17.11.12 -- 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 21417] EDI ordering fails when basket and EAN libraries do not match
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21417 Martin Renvoize changed: 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 21417] EDI ordering fails when basket and EAN libraries do not match
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21417 Martin Renvoize changed: What|Removed |Added Status|Pushed to Master|Pushed to Stable --- Comment #17 from Martin Renvoize --- Pushed to 18.05.x for 18.05.05 -- 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 21417] EDI ordering fails when basket and EAN libraries do not match
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21417 Nick Clemens changed: What|Removed |Added CC||n...@bywatersolutions.com Status|Passed QA |Pushed to Master --- Comment #16 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 21417] EDI ordering fails when basket and EAN libraries do not match
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21417 Martin Renvoize changed: What|Removed |Added Version|17.11 |master -- 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 21417] EDI ordering fails when basket and EAN libraries do not match
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21417 Martin Renvoize changed: What|Removed |Added Status|Signed Off |Passed QA --- Comment #15 from Martin Renvoize --- I'm a little disturbed by the lack of tests for Koha::EDI, but can't warrant failing qa here for that as it's the entire module that's lacking, not just this bugfix. -- 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 21417] EDI ordering fails when basket and EAN libraries do not match
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21417 Martin Renvoize changed: What|Removed |Added QA Contact|testo...@bugs.koha-communit |martin.renvoize@ptfs-europe |y.org |.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 21417] EDI ordering fails when basket and EAN libraries do not match
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21417 Martin Renvoize changed: What|Removed |Added Attachment #79525|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 21417] EDI ordering fails when basket and EAN libraries do not match
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21417 Martin Renvoize changed: What|Removed |Added Attachment #79607|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 21417] EDI ordering fails when basket and EAN libraries do not match
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21417 --- Comment #14 from Martin Renvoize --- Created attachment 80221 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=80221=edit Bug 21417: (QA follow-up) Display default EAN as such in admin display A branch value of null is interpreted as the default Add text 'All libraries' to this case in the display rather than blank so that its function is clear to the user Signed-off-by: Pierre-Marc Thibault 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 21417] EDI ordering fails when basket and EAN libraries do not match
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21417 --- Comment #13 from Martin Renvoize --- Created attachment 80220 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=80220=edit Bug 21417: EDI ordering fails when basket and EAN libraries do not match When clicking Create EDIFACT Order on /acqui/basket.pl, an EDIFACT message will not be generated if aqbasket.branch does not match edifact_ean.branchcode. This failure does not generate any sort of error message, it just fails to produce a message. We should allow ean's to not require a branch to be set, then if we don't find a branch specific each, we can look for the default version of the ean. Test Plan: 1) Apply this patch set 2) Run updatedatabase 3) Verify you can create a Library EAN without setting a branchcode for it 4) Verify you can use this EAN to send an EDI order where the basket has a branchcode set Signed-off-by: Colin Campbell Signed-off-by: Pierre-Marc Thibault 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 21417] EDI ordering fails when basket and EAN libraries do not match
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21417 Martin Renvoize changed: What|Removed |Added CC||martin.renvoize@ptfs-europe ||.com Attachment #79525|signed off patch adding |Bug 21417: EDI ordering description|default option |fails when basket and EAN ||libraries -- 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 21417] EDI ordering fails when basket and EAN libraries do not match
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21417 Kyle M Hall changed: What|Removed |Added Attachment #79528|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 21417] EDI ordering fails when basket and EAN libraries do not match
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21417 Kyle M Hall changed: What|Removed |Added Assignee|koha-b...@lists.koha-commun |k...@bywatersolutions.com |ity.org | -- 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/
[Koha-bugs] [Bug 21417] EDI ordering fails when basket and EAN libraries do not match
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21417 Pierre-Marc Thibault changed: What|Removed |Added Status|Needs Signoff |Signed Off -- 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/
[Koha-bugs] [Bug 21417] EDI ordering fails when basket and EAN libraries do not match
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21417 --- Comment #12 from Pierre-Marc Thibault --- Created attachment 79607 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=79607=edit Bug 21417 - Display default EAN as such in admin display A branch value of null is interpreted as the default Add text 'All libraries' to this case in the display rather than blank so that its function is clear to the user Signed-off-by: Pierre-Marc Thibault -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ 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 21417] EDI ordering fails when basket and EAN libraries do not match
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21417 Pierre-Marc Thibault changed: What|Removed |Added CC||pierre-marc.thibault@inlibr ||o.com --- Comment #11 from Pierre-Marc Thibault --- (In reply to Colin Campbell from comment #10) > Created attachment 79528 [details] [review] > Supplemental patch to display the null entry > > This adds text 'All libraries' to display in the admin screen rather than > leaving the column blank to clarify function to user Can you provide a test plan to see the change easily please? The branchcode column cannot be NULL and you have no choice to assign a library to an EAN, so it's a bit hard to test it. -- 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/
[Koha-bugs] [Bug 21417] EDI ordering fails when basket and EAN libraries do not match
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21417 --- Comment #10 from Colin Campbell --- Created attachment 79528 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=79528=edit Supplemental patch to display the null entry This adds text 'All libraries' to display in the admin screen rather than leaving the column blank to clarify function to user -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ 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 21417] EDI ordering fails when basket and EAN libraries do not match
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21417 Colin Campbell changed: What|Removed |Added Attachment #79476|0 |1 is obsolete|| --- Comment #9 from Colin Campbell --- Created attachment 79525 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=79525=edit signed off patch adding default option -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ 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 21417] EDI ordering fails when basket and EAN libraries do not match
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21417 Kyle M Hall changed: What|Removed |Added Attachment #79475|0 |1 is obsolete|| -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ 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 21417] EDI ordering fails when basket and EAN libraries do not match
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21417 --- Comment #8 from Colin Campbell --- (In reply to Kyle M Hall from comment #6) > Created attachment 79475 [details] [review] > Bug 21417: Fix the warning lines to output the basketno just submitted patch against bug 21425 -- 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/
[Koha-bugs] [Bug 21417] EDI ordering fails when basket and EAN libraries do not match
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21417 --- Comment #7 from Kyle M Hall --- Created attachment 79476 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=79476=edit Bug 21417: EDI ordering fails when basket and EAN libraries do not match When clicking Create EDIFACT Order on /acqui/basket.pl, an EDIFACT message will not be generated if aqbasket.branch does not match edifact_ean.branchcode. This failure does not generate any sort of error message, it just fails to produce a message. We should allow ean's to not require a branch to be set, then if we don't find a branch specific each, we can look for the default version of the ean. Test Plan: 1) Apply this patch set 2) Run updatedatabase 3) Verify you can create a Library EAN without setting a branchcode for it 4) Verify you can use this EAN to send an EDI order where the basket has a branchcode set -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ 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 21417] EDI ordering fails when basket and EAN libraries do not match
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21417 --- Comment #6 from Kyle M Hall --- Created attachment 79475 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=79475=edit Bug 21417: Fix the warning lines to output the basketno -- 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/
[Koha-bugs] [Bug 21417] EDI ordering fails when basket and EAN libraries do not match
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21417 Kyle M Hall changed: What|Removed |Added Status|NEW |Needs Signoff -- 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/
[Koha-bugs] [Bug 21417] EDI ordering fails when basket and EAN libraries do not match
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21417 --- Comment #5 from Colin Campbell --- I'm thinking there should be a default line in the config, which should cope with null as well. We should fall back to that if exists if no library-specific entry exists. -- 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/
[Koha-bugs] [Bug 21417] EDI ordering fails when basket and EAN libraries do not match
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21417 --- Comment #4 from Andrew --- A bit of correction to my original comment: I'd thought that aqbasket.branch was auto-populating with the branch the user is logged into. In fact, aqbasket.branch is null by default. The dropdown with which one would change it to specify a branch gives the current branch as a default option, but does not actually set it. This explains why we haven't seen this error more often and makes it a bit less dire -- the librarian needs to do a little extra work to make the system break. -- 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/
[Koha-bugs] [Bug 21417] EDI ordering fails when basket and EAN libraries do not match
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21417 Colin Campbell changed: What|Removed |Added CC||colin.campbell@ptfs-europe. ||com --- Comment #3 from Colin Campbell --- Must admit that I've been wondering whether the branch - ean link is really useful. We inherited that from the original EDI implementation that envisioned a situation where the system may have had multiple eans. Thats not a case we've encountered in practice. Wonder if there should be a global all libraries default, and a branch requiring a different san/ean would be the special case. -- 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/
[Koha-bugs] [Bug 21417] EDI ordering fails when basket and EAN libraries do not match
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21417 --- Comment #2 from Kyle M Hall --- Not sure what the correct fix here is. There are a number of options: A) Strip EANs from the pulldown that don't match the basket's branchcode B) Ignore the branchcode for the purpose of the search i) Add option for 'all libraries' to EAN editor? C) Halt with a screen error I suppose we could implement all of the above, but really only one should be sufficient. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ 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 21417] EDI ordering fails when basket and EAN libraries do not match
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21417 Kyle M Hall changed: What|Removed |Added CC||k...@bywatersolutions.com --- Comment #1 from Kyle M Hall --- When this situation is triggered, the error "No sender ean passed to order creation: basket = $self->{basket}->basketno()"; is written to the log. It should be noted the use "$self->{basket}->basketno()" does not get interpolated correctly and should be moved outside the string. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ 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/