Bug 4439

Summary: Two acq webservices should use REST API
Product: Koha Reporter: Galen Charlton <gmcharlt>
Component: AcquisitionsAssignee: Bugs List <koha-bugs>
Status: NEW --- QA Contact: Bugs List <koha-bugs>
Severity: minor    
Priority: P5 - low CC: dcook, hugh, veron
Version: master   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on: 9412    
Bug Blocks:    
Attachments: bug 4439 Change some helper files to use C4::Service

Description Chris Cormack 2010-05-21 01:27:11 UTC


---- Reported by gmcharlt@gmail.com 2010-05-03 16:23:07 ----

To be consistent with previously-defined web services, admin/check_budget_parent.pl and admin/check_parent_total.pl should use C4::Service instead of hand-written code and ./koha-tmpl/intranet-tmpl/prog/en/modules/acqui/ajax.tmpl.



--- Bug imported by chris@bigballofwax.co.nz 2010-05-21 01:27 UTC  ---

This bug was previously known as _bug_ 4439 at http://bugs.koha.org/cgi-bin/bugzilla3/show_bug.cgi?id=4439

Actual time not defined. Setting to 0.0
Setting qa contact to the default for this product.
   This bug either had no qa contact or an invalid one.

Comment 1 Hugh Davenport 2013-01-17 05:20:50 UTC
Created attachment 14653 [details] [review]
bug 4439 Change some helper files to use C4::Service

Depends on bug 4437 to avoid conflicts

Depends on bug 9412 for a new feature used

Also fixes bug 9411

Signed-off-by: Hugh Davenport <hugh@davenport.net.nz>
Comment 2 Owen Leonard 2013-03-19 09:50:30 UTC
This patch needs a test plan.
Comment 3 Hugh Davenport 2013-07-05 21:13:00 UTC
Hey Owen,

What would the format of a test plan be?

Cheers,

Hugh
Comment 4 Marc VĂ©ron 2016-08-22 08:38:11 UTC
Still valid?
Comment 5 Jonathan Druart 2018-06-28 18:34:43 UTC
*** Bug 9411 has been marked as a duplicate of this bug. ***
Comment 6 David Cook 2022-08-22 03:03:32 UTC
This should use the REST API instead of legacy C4::Service
Comment 7 Katrin Fischer 2023-01-07 22:09:00 UTC
Resetting assignee since this hasn't been updated in several years.

Also updated the bug description as per David's comment.

Assume that the patch would need a complete rewrite.