https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34643 Victor Grousset/tuxayo <victor@tuxayo.net> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Failed QA --- Comment #20 from Victor Grousset/tuxayo <victor@tuxayo.net> ---
4) Attempt to checkout out an item with materials via SCO, it should fail
Turns out it worked >_< Another try: 1. CircConfirmItemParts: don't require CircConfirmItemPartsSCI: allow CircConfirmItemPartsSCO: Don't allow 2. have an item with "3 - Materials specified (bound volume or other part)" filled 3. go to http://localhost:8080/cgi-bin/koha/sco/sco-main.pl 4. try to check out the item 5. it works! :o After more testing, it needs only CircConfirmItemParts: "Require" to finally get "Item must be checked out at a circulation desk" Looked at the code, no idea why delete $needconfirm->{ADDITIONAL_MATERIALS} unless C4::Context->preference("CircConfirmItemPartsSCO"); wouldn't do it's work or would be circumvented by something else. After it's fixed, I think it should be back to need signoff. We can see that there were enough changes between the iterations so that basic workflow was compromised. Otherwise it would make QA be the only up to date test. --- Also, any sponsor for this change? -- You are receiving this mail because: You are watching all bug changes.