Like Bug 19372, selecting MARC framework currently doesn't work when adding to basket from an external source. Strangly I can reproduce on koha-testing-docker, but we have this issue with a Ubuntu Focal install. Looks like it comes from a bad syntaxe than needs to be replaced in any case.
Created attachment 161576 [details] [review] Bug 35927: Selecting MARC framework again doesn't work when adding to basket from an external source Like Bug 19372, selecting MARC framework currently doesn't work when adding to basket from an external source. Strangly I can reproduce on koha-testing-docker, but we have this issue with a Ubuntu Focal install. Looks like it comes from a bad syntaxe than needs to be replaced in any case. Test plan: 1) Add an order to a basket from an external source 2) Select another framework than the default one 3) Chek the framework code you will pick will be used in the created biblio record
I'm not seeing this on the current master. Are there any additional steps I need to do? Happy to sign off if this is aimed at improving the code and no change in behavour. Or should I test using Ubuntu Focal as the operating system? How I tested: 1. Go to Acquisitions, then click 'Search'. 2. Select 'My Basket' > Add to basket > From an external source. 3. In the 'Order from external source' form: - Title: DITA best practices - Select MARC framework: select another framework, such as 'Books. Booklets, Workbooks' 4. For actions, select +Order. 5. Complete the required fields and click 'Add item'. 6. Complete the accounting details form and click 'Save'. 7. Edit the newly created record. 8. Under the Settings menu, the framework should be the same as you selected in step 3. Note: If I repeat and add the same record again (for duplicate warning: create new record), the new record shows as the default framework. (I'm not sure why you would do this, but it is possible to. The patch doesn't fix this.)
Can't reproduce on a Ubuntu 20.04 ktd. tested on marc21 and unimarc an picked an appropriate external server source for each. Each time on the record, I got the framework that I picked in the search result view. --- Fridolin: > Strangly I can reproduce on koha-testing-docker, but we have this issue with a Ubuntu Focal install. Did you mean "I can't reproduce" ? --- David: > Happy to sign off if this is aimed at improving the code and no change in behavour. Tested with patches on Ubuntu 20.04 with unimarc. At least no sign of regression indeed. @Fridolin: was this patch tested on the instance that had the issue? Back to assignee to avoid having more people try to test while a way is found to reproduce. And if not, we say that it's a hardening patch and testing is just needed to see that there is no obvious regression.
Indeed, "I can't reproduce" on ktd Indeed the patch is tested on the instance that had the issue? Ah it is a Debian Stretch I see.
Oh no, it's Debian 9 and it's not supported anymore :o After many attempts, here is how to start an old image: # Somehow it needs elasticsearch, here opensearch works fine KOHA_IMAGE=22.05-stretch bin/ktd --os1 up # It works with the master branch of KTD repo # After trying, older branches don't help to simplify the operation. # and **while** it's starting go into the container and edit the repositories vim /etc/apt/sources.list # replace "httpredir" with "archive" and removed the 2nd line. # In the end the content should be ``` deb http://archive.debian.org/debian stretch main ``` # otherwise it will 404 when trying to install the cpanoutdated package and crash
Created attachment 162487 [details] [review] Bug 35927: Selecting MARC framework again doesn't work when adding to basket from an external source Like Bug 19372, selecting MARC framework currently doesn't work when adding to basket from an external source. Strangly I can reproduce on koha-testing-docker, but we have this issue with a Ubuntu Focal install. Looks like it comes from a bad syntaxe than needs to be replaced in any case. Test plan: 1) Add an order to a basket from an external source 2) Select another framework than the default one on the search result view. Before doing 'add order' on choosen search result line. 3) Chek the framework code you will pick will be used in the created biblio record Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Victor Grousset/tuxayo <victor@tuxayo.net>
> After many attempts, here is how to start an old image: (and Koha's code should be on 22.05.x) And all this to no even reproduce the bug T_T Anyway, let's go with David's suggestion to sign-off that there is no visible regression. As it fixed the bug on your production instance.
If I understand here, this is a 22.05 bug? I updated the version on this report
> this is a 22.05 bug? No, it's just that one must go back to 22.05 to start a Debian 9, I tried that in case that was needed to reproduce.
Created attachment 166555 [details] [review] Bug 35927: Selecting MARC framework again doesn't work when adding to basket from an external source Like Bug 19372, selecting MARC framework currently doesn't work when adding to basket from an external source. Strangly I can reproduce on koha-testing-docker, but we have this issue with a Ubuntu Focal install. Looks like it comes from a bad syntaxe than needs to be replaced in any case. Test plan: 1) Add an order to a basket from an external source 2) Select another framework than the default one on the search result view. Before doing 'add order' on choosen search result line. 3) Chek the framework code you will pick will be used in the created biblio record Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Victor Grousset/tuxayo <victor@tuxayo.net> Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Pushed for 24.05! Well done everyone, thank you!
Pushed to 23.11.x for 23.11.06
Backported to 23.05.x for upcoming 23.05.12