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

Class was supporting limit + offset or limit, but only offset does not support #4032

Merged
merged 1 commit into from
Apr 30, 2013

Conversation

romulobusatto
Copy link
Contributor

Class was supporting limit + offset or limit, but only offset does not support

Class was supporting limit + offset or limit, but only offset does not support
@ralphschindler
Copy link
Member

Can you provide me with some code use cases to better understand the fix here?
Are you saying it wont work if offset is used without limit?

@romulobusatto
Copy link
Contributor Author

yes, only use off, as in the example below will not work.
Sample:
$sql = new Sql($adapter);
$select = $sql->select();
$select->from('album');
$select->offset(5);

@ghost ghost assigned ralphschindler Mar 27, 2013
ralphschindler pushed a commit that referenced this pull request Apr 30, 2013
Merge branch 'master' of git://github.com/romulobusatto/zf2 into romulobusatto-master
ralphschindler pushed a commit that referenced this pull request Apr 30, 2013
Merge branch 'romulobusatto-master'
@ralphschindler ralphschindler merged commit e7a2618 into zendframework:master Apr 30, 2013
ralphschindler pushed a commit that referenced this pull request Apr 30, 2013
Merge branch 'romulobusatto-master' into develop
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