Add default property to getIdAttribute (#1098)

benargo · jenssegers · commit 92f108931fea · 2017-02-15T14:55:13.000+01:00
Since the function has a check for `! $value` and assigns the MongoDB default of ‘_id’ we should be able to omit the property.
diff --git a/src/Jenssegers/Mongodb/Eloquent/Model.php b/src/Jenssegers/Mongodb/Eloquent/Model.php
@@ -39,7 +39,7 @@ abstract class Model extends BaseModel
      * @param  mixed $value
      * @return mixed
      */
-    public function getIdAttribute($value)
+    public function getIdAttribute($value = null)
     {
         // If we don't have a value for 'id', we will use the Mongo '_id' value.
         // This allows us to work with models in a more sql-like way.

Original file line number	Diff line number	Diff line change
`@@ -39,7 +39,7 @@ abstract class Model extends BaseModel`
`39`	`39`	`* @param mixed $value`
`40`	`40`	`* @return mixed`
`41`	`41`	`*/`
`42`		`- public function getIdAttribute($value)`
	`42`	`+ public function getIdAttribute($value = null)`
`43`	`43`	`{`
`44`	`44`	`// If we don't have a value for 'id', we will use the Mongo '_id' value.`
`45`	`45`	`// This allows us to work with models in a more sql-like way.`