Bug 17736: [QA Follow-up] Solve warning from qa tools on Holds.t
authorMarcel de Rooy <m.de.rooy@rijksmuseum.nl>
Fri, 6 Jan 2017 10:45:17 +0000 (11:45 +0100)
committerKyle M Hall <kyle@bywatersolutions.com>
Fri, 31 Mar 2017 12:02:15 +0000 (12:02 +0000)
FAIL   t/db_dependent/Holds.t
"my" variable $hold masks earlier declaration in same scope

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
t/db_dependent/Holds.t

index b58753f..5825f7c 100755 (executable)
@@ -204,7 +204,7 @@ ok( $reserve->{'reserve_id'} eq $reserve2->{'reserve_id'}, "Test GetReserveInfo(
 
 
 $holds = $biblio->holds;
-my $hold = $holds->next;
+$hold = $holds->next;
 AlterPriority( 'top', $hold->reserve_id );
 $reserve = GetReserve( $reserve->{'reserve_id'} );
 is( $reserve->{'priority'}, '1', "Test AlterPriority(), move to top" );