Bug 16717: Improve variable naming
authorJonathan Druart <jonathan.druart@bugs.koha-community.org>
Tue, 14 Jun 2016 10:18:32 +0000 (11:18 +0100)
committerKyle M Hall <kyle@bywatersolutions.com>
Fri, 17 Jun 2016 14:45:35 +0000 (14:45 +0000)
$category was a categorycode, to help the readability it's better to
name it $categorycode or assign a Category to $category

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
t/db_dependent/Holds.t

index b368481..f9ddc46 100755 (executable)
@@ -34,7 +34,7 @@ my $dbh     = C4::Context->dbh;
 my $branch_1 = $builder->build({ source => 'Branch' })->{ branchcode };
 my $branch_2 = $builder->build({ source => 'Branch' })->{ branchcode };
 
-my $category = $builder->build({ source => 'Category' })->{categorycode};
+my $category = $builder->build({ source => 'Category' });
 
 my $borrowers_count = 5;
 
@@ -60,7 +60,7 @@ foreach (1..$borrowers_count) {
     my $borrowernumber = AddMember(
         firstname =>  'my firstname',
         surname => 'my surname ' . $_,
-        categorycode => $category,
+        categorycode => $category->{categorycode},
         branchcode => $branch_1,
     );
     push @borrowernumbers, $borrowernumber;