Bug 32516

Summary: SIP2 no block flag on renewals is passed, but not used
Product: Koha Reporter: Nick Clemens (kidclamp) <nick>
Component: SIP2Assignee: Bugs List <koha-bugs>
Status: NEW --- QA Contact: Testopia <testopia>
Severity: normal    
Priority: P5 - low CC: david, martin.renvoize
Version: Main   
Hardware: All   
OS: All   
See Also: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32515
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on: 12225    
Bug Blocks:    

Description Nick Clemens (kidclamp) 2022-12-22 14:13:52 UTC
C4/SIP/ILS.pm
429 sub renew {
430     my ($self, $patron_id, $patron_pwd, $item_id, $title_id,
431     $no_block, $nb_due_date, $third_party,

no_block does not appear elsewhere in the routine
Comment 1 Nick Clemens (kidclamp) 2022-12-27 20:28:56 UTC
Marking as major - this is a behaviour change from bug 12225 - it seems we have allowed machines to send the no-block in the past and treated it as a normal checkin - now we die on calling a non-existent routine

We should add real support for this, but until we do we should stop failing
Comment 2 Martin Renvoize 2023-09-12 16:30:42 UTC
Is this still a major Nick?

I've just stumbled into it when working through fee acknowledgement fixes.. but I must admit, I've not come across this one in real life yet.
Comment 3 Nick Clemens (kidclamp) 2023-09-12 16:52:58 UTC
I think we saw that some vendors started failing on upgrade because of this, and that's why I marked major. Bug 32515 may have mitigated the issue for us. Marking as normal