Bug 7674 - Followup - Add missing right-hand curly bracket
authorKyle M Hall <kyle@bywatersolutions.com>
Fri, 2 Nov 2012 12:11:00 +0000 (08:11 -0400)
committerJared Camins-Esakov <jcamins@cpbibliography.com>
Mon, 17 Dec 2012 14:41:20 +0000 (09:41 -0500)
commit63ec7a4fce6e49b35dd88438514868db58ca33d8
tree6ca469e10d957a7a99c887f7626039611a9e5aa1
parentb39c762ddeec0ee3e99595937b35eae052abadd7
Bug 7674 - Followup - Add missing right-hand curly bracket

Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>

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 <jcamins@cpbibliography.com>
catalogue/detail.pl
installer/data/mysql/updatedatabase.pl