Merge pull request #2095 from soyuka/fix-subresource_context

soyuka · web-flow · commit 14756cb872ad · 2018-07-25T18:31:43.000+02:00
fix #2094 by improving error message
diff --git a/src/DataProvider/OperationDataProviderTrait.php b/src/DataProvider/OperationDataProviderTrait.php
@@ -99,6 +99,10 @@ private function extractIdentifiers(array $parameters, array $attributes)
             return $id;
         }
 
+        if (!isset($attributes['subresource_context'])) {
+            throw new RuntimeException('Either "item_operation_name" or "collection_operation_name" must be defined, unless the "_api_receive" request attribute is set to false.');
+        }
+
         $identifiers = [];
 
         foreach ($attributes['subresource_context']['identifiers'] as $key => list($id, $resourceClass, $hasIdentifier)) {

Original file line number	Diff line number	Diff line change
`@@ -99,6 +99,10 @@ private function extractIdentifiers(array $parameters, array $attributes)`
`99`	`99`	`return $id;`
`100`	`100`	`}`
`101`	`101`
	`102`	`+ if (!isset($attributes['subresource_context'])) {`
	`103`	`+ throw new RuntimeException('Either "item_operation_name" or "collection_operation_name" must be defined, unless the "_api_receive" request attribute is set to false.');`
	`104`	`+ }`
	`105`	`+`
`102`	`106`	`$identifiers = [];`
`103`	`107`
`104`	`108`	`foreach ($attributes['subresource_context']['identifiers'] as $key => list($id, $resourceClass, $hasIdentifier)) {`