| Summary: | Add the Koha::Objects->update method | ||
|---|---|---|---|
| Product: | Koha | Reporter: | Jonathan Druart <jonathan.druart> |
| Component: | Architecture, internals, and plumbing | Assignee: | Jonathan Druart <jonathan.druart> |
| Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
| Severity: | enhancement | ||
| Priority: | P5 - low | CC: | hector.hecaxmmx, kyle, m.de.rooy |
| Version: | Main | ||
| Hardware: | All | ||
| OS: | All | ||
| See Also: | https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18174 | ||
| GIT URL: | Initiative type: | --- | |
| Sponsorship status: | --- | Comma delimited list of Sponsors: | |
| Crowdfunding goal: | 0 | Patch complexity: | Small patch |
| Documentation contact: | Documentation submission: | ||
| Text to go in the release notes: | Version(s) released in: | ||
| Circulation function: | |||
| Bug Depends on: | 13019 | ||
| Bug Blocks: | 16965, 16966, 17091 | ||
| Attachments: |
Bug 16961: Add Koha::Objects->update
[SIGNED-OFF]Bug 16961: Add Koha::Objects->update [PASSED QA] Bug 16961: Add Koha::Objects->update |
||
|
Description
Jonathan Druart
2016-07-22 12:55:56 UTC
Created attachment 53645 [details] [review] Bug 16961: Add Koha::Objects->update In order to update several rows in a single query, we need this new method. Test plan: Confirm that the changes in Objects.t make sense and that the tests pass. Created attachment 53655 [details] [review] [SIGNED-OFF]Bug 16961: Add Koha::Objects->update In order to update several rows in a single query, we need this new method. Test plan: Confirm that the changes in Objects.t make sense and that the tests pass. Signed-off-by: Hector Castro <hector.hecaxmmx@gmail.com> Test passed successfully Time for an AUTOLOAD ? (In reply to Marcel de Rooy from comment #3) > Time for an AUTOLOAD ? Yes, it's in my todo list :) I have just needed it for bug 16966 and did not want to add a 'big' blocker. Hm, big dependency tree starting here. I am not sure if this line still makes sense...
Koha::Cities->search( { city_country => 'UK' } )->update( { city_country => 'EU' } );
But the test works :)
Created attachment 55654 [details] [review] [PASSED QA] Bug 16961: Add Koha::Objects->update In order to update several rows in a single query, we need this new method. Test plan: Confirm that the changes in Objects.t make sense and that the tests pass. Signed-off-by: Hector Castro <hector.hecaxmmx@gmail.com> Test passed successfully Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de> Pushed to master for 16.11, thanks Jonathan! |