Merge pull request #5 from apility/dev

Adds missing fields for TransactionLog and fixed serialzation

Author: thomas-alrek

src/Ecommerce/TransactionLog.php

@@ -11,6 +11,7 @@
  * @property string $transactionText
  * @property string $timeStamp
  * @property string $operation
+ * @property string $requestId
  * @property bool $operationSuccess
  */
 class TransactionLog extends Model
@@ -21,9 +22,11 @@ class TransactionLog extends Model
   protected $fields = [
     'amount',
     'transactionText',
+    'transactionId',
     'timeStamp',
     'operation',
-    'operationSuccess'
+    'requestId',
+    'operationSuccess',
   ];
 
   /** @var array */
@@ -36,6 +39,6 @@ class TransactionLog extends Model
    */
   public function getOperationSuccess(): bool
   {
-    return boolval($this->_operationSuccess);
+    return boolval($this->attributes['operationSuccess']);
   }
 }

src/Model.php

@@ -62,7 +62,9 @@ public static function create(array $attributes = [], $immuteable = false)
    */
   public function jsonSerialize(): array
   {
-    $model = array_filter($this->__debugInfo());
+    $model = array_filter($this->__debugInfo(), function ($value) {
+      return !is_null($value);
+    });
 
     foreach ($this->rawValues as $key) {
       $model[$key] = $this->attributes[$key] ?? null;