Create Callback adapter for Zend\Paginator #5272

Closed
wants to merge 1 commit into from

5 participants

@henriquemoody

No description provided.

@danizord

👍

@macnibblet

@henriquemoody please add docblock statements on all the methods

@henriquemoody

@macnibblet, I really think this is unnecessary, but I amended the commit for it.

@macnibblet

@henriquemoody Coding standards.... :)

@henriquemoody

@macnibblet, I thought Zend Framework was PSR compliance and there is no requirement of documentation blocks on PSR's.

@stefanotorresi

@henriquemoody PSR is just a standard that ZF2 own standard happens to follow. Check out this ;) http://framework.zend.com/wiki/display/ZFDEV2/Coding+Standards#CodingStandards-Functions

that wiki itself is no more mantained, but i think most of that page currently applies.

@henriquemoody

Thanks for this link, @stefanotorresi. :)

@weierophinney weierophinney added a commit that referenced this pull request Oct 23, 2013
@weierophinney weierophinney [#5272] Grammar cleanup
- Updated docblocks to use better grammatical phrasing.
d236723
@weierophinney weierophinney added a commit that referenced this pull request Oct 23, 2013
@weierophinney weierophinney Merge branch 'feature/5272' into develop
Close #5272
bcc9a69
@weierophinney
Zend Framework member

Merged to develop for release with 2.3.0.

@weierophinney weierophinney added a commit to zendframework/zend-paginator that referenced this pull request May 15, 2015
@weierophinney weierophinney Merge pull request zendframework/zendframework#5272 from henriquemood…
…y/paginator_adapter_callback

Create Callback adapter for Zend\Paginator
9a939ea
@weierophinney weierophinney added a commit to zendframework/zend-paginator that referenced this pull request May 15, 2015
@weierophinney weierophinney [zendframework/zendframework#5272] Grammar cleanup
- Updated docblocks to use better grammatical phrasing.
20f04c6
@weierophinney weierophinney added a commit to zendframework/zend-paginator that referenced this pull request May 15, 2015
@weierophinney weierophinney Merge branch 'feature/5272' into develop 556ce3e
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment