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

Skip to content
This repository was archived by the owner on Jan 8, 2020. It is now read-only.

Db order expressioninterface#6722

Closed
Pittiplatsch wants to merge 6 commits into
zendframework:masterfrom
Pittiplatsch:db_order_expressioninterface
Closed

Db order expressioninterface#6722
Pittiplatsch wants to merge 6 commits into
zendframework:masterfrom
Pittiplatsch:db_order_expressioninterface

Conversation

@Pittiplatsch
Copy link
Copy Markdown

PR for #6719

Lars Heber added 5 commits October 1, 2014 14:43
…ionInterface

instead of Expression
* Add unit test for ordering using ExpressionInterface
* Improve existing unit test for ordering using Expression - the
original test actually tested expression functionality, not order
functionality
@turrsis
Copy link
Copy Markdown
Contributor

turrsis commented Nov 19, 2014

May be fix this for Select::quantifier() method too?

@Pittiplatsch
Copy link
Copy Markdown
Author

If you could provide unit tests, extending quantifier()'s API should be trivial 👍

@turrsis
Copy link
Copy Markdown
Contributor

turrsis commented Nov 19, 2014

    public function testQuantifierParameter()
    {
        $expr = $this->getMock('Zend\Db\Sql\ExpressionInterface');
        $select = new Select;
        $select->quantifier($expr);
        $this->assertSame(
            $expr,
            $select->getRawState(Select::QUANTIFIER)
        );
    }

@Ocramius Ocramius self-assigned this Dec 11, 2014
@Ocramius Ocramius modified the milestones: 2.4.0, 2.3.4 Dec 11, 2014
@Ocramius Ocramius closed this in 530fab5 Dec 16, 2014
Ocramius added a commit that referenced this pull request Dec 16, 2014
@Ocramius
Copy link
Copy Markdown
Member

@Pittiplatsch merged, thanks!

master: 530fab5
develop: 088c2ee

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants