Bug 36245 - ILL - Custom backend form action is broken
Summary: ILL - Custom backend form action is broken
Status: RESOLVED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: ILL (show other bugs)
Version: Main
Hardware: All All
: P5 - low critical
Assignee: Pedro Amorim
QA Contact: Martin Renvoize (ashimema)
URL:
Keywords: regression
Depends on: 36249
Blocks: 36273 35581
  Show dependency treegraph
 
Reported: 2024-03-06 13:12 UTC by Pedro Amorim
Modified: 2024-04-17 08:47 UTC (History)
7 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.05.00
Circulation function:


Attachments
Bug 36245: Remove cud- from op (1.54 KB, patch)
2024-03-06 13:15 UTC, Pedro Amorim
Details | Diff | Splinter Review
Bug 36245: Remove cud- from op (1.61 KB, patch)
2024-03-07 09:49 UTC, Martin Renvoize (ashimema)
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Pedro Amorim 2024-03-06 13:12:02 UTC

    
Comment 1 Pedro Amorim 2024-03-06 13:15:56 UTC
Created attachment 162834 [details] [review]
Bug 36245: Remove cud- from op

1) Install FreeForm, enable ILL module, run:
bash <(curl -s https://raw.githubusercontent.com/ammopt/koha-ill-dev/master/start-ill-dev.sh)
2) You'll have to switch the FreeForm branch to the one compatible with this work, i.e.:
cd /kohadevbox/koha/Koha/Illbackends/FreeForm
git checkout b_36243
3) Restart plack:
koha-plack --restart kohadev
4) Create a new FreeForm request, visit:
<staff_url>/cgi-bin/koha/ill/ill-requests.pl?method=create&backend=FreeForm
5) Set 'type' enter '42' in cardnumber and pick a library. Click 'Create'.
6) Edit that request, click "manage request" or if #1 visit:
<staff_url>/cgi-bin/koha/ill/ill-requests.pl?op=illview&illrequest_id=1
7) Click "Edit item metadata". Click "Submit". Notice it explodes.
9) Apply patch. Restart plack.
10) Repeat 7)
Comment 2 Martin Renvoize (ashimema) 2024-03-07 09:49:21 UTC
Created attachment 162875 [details] [review]
Bug 36245: Remove cud- from op

1) Install FreeForm, enable ILL module, run:
bash <(curl -s https://raw.githubusercontent.com/ammopt/koha-ill-dev/master/start-ill-dev.sh)
2) You'll have to switch the FreeForm branch to the one compatible with this work, i.e.:
cd /kohadevbox/koha/Koha/Illbackends/FreeForm
git checkout b_36243
3) Restart plack:
koha-plack --restart kohadev
4) Create a new FreeForm request, visit:
<staff_url>/cgi-bin/koha/ill/ill-requests.pl?method=create&backend=FreeForm
5) Set 'type' enter '42' in cardnumber and pick a library. Click 'Create'.
6) Edit that request, click "manage request" or if #1 visit:
<staff_url>/cgi-bin/koha/ill/ill-requests.pl?op=illview&illrequest_id=1
7) Click "Edit item metadata". Click "Submit". Notice it explodes.
9) Apply patch. Restart plack.
10) Repeat 7)

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 3 Martin Renvoize (ashimema) 2024-03-07 09:49:35 UTC
All working again now, thanks Pedro.
Comment 4 Pedro Amorim 2024-03-07 10:26:38 UTC
Updating this dependency to be able to apply whole tree in one go,
Comment 5 Katrin Fischer 2024-03-07 15:38:25 UTC
Pushed for 24.05!

Well done everyone, thank you!
Comment 6 Fridolin Somers 2024-04-17 08:47:35 UTC
Depends on Bug 36249 not in 23.11.x