improved error when controller was not found

OpenPonk · Nov 2, 2024 · ecaf4b1 · ecaf4b1
1 parent 81aa242
commit ecaf4b1
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 4 deletions.
diff --git a/repository/OpenPonk-Core/OPController.class.st b/repository/OpenPonk-Core/OPController.class.st
@@ -274,14 +274,14 @@ OPController >> removeControllerAndDiagramElement [
 { #category : 'destruction' }
 OPController >> removeControllerAndModel [
 
-	self removeDependent.
+	self removeDependentConrollersAndModels.
 	self removeControllerAndDiagramElement.
 	self removeModel.
 	self model: nil
 ]
 
 { #category : 'destruction' }
-OPController >> removeDependent [
+OPController >> removeDependentConrollersAndModels [
 	self dependentControllers
 		do: [ :each | 
 			self diagramController controllers

diff --git a/repository/OpenPonk-Core/OPControllerFactory.class.st b/repository/OpenPonk-Core/OPControllerFactory.class.st
@@ -37,9 +37,14 @@ OPControllerFactory >> initialize [
 
 { #category : 'api - accessing' }
 OPControllerFactory >> newControllerFor: aModel [
+
 	| controller |
-	controller := (registrations detect: [ :each | each isForModel: aModel ]) controllerClass
-		new.
+	controller := (registrations
+		               detect: [ :each | each isForModel: aModel ]
+		               ifNone: [
+			               self error:
+				               'Unable to find controller class for model: '
+				               , aModel asString ]) controllerClass new.
 	controller model: aModel.
 	^ controller
 ]