Bug 23246: (follow-up) Hardcoding tab position is not enough
authorTomas Cohen Arazi <tomascohen@theke.io>
Mon, 1 Jul 2019 17:13:15 +0000 (14:13 -0300)
committerMartin Renvoize <martin.renvoize@ptfs-europe.com>
Mon, 9 Dec 2019 14:33:34 +0000 (14:33 +0000)
commit6938b0eac577b1f0e3833ccf5cc0c05aa133d879
treec4a22eaf0289478cf5599f7e73d0890094ca3a28
parent142a0235ad9a266aeb5fdc0f0686aabdce43f5e6
Bug 23246: (follow-up) Hardcoding tab position is not enough

There are many conditions that determine the tab number for 'media'.
Translated into variables that hold the tabs count on each condition, it
is clear that '3' was not good enough:
            [%- holdings      = (SeparateHoldings)   ? 2 : 1
                notes         = (MARCNOTES || notes) ? 1 : 0
                acq           = (Koha.Preference('AcquisitionDetails')) ? 1 : 0
                frbr          = (FRBRizeEditions  && XISBNS) ? 1 : 0
                local_covers  = (LocalCoverImages && (localimages || CAN_user_tools_upload_local_cover_images)) ? 1 : 0
                media_tab     = holdings + notes + acq + frbr + local_covers
            -%]

This patch gets rid of the hardcoded value, adds an id to the 'media'
tab (i.e. 'media_tab') and uses JS to get the right index.

To test:
- Play with the different options governing those variables
=> FAIL: 3 is not always the 'media' tab
- Apply this patch
- Repeat your tests
=> SUCCESS: It always picks the 'media' tab
- Sign off :-D

Sponsored-by: American Numismatics Society
Signed-off-by: Maryse Simard <maryse.simard@inlibro.com>
Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt