Summary: | Branchcode used for routing preview can be wrong | ||
---|---|---|---|
Product: | Koha | Reporter: | Jonathan Druart <jonathan.druart> |
Component: | Architecture, internals, and plumbing | Assignee: | Jonathan Druart <jonathan.druart> |
Status: | In Discussion --- | QA Contact: | Testopia <testopia> |
Severity: | enhancement | ||
Priority: | P5 - low | CC: | fridolin.somers, michaela.sieber, nick, oleonard |
Version: | unspecified | ||
Hardware: | All | ||
OS: | All | ||
See Also: | https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31318 | ||
GIT URL: | Change sponsored?: | --- | |
Patch complexity: | --- | Documentation contact: | |
Documentation submission: | Text to go in the release notes: | ||
Version(s) released in: | Circulation function: | ||
Bug Depends on: | 31318 | ||
Bug Blocks: | 2894 | ||
Attachments: |
Bug 31330: Routing preview - Use the library from the selected issue
Bug 31330: Routing preview - Use the library from the selected issue Bug 31330: Don't pass serial_escaped |
Description
Jonathan Druart
2022-08-09 12:55:07 UTC
The fallback to subscription.branchcode makes sense. But as you wrote, I think using the first item makes no sense - what if you moved part of the holdings into a "closed stacks" library location or just to another branch. This could easily happen in academic libraries where the newest issues might be kept somewhere else than the older less requested items. If an item is linked to the serial via serialitems, we should use the info of this specific item. Created attachment 138912 [details] [review] Bug 31330: Routing preview - Use the library from the selected issue The code looked so wrong. I have the feeling that this patch is correcting a problem, and clean the code a bit, but I need someone familiar with this page to know if it's correct. The change is about the library used to generate the hold, should it be the one from the holding library (if an item has been created with the serial item)? The one from the first published serial item? If no items have been created then the library from the subscription is used. code wise more fixes are needed here, especially the creation of methods for Koha::Serial::Item, but that's not in the scope. > The change is about the library used to generate the hold, should it be the > one from the holding library (if an item has been created with the serial > item)? The one from the first published serial item? > If no items have been created then the library from the subscription is used. Hm, the hold feature has been broken for so long, that I have never seen it working :( Bug 2894 - Routing list holds are broken So you are probably right about the code not being correct. I think it if there is no item, you can't place a hold and the hold should be an item level hold (it needs to be as serial items represent a specific issue and are not identical). If you have created multiple items on receiving the issue (a recent feature given the age of this code) we might need to pick one, which makes it more complicate than it was in the past maybe. I thought this was a display question for the routing slip until now... should we move the hold stuff elsewhere or does it need to be solved? Created attachment 166955 [details] [review] Bug 31330: Routing preview - Use the library from the selected issue Created attachment 166956 [details] [review] Bug 31330: Don't pass serial_escaped What's the point of this? Am I missing something here? Rebased and adjust. Can we get this moving again? I am a bit lost to be honest. What's needed? Happy to work on it if I get some guidance. Hi Any news ? Is this enhancement or bug ? (In reply to Fridolin Somers from comment #9) > Hi > > Any news ? Is this enhancement or bug ? Why are you asking? We need more eyes on this, so if someone at BibLibre could help it would be great. Look at this and bug 2894. What's the expected behaviour? Are the patches fixing the problem correctly? A librarian asked about this saying it is a bug. Can they test the patch and confirm it fixes the issue as they would expect? |