search

mudrd8mz / moodle-tool_pluginskel

Generator of Moodle plugins skeletons
https://moodle.org/plugins/tool_pluginskel
Other
51 stars 46 forks source link

Check for missing capabilities title strings #68

Closed alexandru-elisei closed 8 years ago

mudrd8mz commented 8 years ago

Sorry, I don't think this is a good approach. Forcing the developer to be aware of the need to provide that string manually is not friendly. We should ask for the capability title as a part of the capability declaration. If it is provided, the string should be created automatically. Something like this:

capabilities:
    - name: viewsomething
      title: View something
      captype: view
      contextlevel: CONTEXT_MODULE
      archetypes:
        - role: student
          permission: CAP_ALLOW
        - role: editingteacher
          permission: CAP_ALLOW
      clonepermissionsfrom: moodle/course:view

Note the added title there which would automatically lead to generating the string.

alexandru-elisei commented 8 years ago

Oh, that's a very good idea. I'm going to implement it.

mudrd8mz commented 8 years ago

Good. Sorry if I did not describe the idea clearly at the start.