vlw/scaffold
1
0
Fork 0
mirror of https://codeberg.org/vlw/scaffold.git synced 2026-02-26 03:21:57 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions

fix: pass an optional return value to Database\Model->set() (#28)

Browse source 
This PR addresses a bug that can arise (for example) if an object is passed to a database column that accepts a string. Passing this object (or something else) to the `$return` parameter will return the value of that variable instead of the value of `$value`

Reviewed-on: https://codeberg.org/vlw/scaffold/pulls/28
This commit is contained in:
Victor Westerlund 2026-02-16 15:07:11 +01:00
parent 19e15c0b23
commit fd0c8cbbcd
1 changed files with 3 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
src/Database/Model.php
View file

@ -97,9 +97,10 @@
* *
* @param string $key Target column to update * @param string $key Target column to update
* @param mixed $value New value of target column * @param mixed $value New value of target column
* @param return $return (optional) Return this instead of $value when set
* @return mixed The value that was sent to $value * @return mixed The value that was sent to $value
*/ */
public function set(string $key, mixed $value): mixed { public function set(string $key, mixed $value, mixed $return = null): mixed {
$this->_row[$key] = $value; $this->_row[$key] = $value;
$this->db $this->db
@ -107,6 +108,6 @@
->where($this->where) ->where($this->where)
->update([$key => $value]); ->update([$key => $value]);
return $value; return $return ?? $value;
} }
} }