The benefit of recording vendor issues is that you can identify problems you have had with the service and use this as a negotiating tool with vendors at contract renegotiation.
Created attachment 149509 [details] [review] Bug 33105: DB changes
Created attachment 149510 [details] [review] Bug 33105: DBIC schema changes
Created attachment 149511 [details] [review] Bug 33105: DBIC specific
Created attachment 149512 [details] [review] Bug 33105: Add Koha object classes
Created attachment 149513 [details] [review] Bug 33105: Add vendor issues This patch is the main patch of this patch set, it contains the controller acqui/vendor_issues.pl, its corresponding template, and some links to this script. It adds: * A new DB table aqbookseller_issues linked with the aqbooksellers table * A new subpermission acquisition.issue_manage * A new authorised value category VENDOR_ISSUE_TYPE and two examples MAINTENANCE and OUTAGE * A new controller couple acqui/vendor_issues.[pl,tt] Test plan: 0. Apply the patches, run updatedatabase and restart_all 1. Go to the acquisition module, create a new vendor or use an existing one 2. Create a couple of issues for this vendor 3. Edit/Delete and search for those issues This is the basics for tracking issues with vendors. Suggestions welcome, on follow-up bug reports.
Created attachment 149514 [details] [review] Bug 33105: REST API specs
Created attachment 149515 [details] [review] Bug 33105: Add tests
Deletion is currently failing due to a typo in vendor_issues.pl cgi-bin/koha/acqui/vendor_iddues.pl?op=delete_confirm&issue_id=1 iddues instead of issues. Looking good otherwise!
Created attachment 149530 [details] [review] Bug 33105: Add vendor issues This patch is the main patch of this patch set, it contains the controller acqui/vendor_issues.pl, its corresponding template, and some links to this script. It adds: * A new DB table aqbookseller_issues linked with the aqbooksellers table * A new subpermission acquisition.issue_manage * A new authorised value category VENDOR_ISSUE_TYPE and two examples MAINTENANCE and OUTAGE * A new controller couple acqui/vendor_issues.[pl,tt] Test plan: 0. Apply the patches, run updatedatabase and restart_all 1. Go to the acquisition module, create a new vendor or use an existing one 2. Create a couple of issues for this vendor 3. Edit/Delete and search for those issues This is the basics for tracking issues with vendors. Suggestions welcome, on follow-up bug reports.
Created attachment 149531 [details] [review] Bug 33105: REST API specs
Created attachment 149532 [details] [review] Bug 33105: Add tests
(In reply to Jonathan Field from comment #8) > Deletion is currently failing due to a typo in vendor_issues.pl oops, should be fixed now!
Created attachment 149556 [details] [review] Bug 33105: DB changes Signed-off-by: Jonathan Field <jonathan.fieeld@ptfs-europe.com>
Created attachment 149557 [details] [review] Bug 33105: DBIC schema changes Signed-off-by: Jonathan Field <jonathan.fieeld@ptfs-europe.com>
Created attachment 149558 [details] [review] Bug 33105: DBIC specific Signed-off-by: Jonathan Field <jonathan.fieeld@ptfs-europe.com>
Created attachment 149559 [details] [review] Bug 33105: Add Koha object classes Signed-off-by: Jonathan Field <jonathan.fieeld@ptfs-europe.com>
Created attachment 149560 [details] [review] Bug 33105: Add vendor issues This patch is the main patch of this patch set, it contains the controller acqui/vendor_issues.pl, its corresponding template, and some links to this script. It adds: * A new DB table aqbookseller_issues linked with the aqbooksellers table * A new subpermission acquisition.issue_manage * A new authorised value category VENDOR_ISSUE_TYPE and two examples MAINTENANCE and OUTAGE * A new controller couple acqui/vendor_issues.[pl,tt] Test plan: 0. Apply the patches, run updatedatabase and restart_all 1. Go to the acquisition module, create a new vendor or use an existing one 2. Create a couple of issues for this vendor 3. Edit/Delete and search for those issues This is the basics for tracking issues with vendors. Suggestions welcome, on follow-up bug reports. Signed-off-by: Jonathan Field <jonathan.fieeld@ptfs-europe.com>
Created attachment 149561 [details] [review] Bug 33105: REST API specs Signed-off-by: Jonathan Field <jonathan.fieeld@ptfs-europe.com>
Created attachment 149562 [details] [review] Bug 33105: Add tests Signed-off-by: Jonathan Field <jonathan.fieeld@ptfs-europe.com>