The permissions 'export configuration'
and 'import configuration'
are used in config.routing.yml
but they are not implemented using hook_permission()
.
The only permission defined in config.module currently is 'synchronize configuration'
. However, in the yml file config.routing.yml
these 3 permissions are used:
'synchronize configuration'
'export configuration'
'import configuration'
The 'export configuration'
permission is used for following callbacks:
admin/config/development/export
admin/config/development/export-download
The 'import configuration'
permission is used for following callback:
admin/config/development/import
I think we should either:
1) add the 'export configuration' and 'import configuration' permissions to config.module, or
2) use the 'synchronize configuration' permission for both export and import menu callbacks (change in config.routing.yml)
Comment | File | Size | Author |
---|---|---|---|
#1 | drupal-add-permissions-export-import-configuration-2017859-1.patch | 618 bytes | valdo |
Comments
Comment #1
valdo CreditAttribution: valdo commentedPatch for option 1 (add the permissions) is attached.
Comment #2
amateescu CreditAttribution: amateescu commentedYeah, clear signs of missing tests. Anyway, a test that uses this permission is being added in #2017019: Configuration export action path is broken when Drupal is installed in a subfolder so it's probably ok to get this one in before.
Comment #3
alexpottNice find and I'm looking forward to the test in #2017019: Configuration export action path is broken when Drupal is installed in a subfolder :)
Committed d888189 and pushed to 8.x. Thanks!