Changing the status of an object with a numeric labels works fine now.

seatsio · Oct 3, 2018 · 3eddb32 · 3eddb32
1 parent 38e5464
commit 3eddb32
Show file tree

Hide file tree

Showing 7 changed files with 19 additions and 41 deletions.
diff --git a/src/Events/BestAvailableObjects.php b/src/Events/BestAvailableObjects.php
@@ -2,34 +2,19 @@
 
 namespace Seatsio\Events;
 
-use Seatsio\SeatsioJsonMapper;
-
 class BestAvailableObjects
 {
     /**
      * @var string[]
      */
     public $objects;
     /**
-     * @var array
+     * @var array[Labels]
      */
     public $labels;
     /**
      * @var boolean
      */
     public $nextToEachOther;
 
-    /**
-     * @param $json
-     * @return BestAvailableObjects
-     */
-    static function fromJson($json)
-    {
-        $mapper = SeatsioJsonMapper::create();
-        $result = $mapper->map($json, new BestAvailableObjects());
-        array_walk($result->labels, function ($labels, $id) use ($mapper, $result) {
-            $result->labels[$id] = $mapper->map($labels, new Labels());
-        });
-        return $result;
-    }
 }
diff --git a/src/Events/ChangeObjectStatusResult.php b/src/Events/ChangeObjectStatusResult.php
@@ -2,26 +2,11 @@
 
 namespace Seatsio\Events;
 
-use Seatsio\SeatsioJsonMapper;
-
 class ChangeObjectStatusResult
 {
     /**
-     * @var array
+     * @var array[Labels]
      */
     var $labels;
 
-    /**
-     * @param $json
-     * @return ChangeObjectStatusResult
-     */
-    static function fromJson($json)
-    {
-        $mapper = SeatsioJsonMapper::create();
-        $result = $mapper->map($json, new ChangeObjectStatusResult());
-        array_walk($result->labels, function ($labels, $id) use ($mapper, $result) {
-            $result->labels[$id] = $mapper->map($labels, new Labels());
-        });
-        return $result;
-    }
 }
diff --git a/src/Events/Events.php b/src/Events/Events.php
@@ -261,7 +261,8 @@ public function changeObjectStatus($eventKeyOrKeys, $objectOrObjects, $status, $
             ['json' => $request, 'query' => ['expand' => 'labels']]
         );
         $json = \GuzzleHttp\json_decode($res->getBody());
-        return ChangeObjectStatusResult::fromJson($json);
+        $mapper = SeatsioJsonMapper::create();
+        return $mapper->map($json, new ChangeObjectStatusResult());
     }
 
     /**
@@ -360,7 +361,8 @@ public function changeBestAvailableObjectStatus($eventKey, $number, $status, $ca
             ['json' => $request]
         );
         $json = \GuzzleHttp\json_decode($res->getBody());
-        return BestAvailableObjects::fromJson($json);
+        $mapper = SeatsioJsonMapper::create();
+        return $mapper->map($json, new BestAvailableObjects());
     }
 
     private static function normalizeObjects($objectOrObjects)

diff --git a/src/SeatsioJsonMapper.php b/src/SeatsioJsonMapper.php
@@ -4,15 +4,21 @@
 
 use JsonMapper;
 
-class SeatsioJsonMapper
+class SeatsioJsonMapper extends JsonMapper
 {
     /**
      * @return JsonMapper
      */
     public static function create()
     {
-        $mapper = new JsonMapper();
+        $mapper = new SeatsioJsonMapper();
         $mapper->classMap["\DateTime"] = JsonDateTime::class;
         return $mapper;
     }
+
+    protected function getSafeName($name)
+    {
+        // avoid object labels such as B-4 being converted into B4
+        return $name;
+    }
 }
diff --git a/tests/Events/ChangeObjectStatusForMultipleObjectsTest.php b/tests/Events/ChangeObjectStatusForMultipleObjectsTest.php
@@ -94,15 +94,15 @@ public function testQuantity()
         $event = $this->seatsioClient->events->create($chartKey);
         $objects = [
             (new ObjectProperties("GA1"))->setQuantity(5),
-            (new ObjectProperties("GA2"))->setQuantity(10)
+            (new ObjectProperties("34"))->setQuantity(10)
         ];
 
         $this->seatsioClient->events->changeObjectStatus($event->key, $objects, "lolzor");
 
         $status1 = $this->seatsioClient->events->retrieveObjectStatus($event->key, "GA1");
         self::assertEquals(5, $status1->quantity);
 
-        $status2 = $this->seatsioClient->events->retrieveObjectStatus($event->key, "GA2");
+        $status2 = $this->seatsioClient->events->retrieveObjectStatus($event->key, "34");
         self::assertEquals(10, $status2->quantity);
     }
 

diff --git a/tests/Events/ChangeObjectStatusTest.php b/tests/Events/ChangeObjectStatusTest.php
@@ -48,10 +48,10 @@ public function testGA()
         $chartKey = $this->createTestChart();
         $event = $this->seatsioClient->events->create($chartKey);
 
-        $result = $this->seatsioClient->events->changeObjectStatus($event->key, "GA1", "lolzor");
+        $result = $this->seatsioClient->events->changeObjectStatus($event->key, "34", "lolzor");
 
         self::assertEquals([
-            "GA1" => someLabels("GA1", "generalAdmission")
+            "34" => someLabels("34", "generalAdmission")
         ], $result->labels);
     }
 

diff --git a/tests/sampleChart.json b/tests/sampleChart.json
@@ -421,7 +421,7 @@
         "categoryLabel": "Cat2",
         "categoryKey": 10,
         "capacity": 100,
-        "label": "GA2",
+        "label": "34",
         "labelSize": 24,
         "labelShown": true,
         "labelHorizontalOffset": 0,