Issue #2927748 by mkalkbrenner, mstiem, drunken monkey: Fixed Views field handler for multiple processor-defined fields returned by the server.

Author: mkalkbrenner

Diff for: CHANGELOG.txt

@@ -1,5 +1,7 @@
 Search API 1.x, dev (xxxx-xx-xx):
 ---------------------------------
+- #2927748 by mkalkbrenner, mstiem, drunken monkey: Fixed Views field handler
+  for multiple processor-defined fields returned by the server.
 - #2928944 by drunken monkey: Removed assert() calls with string parameters.
 - #2923976 by Martijn Houtman, drunken monkey: Fixed facets with random sorting
   on Database backend.

Diff for: src/Plugin/views/query/SearchApiQuery.php

@@ -560,7 +560,6 @@ protected function addResults(array $results, ViewExecutable $view) {
       foreach ($result->getFields(FALSE) as $field_id => $field) {
         if ($field->getValues()) {
           $path = $field->getCombinedPropertyPath();
-          $values[$path] = $field->getValues();
           try {
             $property = $field->getDataDefinition();
             // For configurable processor-defined properties, our Views field
@@ -570,13 +569,13 @@ protected function addResults(array $results, ViewExecutable $view) {
             // correctly.
             if ($property instanceof ConfigurablePropertyInterface) {
               $path .= '|' . $field_id;
-              $values[$path] = $field->getValues();
             }
           }
           catch (SearchApiException $e) {
             // If we're not able to retrieve the data definition at this point,
             // it doesn't really matter.
           }
+          $values[$path] = $field->getValues();
         }
       }