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

Bernardo Gonzalez Kriegel <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[email protected]

--- Comment #4 from Bernardo Gonzalez Kriegel <[email protected]> ---
Mark, no problem to test but...

1. Test plan is missing
2. I think that we must try (fervently) not to put HTML in scripts
So, this kind of construct

$row_data{repeatable} = "<input id=\"repeatable$i\" type=\"checkbox\" " .
      ( $data->{'repeatable'} ? 'checked="checked" ' : '') .
      "name=\"repeatable$i\">

it's calling for a FaliedQA :(

Perhaps can be replaced by

(in script)
$row_data{repeatable} = {
 id => repeatable$i,
 name => repeatable$i,
 repeatable => $data->{'repeatable'},
}

(in tt)
[% IF ( ...row.repeatable ) %]
[% IF ( ...row.repeatable.repeatable ) %]
<input id="[%- ...row.repeatable.id -%]" type="checkbox" name="[%-
...row.repeatable.name -%]" checked="checked">
[% ELSE %]
<input id="[%- ...row.repeatable.id -%]" type="checkbox" name="[%-
...row.repeatable.name -%]">
[% END %]
[% END %]

or something similar
3. Bug status, it's ready to be signed? Then change that :)

-- 
You are receiving this mail because:
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