Thanks to visit codestin.com
Credit goes to github.com

Skip to content

bugfix: orHaving does not work when used as first having condition.#5708

Closed
abentkamp wants to merge 4 commits into
laravel:4.2from
abentkamp:4.2
Closed

bugfix: orHaving does not work when used as first having condition.#5708
abentkamp wants to merge 4 commits into
laravel:4.2from
abentkamp:4.2

Conversation

@abentkamp

Copy link
Copy Markdown
Contributor

No description provided.

@Anahkiasen

Copy link
Copy Markdown
Contributor

Should add a test proving the bug exists too.

@GrahamCampbell

Copy link
Copy Markdown
Collaborator

Yeh. We need a test please @benti.

@abentkamp

Copy link
Copy Markdown
Contributor Author

Sure. I'll write one as soon as I have time for it.

@GrahamCampbell

Copy link
Copy Markdown
Collaborator

Great. Thanks. :)

@abentkamp

Copy link
Copy Markdown
Contributor Author

Ok, here's the test :)

@GrahamCampbell

Copy link
Copy Markdown
Collaborator

Could you squash to one commit and force push please?

@GrahamCampbell

Copy link
Copy Markdown
Collaborator

Replaced by #5875.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants