From 82a660dadcc48430202ebb36f6b2120ea934065a Mon Sep 17 00:00:00 2001 From: Ingo Schommer Date: Thu, 13 Aug 2020 20:48:30 +1200 Subject: [PATCH] Require silverstripe/registry Those class_exists() check don't work with class manifests. Check the logic in ClassManifestVisitor - it just parses PHP until it finds class definitions. Which means you'll have those files in the manifest, but when dev/build tries to instantiate them it fails with an InjectorNotFoundException. We could argue that this should be fixed in core, but I'm not sure that this style of conditional classes needs to be supported. --- composer.json | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/composer.json b/composer.json index 717bb13..bfeb44b 100644 --- a/composer.json +++ b/composer.json @@ -16,7 +16,8 @@ "silverstripe/cms": "~4.0", "guzzlehttp/guzzle": "~6.0", "fzaninotto/faker": "^1.7", - "silverstripe/vendor-plugin": "^1.0" + "silverstripe/vendor-plugin": "^1.0", + "silverstripe/registry": "~2" }, "extra": { "expose": [