jacl2db_admin should display the subject more userfriendly

[foxmask]

Instead of having acl.group.create in the table of the right we should read "Create Group" which is more userfriendly to understand clearly what is the goal of this right.

thus to do this i propose a little change in attachment.

[laurentj]

$subject = jDao::get('jelix~jacl2subject',$p)->get($rec->id_aclsbj);

very very ugly to do such things into a loop, whereas we could retrieve it with the same..

and no, the id in the array must not be replaced by the subject label, since a label is not always unique

[laurentj]

sorry, a unfinished sentence : "with the same query"

[laurentj]

and I think it is better to keep the id in the table (it is important for developers or webmaster). The label should be just an additional description.

[foxmask]

So, we can not use label_key nor display something like "Create Group" and keep acl.create.group ?

If yes ; i think this ticket can be closed as wontfix/invalid.

On my side i thought "Create Group" était plus parlant than "acl.create.group" for a user that dont understand anything to the terminology of ACL, but could easyly understand what will happen if he checks "Create Group".

[laurentj]

je vais le faire en français, parce apparemment je me suis mal exprimé. Je n'ai pas dit que c'était impossible, j'ai dit, en résumant, que ton patch fait les choses de façon très très mal (un select dans un select, très beurk, et utiliser le label comme clé dans le tableau php est très mauvais), et qu'il fallait afficher les deux colonnes, id et label.

[foxmask]

J'ai fait plusieurs essais mais rien ne me plait.

Limite rajouter un foreigntable dans la DAO mais je trouve pas

donc si Surfoo qui avait eu le même besoin que moi veut s'y coller ...

[laurentj]

This feature have been fixed in jelix 1.3.