From 130b282938a0616a225fb7772bd158c6225719c0 Mon Sep 17 00:00:00 2001 From: ByWater Staff Date: Mon, 13 Jan 2014 12:39:51 -0500 Subject: [PATCH] [PASSED QA] Bug 11531 - Cannot add new libraries to the transport cost if previously used. 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. --- 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 25d3911..faff6ec 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; } } -- 1.8.3.2