Merge branch 'develop'

flat3 · Oct 31, 2020 · ab6a8b8 · ab6a8b8
2 parents 39f9ff5 + 64a703a
commit ab6a8b8
Show file tree

Hide file tree

Showing 3 changed files with 26 additions and 5 deletions.
diff --git a/src/Entity.php b/src/Entity.php
@@ -117,7 +117,13 @@ public function emit(Transaction $transaction): void
             }
         }
 
-        while ($this->properties->valid()) {
+        $requiresSeparator = false;
+
+        while (true) {
+            if (!$this->properties->valid()) {
+                break;
+            }
+
             /** @var PropertyValue $propertyValue */
             $propertyValue = $this->properties->current();
 
@@ -126,6 +132,10 @@ public function emit(Transaction $transaction): void
                 continue;
             }
 
+            if ($requiresSeparator) {
+                $transaction->outputJsonSeparator();
+            }
+
             if ($propertyValue->getProperty() instanceof NavigationProperty) {
                 $propertyMetadata = $this->getExpansionMetadata($transaction, $propertyValue);
 
@@ -144,11 +154,8 @@ public function emit(Transaction $transaction): void
                 $transaction->outputJsonValue($value);
             }
 
+            $requiresSeparator = true;
             $this->properties->next();
-
-            if ($this->properties->valid() && $this->properties->current()->shouldEmit($transaction)) {
-                $transaction->outputJsonSeparator();
-            }
         }
 
         $transaction->outputJsonObjectEnd();

diff --git a/tests/Unit/Queries/Entity/EntityTest.php b/tests/Unit/Queries/Entity/EntityTest.php
@@ -106,6 +106,15 @@ public function test_read_with_multiple_select()
         );
     }
 
+    public function test_read_with_multiple_select_non_adjacent()
+    {
+        $this->assertJsonResponse(
+            Request::factory()
+                ->path('/flights(1)')
+                ->query('$select', 'origin,gate')
+        );
+    }
+
     public function test_rejects_invalid_select()
     {
         $this->assertBadRequest(

diff --git a/...ries/Entity/__snapshots__/EntityTest__test_read_with_multiple_select_non_adjacent__1.json b/...ries/Entity/__snapshots__/EntityTest__test_read_with_multiple_select_non_adjacent__1.json
@@ -0,0 +1,5 @@
+{
+    "@context": "http://localhost/odata/$metadata#flights(origin,gate)/$entity",
+    "origin": "lhr",
+    "gate": null
+}