Merge pull request #121 from ElliottLandsborough/patch-1

babenkoivan · web-flow · commit b1313b75640f · 2018-08-18T10:40:34.000+02:00
Fix getKey check when ($model['id'] === 0)
diff --git a/src/Payloads/DocumentPayload.php b/src/Payloads/DocumentPayload.php
@@ -13,7 +13,7 @@ class DocumentPayload extends TypePayload
      */
     public function __construct(Model $model)
     {
-        if (!$model->getKey()) {
+        if ($model->getKey() === null) {
             throw new Exception(sprintf(
                 'The key value must be set to construct a payload for the %s instance.',
                 get_class($model)
@@ -25,4 +25,4 @@ public function __construct(Model $model)
         $this->payload['id'] = $model->getKey();
         $this->protectedKeys[] = 'id';
     }
-}
+}

Original file line number	Diff line number	Diff line change
`@@ -13,7 +13,7 @@ class DocumentPayload extends TypePayload`
`13`	`13`	`*/`
`14`	`14`	`public function __construct(Model $model)`
`15`	`15`	`{`
`16`		`- if (!$model->getKey()) {`
	`16`	`+ if ($model->getKey() === null) {`
`17`	`17`	`throw new Exception(sprintf(`
`18`	`18`	`'The key value must be set to construct a payload for the %s instance.',`
`19`	`19`	`get_class($model)`
`@@ -25,4 +25,4 @@ public function __construct(Model $model)`
`25`	`25`	`$this->payload['id'] = $model->getKey();`
`26`	`26`	`$this->protectedKeys[] = 'id';`
`27`	`27`	`}`
`28`		`-}`
	`28`	`+}`