From 92598b86e5feaf82000d35b584aa3f54ac6b138f Mon Sep 17 00:00:00 2001
From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Date: Wed, 20 Jan 2016 11:21:12 +0000
Subject: [PATCH] Bug 15629: Koha::Libraries - Remove GetBranchesInCategory (1)

For consistency, the branchcodes method of Koha::LibraryCategory should be
named libraries.
At the moment, there is only one occurrence, in admin/branches.pl
---
 Koha/LibraryCategory.pm         | 2 +-
 admin/branches.pl               | 2 +-
 t/db_dependent/Koha/Libraries.t | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/Koha/LibraryCategory.pm b/Koha/LibraryCategory.pm
index f675d89..d6b7766 100644
--- a/Koha/LibraryCategory.pm
+++ b/Koha/LibraryCategory.pm
@@ -41,7 +41,7 @@ sub new {
     return $self->SUPER::new( $params );
 }
 
-sub branchcodes{
+sub libraries{
     my ( $self, $params ) = @_;
     # TODO  This should return Koha::Libraries
     return $self->{_result}->branchcodes( $params );
diff --git a/admin/branches.pl b/admin/branches.pl
index 918b10c..a655fc6 100755
--- a/admin/branches.pl
+++ b/admin/branches.pl
@@ -194,7 +194,7 @@ if ( $op eq 'add_form' ) {
     $op = 'list';
 } elsif ( $op eq 'delete_confirm_category' ) {
     my $category = Koha::LibraryCategories->find($categorycode);
-    if ( my $libraries_count = scalar( $category->branchcodes ) ) {
+    if ( my $libraries_count = scalar( $category->libraries ) ) {
         push @messages,
           { type => 'error',
             code => 'cannot_delete_category',
diff --git a/t/db_dependent/Koha/Libraries.t b/t/db_dependent/Koha/Libraries.t
index 8e21567..8126b8c 100644
--- a/t/db_dependent/Koha/Libraries.t
+++ b/t/db_dependent/Koha/Libraries.t
@@ -77,7 +77,7 @@ $retrieved_library_1->update_categories( [ $new_category_2, $new_category_3 ] );
 is( Koha::Libraries->find( $new_library_1->branchcode )->get_categories->count, 2, '2 libraries should have been linked to the category 2' );
 
 my $retrieved_category_2 = Koha::LibraryCategories->find( $new_category_2->categorycode );
-is( $retrieved_category_2->branchcodes->count, 2, '2 libraries should have been linked to the category_2' );
+is( $retrieved_category_2->libraries->count, 2, '2 libraries should have been linked to the category_2' );
 is( $retrieved_category_2->categorycode, uc('my_cc_2'), 'The Koha::LibraryCategory constructor should have upercased the categorycode' );
 
 $retrieved_library_1->delete;
-- 
2.1.0