From 63ec7a4fce6e49b35dd88438514868db58ca33d8 Mon Sep 17 00:00:00 2001 From: Kyle M Hall Date: Fri, 2 Nov 2012 08:11:00 -0400 Subject: [PATCH] Bug 7674 - Followup - Add missing right-hand curly bracket Signed-off-by: Julian Maurice Signed-off-by: Katrin Fischer All tests and perlcritic pass. Tests: Database update - OK, system preferences were added correctly - Version number of updatedatabase was only XXX, changed to 3.11.00.XX OPAC 1) Detail page - OK, no visible change when OpacSeparateHoldings is turned off - OK, switching OpacSeparateHoldingsBranch works correctly - OK, shelf browser opens in correct tab - OK, only one tab shows when logged out INTRANET 2) Detail page - OK, no visible change when SeparateHoldings is turned off - Number of items is missing on top of the tabs Notes: - database update doesn't include bug number - other holdings tab does show when there are 0 items at other branches, it could be hidden in those cases - if there are over 50 items in the holdings or Other holdings tab, the warning and link 'This record has many physical items. Click here to view them all.' is only shown for the first tab. - closing the shelf browser on the second tab goes back to first tab - number ofitems is missing on top of the tabs - there are lots of indentation changes in the second patch, this makes it very hard to see the changes, please provide separate patches in the future Signed-off-by: Jared Camins-Esakov --- catalogue/detail.pl | 14 ++++++++------ installer/data/mysql/updatedatabase.pl | 1 - 2 files changed, 8 insertions(+), 7 deletions(-) diff --git a/catalogue/detail.pl b/catalogue/detail.pl index b20e4c5b73..50d966fd7d 100755 --- a/catalogue/detail.pl +++ b/catalogue/detail.pl @@ -264,14 +264,16 @@ foreach my $item (@items) { $item->{hosttitle} = GetBiblioData($item->{biblionumber})->{title}; } - #count if item is used in analytical bibliorecords - my $countanalytics= GetAnalyticsCount($item->{itemnumber}); - if ($countanalytics > 0){ - $analytics_flag=1; - $item->{countanalytics} = $countanalytics; - } + #count if item is used in analytical bibliorecords + my $countanalytics= GetAnalyticsCount($item->{itemnumber}); + if ($countanalytics > 0){ + $analytics_flag=1; + $item->{countanalytics} = $countanalytics; + } + if (defined($item->{'materials'}) && $item->{'materials'} =~ /\S/){ $materials_flag = 1; + } if ($currentbranch and $currentbranch ne "NO_LIBRARY_SET" and C4::Context->preference('SeparateHoldings')) { diff --git a/installer/data/mysql/updatedatabase.pl b/installer/data/mysql/updatedatabase.pl index 07d9b67768..be1043e5d7 100755 --- a/installer/data/mysql/updatedatabase.pl +++ b/installer/data/mysql/updatedatabase.pl @@ -6112,7 +6112,6 @@ if ( C4::Context->preference("Version") < TransformToNum($DBversion) ) { SetVersion($DBversion); } -<<<<<<< HEAD $DBversion = "3.11.00.005"; if ( C4::Context->preference("Version") < TransformToNum($DBversion) ) { $dbh->do(qq{CREATE TABLE borrower_attribute_types_branches(bat_code VARCHAR(10), b_branchcode VARCHAR(10),FOREIGN KEY (bat_code) REFERENCES borrower_attribute_types(code) ON DELETE CASCADE,FOREIGN KEY (b_branchcode) REFERENCES branches(branchcode) ON DELETE CASCADE ) ENGINE=INNODB DEFAULT CHARSET=utf8;}); -- 2.11.4.GIT