Skip to content

Commit

Permalink
The docblock above \Packaged\Dal\Ql\QlDao::loadOneWhere is incorrect …
Browse files Browse the repository at this point in the history
…as it says that it will always return static which is not true as the default value of \Packaged\Dal\Ql\QlDaoCollection::first is null (#88)

By the same token the return value of \Packaged\Dal\Ql\QlDaoCollection::first should be QlDao|null|mixed as the default value if passed can be set to anything

Co-authored-by: nikolaybrankov <[email protected]>
  • Loading branch information
callyall and nikolaybrankov authored Mar 19, 2024
1 parent afcf618 commit d93e0ab
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/Ql/QlDao.php
Original file line number Diff line number Diff line change
Expand Up @@ -114,7 +114,7 @@ public function getCollection(...$params)
/**
* @param $params
*
* @return static
* @return static|null
*
* @throws MultipleDaoException
*/
Expand Down
2 changes: 1 addition & 1 deletion src/Ql/QlDaoCollection.php
Original file line number Diff line number Diff line change
Expand Up @@ -203,7 +203,7 @@ public function delete()
*
* @param mixed $default
*
* @return QlDao
* @return QlDao|null|mixed
*/
public function first($default = null)
{
Expand Down

0 comments on commit d93e0ab

Please sign in to comment.