Merge pull request #3091 from michalsn/fix_insert_empty_object

Fix - throw an exception on insert for an empty object

system/Model.php

@@ -695,6 +695,11 @@ public function insert($data = null, bool $returnID = true)
 			$data = (array) $data;
 		}
 
+		if (empty($data))
+		{
+			throw DataException::forEmptyDataset('insert');
+		}
+
 		// Validate data before saving.
 		if ($this->skipValidation === false)
 		{
@@ -1439,7 +1444,7 @@ public function cleanRules(bool $choice = false)
 	public function validate($data): bool
 	{
 		$rules = $this->getValidationRules();
-		
+
 		if ($this->skipValidation === true || empty($rules) || empty($data))
 		{
 			return true;
@@ -1580,7 +1585,7 @@ protected function fillPlaceholders(array $rules, array $data): array
 	public function getValidationRules(array $options = []): array
 	{
 		$rules = $this->validationRules;
-		
+
 		// ValidationRules can be either a string, which is the group name,
 		// or an array of rules.
 		if (is_string($rules))

tests/system/Database/Live/ModelTest.php

@@ -1798,7 +1798,7 @@ public function testUndefinedEntityPropertyReturnsNull()
 
 	//--------------------------------------------------------------------
 
-	public function testInsertWithNoDataException()
+	public function testInsertArrayWithNoDataException()
 	{
 		$model = new UserModel();
 		$data  = [];
@@ -1809,7 +1809,7 @@ public function testInsertWithNoDataException()
 
 	//--------------------------------------------------------------------
 
-	public function testUpdateWithNoDataException()
+	public function testUpdateArrayWithNoDataException()
 	{
 		$model = new EventModel();
 
@@ -1832,6 +1832,40 @@ public function testUpdateWithNoDataException()
 
 	//--------------------------------------------------------------------
 
+	public function testInsertObjectWithNoDataException()
+	{
+		$model = new UserModel();
+		$data  = new \stdClass();
+		$this->expectException(DataException::class);
+		$this->expectExceptionMessage('There is no data to insert.');
+		$model->insert($data);
+	}
+
+	//--------------------------------------------------------------------
+
+	public function testUpdateObjectWithNoDataException()
+	{
+		$model = new EventModel();
+
+		$data = (object) [
+							 'name'    => 'Foo',
+							 'email'   => 'foo@example.com',
+							 'country' => 'US',
+							 'deleted' => 0,
+						 ];
+
+		$id = $model->insert($data);
+
+		$data = new \stdClass();
+
+		$this->expectException(DataException::class);
+		$this->expectExceptionMessage('There is no data to update.');
+
+		$model->update($id, $data);
+	}
+
+	//--------------------------------------------------------------------
+
 	public function testInvalidAllowedFieldException()
 	{
 		$model = new JobModel();