Skip to content

Conversation

@dbwhddn10
Copy link
Contributor

Fix Query Builder GroupBy Method Doc Block

Fix Query Builder GroupBy Method Doc Block
@Anahkiasen
Copy link
Contributor

Should be @param string $column,... if I remember correctly the phpdoc for variadics

@dbwhddn10
Copy link
Contributor Author

@GrahamCampbell
Copy link
Collaborator

@Anahkiasen is correct. Also, you need two spaces before and after the type.

@dbwhddn10
Copy link
Contributor Author

@GrahamCampbell

thanks but $user->groupBy(['email','password'])->get(); is available.

===== result =======
select * from users group by email, password

@GrahamCampbell
Copy link
Collaborator

Maybe this is better?

@param  array|string  $column,...

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 0a0331b on dbwhddn10:4.2 into * on laravel:4.2*.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants