| Subject: | checkbox_group code variable $columns misnamed |
Trying to figure out the math in how checkbox_group() nicely reshapes the displayed table I noticed a typo. Line 2116 of version 3.05 should mention $columns rather than $cols. Code around that line currently reads:
return wantarray ? @elements : join(' ',@elements)
unless defined($columns) || defined($rows);
$rows = 1 if $rows && $rows < 1;
$cols = 1 if $cols && $cols < 1;
return _tableize($rows,$columns,$rowheaders,$colheaders,@elements);
I note also that very similar code in radio_group() is missing the two "= 1" lines and currently reads:
return wantarray ? @elements : join(' ',@elements)
unless defined($columns) || defined($rows);
return _tableize($rows,$columns,$rowheaders,$colheaders,@elements);
Just wondering if the motivation for the two lines really only applies to checkbox_group() and not also to radio_group().