From c08a1e6455101184f833f14544fd83f115de2777 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Thomas=20M=C3=BCller?= Date: Sun, 17 Aug 2014 22:07:23 +0200 Subject: [PATCH] according to PHPDoc this function shall return the entity --- lib/public/appframework/db/mapper.php | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/lib/public/appframework/db/mapper.php b/lib/public/appframework/db/mapper.php index a23149e796b..03f0fe3aa75 100644 --- a/lib/public/appframework/db/mapper.php +++ b/lib/public/appframework/db/mapper.php @@ -125,6 +125,7 @@ abstract class Mapper { * Updates an entry in the db from an entity * @throws \InvalidArgumentException if entity has no id * @param Entity $entity the entity that should be created + * @return Entity the saved entity with the set id */ public function update(Entity $entity){ // if entity wasn't changed it makes no sense to run a db query @@ -142,7 +143,7 @@ abstract class Mapper { // get updated fields to save, fields have to be set using a setter to // be saved - // dont update the id field + // do not update the id field unset($properties['id']); $columns = ''; @@ -171,6 +172,8 @@ abstract class Mapper { array_push($params, $id); $this->execute($sql, $params); + + return $entity; } -- 2.39.5