Attachment 144206 Details for Bug 32335 – Bug 32335: Allow stock rotation items to be advanced when in transit

[patch] Bug 32335: Allow stock rotation items to be advanced when in transit

Bug-32335-Allow-stock-rotation-items-to-be-advance.patch (text/plain), 2.34 KB, created by Martin Renvoize (ashimema) on 2022-11-23 13:37:34 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Martin Renvoize (ashimema)

Created: 2022-11-23 13:37:34 UTC

Size: 2.34 KB

patch

obsolete

>From ed2280ca65fba9ef416f2c4f06244cc01ef74649 Mon Sep 17 00:00:00 2001
>From: Nick Clemens <nick@bywatersolutions.com>
>Date: Wed, 23 Nov 2022 13:24:12 +0000
>Subject: [PATCH] Bug 32335: Allow stock rotation items to be advanced when in
> transit
>
>This patch simply removes the disabling of the 'Move to next stage' button
>and adds a title to let the librarian know the item is in transit
>
>To test:
>1 - Setup a rota with multiple stages
>2 - Add an item
>3 - Advance to next stage
>4 - Observe 'Move to next stage' button is disabled
>5 - Check item in at destination
>6 - Reload rota - observe 'Move to next stage' is enabled
>7 - Apply patch
>8 - Click 'Move to next stage'
>9 - Note button is not disabled
>10 - Hover and confirm note about transit
>11 - Click 'Move to next stage'
>12 - Checkin item at wrong branch, confirm it is correctly directed to correct stage
>
>Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
>---
> koha-tmpl/intranet-tmpl/prog/en/modules/tools/stockrotation.tt | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/stockrotation.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/stockrotation.tt
>index 1db4229438..a1807ae20e 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/stockrotation.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/stockrotation.tt
>@@ -483,7 +483,7 @@
>                                                 [% IF !in_transit && stages.size > 1 %]
>                                                     <a class="btn btn-default btn-xs" href="?op=move_to_next_stage&amp;rota_id=[% rota.id | uri %]&amp;item_id=[% sritem.id | uri %]&amp;stage_id=[% sritem.stage.stage_id | uri %]">
>                                                 [% ELSE %]
>-                                                    <a class="btn btn-default btn-xs" disabled>
>+                                                    <a class="btn btn-default btn-xs" href="?op=move_to_next_stage&amp;rota_id=[% rota.id | uri %]&amp;item_id=[% sritem.id | uri %]&amp;stage_id=[% sritem.stage.stage_id | uri %]" title="Item is in transit, it will be directed to new stage when checked in">
>                                                 [% END %]
>                                                     <i class="fa fa-arrow-right"></i>
>                                                     Move to next stage
>-- 
>2.20.1

Actions: View | Diff | Splinter Review

Attachments on bug 32335: 144204 | 144206 | 148747 | 148748 | 151491 | 155322 | 155323 | 155324