http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8587

Jonathan Druart <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Signed Off                  |Failed QA
                 CC|                            |[email protected]
                   |                            |m

--- Comment #4 from Jonathan Druart <[email protected]> ---
QA comments:

Hi Maxime, 

1/+  `seriallifespan` TINYINT(1) DEFAULT -1
You cannot store '10' into a TINYINT(1)

2/ 3 loops can be created  to avoid a list of 10 lines:
[% FOREACH i IN [ 2 .. 10 ] %]
  <option value="[% i %]" [% IF ( seriallifespan == [% i %] )
%]selected="selected"[% END %]>[% i %] years</option>
[% END %]

Furthermore you cannot write this line :-/ (see Bug 8407)

And:
[% IF seriallifespan == -1 %]
  <span>Immortal</span>
[% ELSIF ( seriallifespan == 1 ) %]
  <span>1 year</span>
[% ELSIF ( seriallifespan >= 10 ) %]
  [% FOREACH i IN [ 2 .. 10 ] %]
    <span>[% seriallifespan %] years</span>
  [% END %]
[% END %]

Something like that, I didn't test.

3/ Fonctionnaly what is the goal of this field? Indeed, you don't use this
field, just store and display.

Failed QA for 1 and 2

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to