| Summary: | import_webservice_batch.pl cronjob completely broken | ||
|---|---|---|---|
| Product: | Koha | Reporter: | Joonas Kylmälä <joonas.kylmala> |
| Component: | Command-line Utilities | Assignee: | Joonas Kylmälä <joonas.kylmala> |
| Status: | CLOSED FIXED | QA Contact: | Marcel de Rooy <m.de.rooy> |
| Severity: | normal | ||
| Priority: | P5 - low | CC: | m.de.rooy, robin |
| Version: | Main | ||
| Hardware: | All | ||
| OS: | All | ||
| Change sponsored?: | --- | Patch complexity: | Trivial patch |
| Documentation contact: | Documentation submission: | ||
| Text to go in the release notes: | Version(s) released in: |
22.05.00
|
|
| Circulation function: | |||
| Attachments: |
Bug 30776: import missing subroutine to fix import_webservice_batch.pl
Bug 30776: import missing subroutine to fix import_webservice_batch.pl |
||
Created attachment 135046 [details] [review] Bug 30776: import missing subroutine to fix import_webservice_batch.pl Test plan: 1) Apply patch and notice the following error is gone: $ perl misc/cronjobs/import_webservice_batch.pl Undefined subroutine &main::GetStagedWebserviceBatches called at misc/cronjobs/import_webservice_batch.pl line 46. Moving directly to signed-off since this is a very trivial patch. Created attachment 135099 [details] [review] Bug 30776: import missing subroutine to fix import_webservice_batch.pl Test plan: 1) Apply patch and notice the following error is gone: $ perl misc/cronjobs/import_webservice_batch.pl Undefined subroutine &main::GetStagedWebserviceBatches called at misc/cronjobs/import_webservice_batch.pl line 46. Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> I really wonder how many people are using this? Pushed to master for 22.05, thanks to everybody involved [U+1F984] |
The import_webservice_batch.pl cronjob doesn't work at all due to a missing import. > $ perl misc/cronjobs/import_webservice_batch.pl > Undefined subroutine &main::GetStagedWebserviceBatches called at misc/cronjobs/import_webservice_batch.pl line 46 Not sure if anyone uses this but this is an easy fix, we can remove the feature later if it is unused.