Skip to content

Fix and simplify permissions

tschuettler requested to merge (removed):130-fix-permissions into master

The current xml does not meet the specs: https://docs.civicrm.org/dev/en/latest/framework/routing/#access_arguments But I guess it just used the last element found.

So in theory it should be <access_arguments>access CiviCRM,administer CiviCRM</access_arguments>, but I would say that requiring access CiviCRM has no additional benefit. CiviCRM admin menu also only require administer CiviCRM.

The two additions were accidentally?! missed when adding stricter permissions.

Closes #130 (closed)

Merge request reports