Fix editing a role permissions erase all permissions

Fix userfrosting/UserFrosting#1240
userfrosting · Jan 30, 2024 · 33e5515 · 33e5515
1 parent 316128c
commit 33e5515
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -5,6 +5,9 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/)
 and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html).
 
+## [5.0.2](https://github.com/userfrosting/sprinkle-admin/compare/5.0.1...5.0.2)
+- Fix editing a role permissions erase all permissions - Fix [#1240](https://github.com/userfrosting/UserFrosting/issues/1240)
+
 ## [5.0.1](https://github.com/userfrosting/sprinkle-admin/compare/5.0.0...5.0.1)
 - Update success message when admin resets password for a user - Fix [#852](https://github.com/userfrosting/UserFrosting/issues/852)
 

diff --git a/app/src/Controller/Role/RoleUpdateFieldAction.php b/app/src/Controller/Role/RoleUpdateFieldAction.php
@@ -108,7 +108,7 @@ protected function handle(
         if (isset($put[$fieldName])) {
             $fieldData = $put[$fieldName];
         } elseif ($fieldName === 'permissions') {
-            $fieldData = [];
+            $fieldData = $put['value'];
         } else {
             $e = new MissingRequiredParamException();
             $e->setParam($fieldName);