Skip to content

Commit

Permalink
Merge pull request #677 from DirectoryTree/BUG-676
Browse files Browse the repository at this point in the history
Bug 676 -  Missing return type findManyBy()
  • Loading branch information
stevebauman authored Jan 9, 2024
2 parents 617102c + a8dd2a3 commit da3d5df
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 1 deletion.
2 changes: 1 addition & 1 deletion src/Query/Builder.php
Original file line number Diff line number Diff line change
Expand Up @@ -764,7 +764,7 @@ public function findMany(array|string $dns, array|string $columns = ['*']): Coll
/**
* Finds many records by the specified attribute.
*/
public function findManyBy(string $attribute, array $values = [], array|string $columns = ['*']): Collection
public function findManyBy(string $attribute, array $values = [], array|string $columns = ['*']): Collection|array
{
$query = $this->select($columns);

Expand Down
8 changes: 8 additions & 0 deletions src/Query/Model/Builder.php
Original file line number Diff line number Diff line change
Expand Up @@ -160,6 +160,14 @@ public function findMany(array|string $dns, array|string $columns = ['*']): Coll
return parent::findMany($dns, $columns);
}

/**
* {@inheritDoc}
*/
public function findManyBy(string $attribute, array $values = [], array|string $columns = ['*']): Collection
{
return parent::findManyBy($attribute, $values, $columns);
}

/**
* Finds a record using ambiguous name resolution.
*/
Expand Down

0 comments on commit da3d5df

Please sign in to comment.