Fixed permission check for configuration save [#2130]

This commit is contained in:
Matias Griese
2021-04-23 11:21:41 +03:00
parent 1f529241fc
commit 1b26b4ca7d
3 changed files with 6 additions and 9 deletions

View File

@@ -15,6 +15,7 @@
* Fixed audit vulnerabilities in module dependencies and house cleanup [#2096](https://github.com/getgrav/grav-plugin-admin/issues/2096) * Fixed audit vulnerabilities in module dependencies and house cleanup [#2096](https://github.com/getgrav/grav-plugin-admin/issues/2096)
* Fixed issue preventing Drag & Drop of media files while in Expert Mode [#1927](https://github.com/getgrav/grav-plugin-admin/issues/1927) * Fixed issue preventing Drag & Drop of media files while in Expert Mode [#1927](https://github.com/getgrav/grav-plugin-admin/issues/1927)
* Fixed broken link colors in `preset.css` which was causing issues with tabs and dropdowns * Fixed broken link colors in `preset.css` which was causing issues with tabs and dropdowns
* Fixed permission check for configuration save [#2130](https://github.com/getgrav/grav-plugin-admin/issues/2130)
# v1.10.12 # v1.10.12
## 04/15/2021 ## 04/15/2021

View File

@@ -528,14 +528,9 @@ class AdminBaseController
$permissions = ['admin.super']; $permissions = ['admin.super'];
switch ($type) { switch ($type) {
case 'configuration':
case 'config': case 'config':
case 'system': $type = $this->route ?: 'system';
$permissions[] = 'admin.configuration.system'; $permissions[] = 'admin.configuration.' . $type;
break;
case 'settings':
case 'site':
$permissions[] = 'admin.configuration.site';
break; break;
case 'plugins': case 'plugins':
$permissions[] = 'admin.plugins'; $permissions[] = 'admin.plugins';

View File

@@ -1,9 +1,10 @@
{% extends 'partials/base.html.twig' %} {% extends 'partials/base.html.twig' %}
{% set configurations = admin.configurations(true) %} {% set configurations = admin.configurations(true) %}
{% set config_slug = uri.basename %} {% set config_slug = admin.route %}
{% if config_slug == 'config' %} {% if not config_slug %}
{% set config_slug = configurations|first %} {% set config_slug = configurations|first %}
{% do admin.redirect('config/' ~ config_slug, 302) %}
{% endif %} {% endif %}
{% set isInfo = (config_slug == 'info') %} {% set isInfo = (config_slug == 'info') %}