]> git.koha-community.org Git - koha.git/commit
Bug 12176: [QA Follow-up] Small additem adjustments
authorMarcel de Rooy <m.de.rooy@rijksmuseum.nl>
Wed, 20 May 2015 12:56:48 +0000 (14:56 +0200)
committerChris Cormack <chris@bigballofwax.co.nz>
Wed, 3 Jun 2015 07:35:34 +0000 (19:35 +1200)
commitb85094aa10c9c6c3bc9d5bc83d9bfd04af4c7c0d
treee54f1e49358f629a4d1651f0d2e74bc9b51e3e52
parent2b9ef94685ea014a014fdc994f8def827040e6f2
Bug 12176: [QA Follow-up] Small additem adjustments

Adjusting a few small things and making qa tools happy:

[1] Remove .hidden after [% avalue %] in additem.tt. (Typo)
[2] A closer look revealed that mv.avalue is useless too. An input element
    does not contain any content. Putting it after the hidden element
    in a non-visible context has no meaning.
[3] Change handling of select attributes readonly and disabled.
[4] Remove unused variable $attributes_no_value_textarea
[5] Removed a comment with TODO referring to this report.
[6] Moving a duplicated TT variable (mv.javascript) outside IF statement.
[7] And finally could not resist this one: Moving strings Tag editor and
    No popup from script to template.  Plugins++
    NOTE: Most item plugins redirect click to focus. In that case there is
    no popup, but unfortunately the text Tag editor comes up.
    When you remove or rename function Click, No popup comes up. So it works.

Sorry that this small list kept growing :)

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com>
Signed-off-by: Jonathan Druart <jonathan.druart@koha-community.org>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
(cherry picked from commit 50d2b712883808a36b1d03ac770aed470c8d19d1)
Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
cataloguing/additem.pl
koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/additem.tt