Skip to content
This repository has been archived by the owner on Nov 11, 2020. It is now read-only.

Adding support for the $nor operator #107

Closed
wants to merge 2 commits into from

Conversation

rsandrea
Copy link
Contributor

@rsandrea rsandrea commented May 8, 2013

No description provided.

*
* $qb = $this->createQueryBuilder('User');
* $qb
* ->addNor($qb->expr()->field('first_name')->equals('Kris')->field('first_name')->equals('Chris'));
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This would only add a single $nor clause, unlike the previous examples for addAnd() and addOr(). Also, { first_name: "Chris" } would override { first_name: "Kris" }, since you're setting them on the same expression object.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You are right, there is an error on the DocBlock

fix for the DocBlock
@jmikola
Copy link
Member

jmikola commented May 13, 2013

Ah, thanks for updating it, but I just merged the original commit with a second (dae5464) to fix the documentation and add a unit test.

@jmikola jmikola closed this May 13, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants