Bugzilla – Attachment 97503 Details for
Bug 24292
Upgrade failed, from 19.05 to 19.11 on Ubuntu 19.10: Can't load Unaccent.so
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 24296: Update stockrotation process to use branchtransfers.trigger
Bug-24296-Update-stockrotation-process-to-use-bran.patch (text/plain), 5.19 KB, created by
Martin Renvoize (ashimema)
on 2020-01-17 11:40:44 UTC
(
hide
)
Description:
Bug 24296: Update stockrotation process to use branchtransfers.trigger
Filename:
MIME Type:
Creator:
Martin Renvoize (ashimema)
Created:
2020-01-17 11:40:44 UTC
Size:
5.19 KB
patch
obsolete
>From 9c77ff6ee500d5bac8b3d2cec0458a16b5e9e305 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.trigger > >This patch updates all occurences of stockrotation related lookups of >branchtransfers status from using the 'comments' field to using the new >'trigger' 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. > >https://bugs.koha-community.org/show_bug.cgi?id=24292 >--- > 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..9ed892a5cc 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" }, >+ { 'trigger' => "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, >+ 'trigger' => "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" >+ 'trigger' => "StockrotationAdvance" > }); > > if ( $self->indemand && !$self->fresh ) { >diff --git a/t/db_dependent/StockRotationItems.t b/t/db_dependent/StockRotationItems.t >index 2d9bf89a39..583ae0375b 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", >+ 'trigger' => "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", >+ 'trigger' => "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", >+ 'trigger' => "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", >+ 'trigger' => "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", >+ 'trigger' => "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", >+ 'trigger' => "StockrotationAdvance", > })->store; > is($dbitem->investigate->{reason}, 'repatriation', > "Item advances, but not at stage branch."); >-- >2.20.1
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 24292
:
97502
|
97503