Bugzilla – Attachment 98025 Details for
Bug 24296
Move stock rotation transfer triggers from `comments` to `reason`
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 24296: Update stockrotation process to use branchtransfers.reason
Bug-24296-Update-stockrotation-process-to-use-bran.patch (text/plain), 5.25 KB, created by
Jonathan Druart
on 2020-01-28 12:56:21 UTC
(
hide
)
Description:
Bug 24296: Update stockrotation process to use branchtransfers.reason
Filename:
MIME Type:
Creator:
Jonathan Druart
Created:
2020-01-28 12:56:21 UTC
Size:
5.25 KB
patch
obsolete
>From 05863251e283ab276487b71b1e58955d2aee86ad Mon Sep 17 00:00:00 2001 >From: Martin Renvoize <martin.renvoize@ptfs-europe.com> >Date: Thu, 16 Jan 2020 09:52:20 +0000 >Subject: [PATCH] Bug 24296: Update stockrotation process to use > branchtransfers.reason > >This patch updates all occurences of stockrotation related lookups of >branchtransfers status from using the 'comments' field to using the new >'reason' field. > >Test plan: >1) Work through the process of creating a stock rotation plan and adding > items to the plan.. no errors should be encoutered during this process. >2) Run the stockrotation cronjob to move some items on.. no errors > should be encountered. > >Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> > >Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> >--- > Koha/StockRotationItem.pm | 7 ++++--- > t/db_dependent/StockRotationItems.t | 12 ++++++------ > 2 files changed, 10 insertions(+), 9 deletions(-) > >diff --git a/Koha/StockRotationItem.pm b/Koha/StockRotationItem.pm >index e451bb08db..41753dc020 100644 >--- a/Koha/StockRotationItem.pm >+++ b/Koha/StockRotationItem.pm >@@ -119,7 +119,7 @@ sub needs_advancing { > return 0 if $self->itemnumber->get_transfer; # intransfer: don't advance. > return 1 if $self->fresh; # Just on rota: advance. > my $completed = $self->itemnumber->_result->branchtransfers->search( >- { 'comments' => "StockrotationAdvance" }, >+ { 'reason' => "StockrotationAdvance" }, > { order_by => { -desc => 'datearrived' } } > ); > # Do maths on whether we need to be moved on. >@@ -156,7 +156,8 @@ sub repatriate { > 'frombranch' => $self->itemnumber->holdingbranch, > 'tobranch' => $self->stage->branchcode_id, > 'datesent' => DateTime->now, >- 'comments' => $msg || "StockrotationRepatriation", >+ 'comments' => $msg, >+ 'reason' => "StockrotationRepatriation" > })->store; > $self->itemnumber->homebranch($self->stage->branchcode_id)->store; > return $transfer_stored; >@@ -185,7 +186,7 @@ sub advance { > 'itemnumber' => $self->itemnumber_id, > 'frombranch' => $item->holdingbranch, > 'datesent' => DateTime->now, >- 'comments' => "StockrotationAdvance" >+ 'reason' => "StockrotationAdvance" > }); > > if ( $self->indemand && !$self->fresh ) { >diff --git a/t/db_dependent/StockRotationItems.t b/t/db_dependent/StockRotationItems.t >index 2d9bf89a39..3718654443 100644 >--- a/t/db_dependent/StockRotationItems.t >+++ b/t/db_dependent/StockRotationItems.t >@@ -175,7 +175,7 @@ subtest "Tests for needs_advancing." => sub { > 'tobranch' => $dbitem->stage->branchcode_id, > 'datesent' => DateTime->now, > 'datearrived' => undef, >- 'comments' => "StockrotationAdvance", >+ 'reason' => "StockrotationAdvance", > })->store; > > # Test item will not be advanced if in transit. >@@ -307,7 +307,7 @@ subtest "Tests for investigate (singular)." => sub { > 'tobranch' => $dbitem->itemnumber->homebranch, > 'datesent' => DateTime->now, > 'datearrived' => DateTime->now, >- 'comments' => "StockrotationAdvance", >+ 'reason' => "StockrotationAdvance", > })->store; > is($dbitem->investigate->{reason}, 'repatriation', "older item repatriates."); > >@@ -323,7 +323,7 @@ subtest "Tests for investigate (singular)." => sub { > 'tobranch' => $dbitem->stage->branchcode_id, > 'datesent' => DateTime->now, > 'datearrived' => DateTime->now, >- 'comments' => "StockrotationAdvance", >+ 'reason' => "StockrotationAdvance", > })->store; > $dbitem->itemnumber->homebranch($dbitem->stage->branchcode_id)->store; > $dbitem->itemnumber->holdingbranch($dbitem->stage->branchcode_id)->store; >@@ -342,7 +342,7 @@ subtest "Tests for investigate (singular)." => sub { > 'tobranch' => $dbitem->stage->branchcode_id, > 'datesent' => DateTime->now - $sent_duration, > 'datearrived' => DateTime->now - $arrived_duration, >- 'comments' => "StockrotationAdvance", >+ 'reason' => "StockrotationAdvance", > })->store; > $dbitem->itemnumber->homebranch($dbitem->stage->branchcode_id)->store; > $dbitem->itemnumber->holdingbranch($dbitem->stage->branchcode_id)->store; >@@ -362,7 +362,7 @@ subtest "Tests for investigate (singular)." => sub { > 'tobranch' => $dbitem->stage->branchcode_id, > 'datesent' => DateTime->now - $sent_duration, > 'datearrived' => DateTime->now - $arrived_duration, >- 'comments' => "StockrotationAdvance", >+ 'reason' => "StockrotationAdvance", > })->store; > $dbitem->itemnumber->homebranch($dbitem->stage->branchcode_id)->store; > $dbitem->itemnumber->holdingbranch($dbitem->stage->branchcode_id)->store; >@@ -382,7 +382,7 @@ subtest "Tests for investigate (singular)." => sub { > 'tobranch' => $dbitem->stage->branchcode_id, > 'datesent' => DateTime->now - $sent_duration, > 'datearrived' => DateTime->now - $arrived_duration, >- 'comments' => "StockrotationAdvance", >+ 'reason' => "StockrotationAdvance", > })->store; > is($dbitem->investigate->{reason}, 'repatriation', > "Item advances, but not at stage branch."); >-- >2.11.0
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 24296
:
97433
|
97434
|
97504
|
97505
|
97517
|
97518
|
97533
|
97534
|
98024
| 98025