https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34643
Victor Grousset/tuxayo <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Failed QA --- Comment #20 from Victor Grousset/tuxayo <[email protected]> --- > 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. _______________________________________________ Koha-bugs mailing list [email protected] https://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/
