Generate jACL2 rights with the createdao command

[mike]

It would be so great if we could generate ACL2 rights at the same time we create the CRUD! ACL2 rights have to match CRUD methods so we must have these rights :

• list
• view
• create
• modify
• delete

And a special one for all others methods not specified in the previous list. I'm writing the patch right now !

[mike]

It works. This the very first version in order to discuss.

[mike]

Same patch with minor improvements

[mike]

add a module listener to generate CRUD menu, events.xml is missing.

[mike]

A lot of improvements, also create or update events.xml if needed.

[mike]

Last version, correct a typo mistake on -force option

[mike]

For the option "-force" we have to update some commands and methods used by createdacrud. Maybe we have to look further later ?!

[laurentj]

+ menu.item.crud = CRUD 

I prefer a better readable label, like "Data management" or something like that. For most of users, "CRUD" doesn't mean something.

    public  $allowed_parameters=array('module'=>true, 'table'=>true, 'ctrlname'=>false, 'acl2'=>false);

I don't know why you added the acl2 parameter, since you don't use it in the command.

$options['-force'] = ''; 

You give this option to createdao and createform, but these commands don't support it.

Same thing for calls of createFile, it doesn't support a "force" parameter.

Did you missed something in your patch ? Or perhaps you could add this support with an other ticket+patch.

[mike]

Replying to laurentj:

+ menu.item.crud = CRUD 

I prefer a better readable label, like "Data management" or something like that. For most of users, "CRUD" doesn't mean something.

Yes it's better you're right.

    public  $allowed_parameters=array('module'=>true, 'table'=>true, 'ctrlname'=>false, 'acl2'=>false);

I don't know why you added the acl2 parameter, since you don't use it in the command.

Actually I don't remember, I have to check this.

$options['-force'] = ''; 

You give this option to createdao and createform, but these commands don't support it.

Same thing for calls of createFile, it doesn't support a "force" parameter.

Did you missed something in your patch ? Or perhaps you could add this support with an other ticket+patch.

In fact the "-force" parameter was discussed on IRC, and it's a good solution to overwrite files already created. But I didn't develop anything right now to do this. If you want I can create the ticket and someone can do this later (if it's not me)

[laurentj]

In fact the "-force" parameter was discussed on IRC, and it's a good solution to overwrite files already created.

Yes I agree, it can be useful. But don't provide an uncomplete implementation here :-) Remove this parameter in this patch, and fill a new ticket to bring the support of -force on some commands.

[laurentj]

any news ?

[laurentj]

Here is an improvement of the patch. However, I'm not sure all is perfect. And it needs to be tested. Too late for 1.2.

[laurentj]

Patch has been improved, tested. ​https://bitbucket.org/jelix/jelix-trunk/changeset/0ea7a9f4fc35