Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Update lib/Doctrine/ORM/Query/AST/GeneralCaseExpression.php #277

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
5 participants

Changed to make the CASE EXPRESSION uses strings into DQL's.

e.g.

$source = $this->em->createQueryBuilder()
    ->select('m')
            ->addSelect("CASE WHEN (date >= newDate) THEN 'Active' ELSE 'Inactive' END status")
    ->from('sys:Mask', 'm')
    ->orderBy('status', 'ASC');
Owner

beberlei commented Feb 19, 2012

Ah its here. two patches :-) Can you merge them into one?

Hey Berbelei,

Sorry for the late answer, it was carnaval here in Brazil. So, the patches are 2 (277, 278), sorry for my mistake, but I've tried to do the merge but I didn't find how. Could you give me a tip?

Cheers.

Member

stof commented Feb 23, 2012

@medinadato do the changes in the same branch instead of using 2 different branches

Member

asm89 commented Mar 11, 2012

I can confirm that the issue is valid, the fix however is not. Discussed with @guilhermeblanco that he'll work on it. JIRA ticket here: http://www.doctrine-project.org/jira/browse/DDC-1696.

@asm89 asm89 closed this Mar 11, 2012

Owner

guilhermeblanco commented Mar 12, 2012

Fixed this issue in 7954386

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