Error should be collected at Schema->errors[] and include in GraphQL introspec.

Author: olragon

includes/graphql_api_page_callback.inc

@@ -32,7 +32,7 @@ function graphql_api_page_callback() {
 
   $is_introspec = FALSE;
   try {
-    if ($is_introspec && $cache = cache_get('graphql_api_introspec')) {
+    if ($is_introspec && $cache = cache_get('graphql_api_introspec') && empty($_GET['nocache'])) {
       $result = $cache->data;
     } else {
       // Define your schema:
@@ -47,6 +47,13 @@ function graphql_api_page_callback() {
         $variableValues,
         $operationName
       );
+      
+      if (!empty($schemaBuilder->errors)) {
+        $result['errors'] = isset($result['errors']) ? $result['errors'] : [];
+        $result['errors'] += $schemaBuilder->errors;
+        $result['errors'] = array_unique($result['errors']);
+      }
+
 
       if ($is_introspec) {
         cache_set('graphql_api_introspec', $result);

src/Schema.php

@@ -48,7 +48,7 @@ class Schema {
    *
    * @var array
    */
-  private $errors = [];
+  public $errors = [];
 
 
   private $allArgTypes = [];
@@ -314,7 +314,7 @@ public function entityToGqlQueryArg($entity_type, $bundle = NULL) {
         if ($field_info['type'] instanceof ListOfType) {
           if (isset($this->objectTypes[$field])) {
             $field_info['type'] = $this->objectTypes[$field];
-          }
+          }     
           else {
             if (!($field_info['type']->ofType instanceof \Closure) && !($field_info['type']->ofType instanceof InterfaceType)) {
               $args[$field] = [
@@ -323,10 +323,6 @@ public function entityToGqlQueryArg($entity_type, $bundle = NULL) {
               ];
               $this->allArgTypes[get_class($field_info['type']->ofType)] = get_class($field_info['type']->ofType);
             }
-            else {
-              $type = get_class($field_info['type']->ofType);
-              $this->addError("Type {$type} cannot be used as args for {$field}");
-            }
           }
         }
 
@@ -518,8 +514,9 @@ public function getFields($entity_type, $bundle = '') {
         ]) : Type::string();
 
         if (!$fieldType) {
-          dump($info);
-          die("Cannot detect fieldType for {$entity_type} {$property}");
+          // dump($info);
+          // die("Cannot detect fieldType for {$entity_type} {$property}");
+          $this->addError("Cannot detect fieldType for {$entity_type} {$property}");
           continue;
         }
 
@@ -563,9 +560,9 @@ public function getFields($entity_type, $bundle = '') {
             ]);
 
             if (!$fieldType) {
-              dump($fieldType, $field_info);
-              die("Cannot detect field type of {$field}");
-              $this->addError("Cannot detect field type of {$field}");
+              // dump($fieldType, $field_info);
+              // die("Cannot detect field type of {$field}");
+              $this->addError("Cannot convert Drupal field type '{$field}' -> GrahpQL field type.");
               continue;
             }
 
@@ -741,9 +738,9 @@ public function drupalToGqlFieldType($drupalType, $context = []) {
     }
 
     if (!$type) {
-      dump($context);
-      die("Cannot convert {$drupalType} to GraphQL type.");
-      $this->addError("Cannot convert {$drupalType} to GraphQL type. " . print_r($context, TRUE));
+      // dump($context, debug_backtrace());
+      // die("Cannot convert {$drupalType} to GraphQL type.");
+      $this->addError("Cannot convert Drupal property type '{$drupalType}' -> GraphQL type. Please register this type with hook_graphql_api_info()");
     }
 
     return $type;