Bug 38073 - Missing use after Bug 25812
Summary: Missing use after Bug 25812
Status: Pushed to main
Alias: None
Product: Koha
Classification: Unclassified
Component: SIP2 (show other bugs)
Version: Main
Hardware: All All
: P5 - low minor
Assignee: Fridolin Somers
QA Contact: Paul Derscheid
URL:
Keywords:
Depends on: 25812
Blocks:
  Show dependency treegraph
 
Reported: 2024-10-03 12:57 UTC by Fridolin Somers
Modified: 2024-11-07 17:51 UTC (History)
2 users (show)

See Also:
Change sponsored?: ---
Patch complexity: Trivial patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
24.11.00
Circulation function:


Attachments
Bug 38073: Add missing use in C4/SIP/ILS.pm (931 bytes, patch)
2024-10-10 09:47 UTC, Fridolin Somers
Details | Diff | Splinter Review
Bug 38073: Add missing use in C4/SIP/ILS.pm (985 bytes, patch)
2024-10-21 20:28 UTC, Brendan Lawlor
Details | Diff | Splinter Review
Bug 38073: Add missing use in C4/SIP/ILS.pm (1.02 KB, patch)
2024-11-04 13:52 UTC, Paul Derscheid
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Fridolin Somers 2024-10-03 12:57:44 UTC
Bug 25812 added some code with Koha::XXX objects.
They must be added in used modules :

use Koha::Account::Lines;
use Koha::Items;
use Koha::Number::Price;
Comment 1 Fridolin Somers 2024-10-10 09:47:48 UTC
Created attachment 172582 [details] [review]
Bug 38073: Add missing use in C4/SIP/ILS.pm

Bug 25812 added some code with Koha::XXX objects.
They must be added in used modules

Test plan :
Check SIP server still runs
Comment 2 Brendan Lawlor 2024-10-21 20:28:15 UTC
Created attachment 173083 [details] [review]
Bug 38073: Add missing use in C4/SIP/ILS.pm

Bug 25812 added some code with Koha::XXX objects.
They must be added in used modules

Test plan :
Check SIP server still runs

Signed-off-by: Brendan Lawlor <blawlor@clamsnet.org>
Comment 3 Brendan Lawlor 2024-10-21 20:37:58 UTC
Test notes:
In ktd repo edit docker-compose-light.yml
Under services: koha: ports: add port - "6001:6001"

In ktd --shell install telnet
sudo apt update && sudo apt install telnet

Set a patron username and password to test SIP. 
I used Henry Acevedo username:ace password: 1234Abcd

Apply patch and restart_all

Connect to SIP server:
telnet localhost 6001

Authenticate as SIP user:
9300CNterm1|COterm1|CPCPL|

Send patron info request:
6300020060329    201700Y         AOCPL|AAace|AD1234Abcd|

Confirm response:
64              00020241021    202455000000000000000000000000AOCPL|AA23529000035676|AEHenry Acevedo|BLY|CQY|BHUSD|BV0|CC5|BD4345 Library Rd. Springfield, MA 44224|BF(212) 555-1212|PB19580530|PCS|PIY|AFGreetings from Koha. |

I'd be happy to test more is there is a more specific test plan.
Comment 4 Paul Derscheid 2024-11-04 13:52:44 UTC
Created attachment 173920 [details] [review]
Bug 38073: Add missing use in C4/SIP/ILS.pm

Bug 25812 added some code with Koha::XXX objects.
They must be added in used modules

Test plan :
Check SIP server still runs

Signed-off-by: Brendan Lawlor <blawlor@clamsnet.org>
Signed-off-by: Paul Derscheid <paul.derscheid@lmscloud.de>
Comment 5 Katrin Fischer 2024-11-07 17:51:21 UTC
Pushed for 24.11!

Well done everyone, thank you!