Merge pull request #17 from robbieaverill/pulls/1.0/user-warning-on-c…

…onfig-update API Throw user warning when using Config::inst()->update()
silverstripe · Jul 17, 2017 · a0b4b8a · a0b4b8a
2 parents d49acbc + aa25663
commit a0b4b8a
Showing 1 changed file with 11 additions and 0 deletions.
diff --git a/src/Collections/CachedConfigCollection.php b/src/Collections/CachedConfigCollection.php
@@ -204,4 +204,15 @@ public function setMiddlewares($middlewares)
             "Please apply middleware to collection factory via setCollectionCreator()"
         );
     }
+
+    /**
+     * @deprecated 4.0...5.0 Please use YAML configuration, ::modify()->set() or ::modify()->merge()
+     * @throws BadMethodCallException
+     */
+    public function update($class, $name, $value)
+    {
+        throw new BadMethodCallException(
+            'Config::inst()->update() is deprecated. Please use YAML configuration, Config::modify()->merge() or ->set()'
+        );
+    }
 }