Fixed #3832

craftcms · Feb 13, 2019 · 8070026 · 8070026
1 parent e1b67f2
commit 8070026
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 6 deletions.
diff --git a/CHANGELOG-v3.md b/CHANGELOG-v3.md
@@ -4,6 +4,7 @@
 
 ### Fixed
 - Fixed a bug where the `users/save-user` action wasn’t deleting user photos properly.
+- Fixed a bug where changes to Matrix block type fields’ settings weren’t always saving. ([#3832](https://github.com/craftcms/cms/issues/3832))
 
 ## 3.1.9.1 - 2019-02-12
 

diff --git a/src/services/ProjectConfig.php b/src/services/ProjectConfig.php
@@ -326,12 +326,7 @@ public function get(string $path = null, $getFromYaml = false)
      */
     public function set(string $path, $value)
     {
-        // Make sure it's actually changing
-        if ($value === $this->get($path)) {
-            return;
-        }
-
-        if ($this->readOnly) {
+        if ($this->readOnly && $value !== $this->get($path)) {
             throw new NotSupportedException('Changes to the project config are not possible while in read-only mode.');
         }