#141 Changing XoopsGroupPermForm->XoopsGroupFormCheckBox->render

open
nobody
Core (214)
5
2012-09-25
2005-03-05
eecsman
No

This is a useabilty issue more than anything else. When
a user goes to set permissions on under a category with
no items under it, all they see is an 'All' checkbox.
It is easy to assume that cheking all will have set
permissions for a group. But it hasn't as there were no
items under that category to set. It seems like a
common confusion and though it is easy to understand
once you realize that You need something to set
permissions on before you can set them it really is
easy to become confused as a user since you are going
to see a check box for 'All'

suggested fix

XoopsGroupFormCheckBox->render()
around say line:274

CHANGE:
$ret .= _ALL." ";

TO:
if (isset($option_ids_str)){
$ret .= _ALL." ";
}
else {
$ret .= _NOCAT;
}

It eliminates the checkbox when it is not needed and
helps useabilty.

IMHO

Discussion