From a72008c3c73ed44fa48886454828f500a9450683 Mon Sep 17 00:00:00 2001 From: Kyle M Hall Date: Mon, 13 Jan 2014 12:39:51 -0500 Subject: [PATCH] Bug 11531: make it possible to edit transport cost for a new libraries If you are using the transport cost matrix, and you create a new library, you cannot edit the fields for this library! Test Plan: 1) Enable the transport cost matrix 2) Set some transport cost values in the matrix 3) Save the matrix 4) Create a new library 5) View the transport cost matrix again 6) Note the uneditable fields for the new branch 7) Apply this patch 8) View the transport cost matrix again 9) Note you can now edit the fields for the new branch Signed-off-by: Jonathan Druart Signed-off-by: Katrin Fischer Works as described, passes all tests and QA script. Signed-off-by: Galen Charlton (cherry picked from commit 4edcec3d5812b159b7012c43a04577bfd7e462bc) Signed-off-by: Fridolin Somers Signed-off-by: Kyle M Hall (cherry picked from commit a143dec8a0b8490ff5ebae3e72599a2e94a0a8f7) --- admin/transport-cost-matrix.pl | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/admin/transport-cost-matrix.pl b/admin/transport-cost-matrix.pl index 25d39118d9..faff6ec8ca 100755 --- a/admin/transport-cost-matrix.pl +++ b/admin/transport-cost-matrix.pl @@ -91,8 +91,12 @@ foreach my $branchfrom ( @branchloop ) { if ( my $cell = $cost_matrix->{$tocode}{$fromcode} ) { $from_to_input_def{value} = $cell->{cost}; $from_to_input_def{disabled} = 1 if $cell->{disable_transfer}; + } else { + # matrix has been previously initialized, but a branch referenced here was created afterward. + $from_to_input_def{disabled} = 1; } } else { + # First time initializing the matrix $from_to_input_def{disabled} = 1; } } -- 2.39.5