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

Create Callback adapter for Zend\Paginator #5272

Closed
wants to merge 1 commit into from
Closed

Create Callback adapter for Zend\Paginator #5272

wants to merge 1 commit into from

Conversation

henriquemoody
Copy link

No description provided.

@danizord
Copy link
Contributor

👍

@macnibblet
Copy link
Contributor

@henriquemoody please add docblock statements on all the methods

@henriquemoody
Copy link
Author

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

@macnibblet
Copy link
Contributor

@henriquemoody Coding standards.... :)

@henriquemoody
Copy link
Author

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

@stefanotorresi
Copy link
Contributor

@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
Copy link
Author

Thanks for this link, @stefanotorresi. :)

@ghost ghost assigned weierophinney Oct 23, 2013
weierophinney added a commit that referenced this pull request Oct 23, 2013
Create Callback adapter for Zend\Paginator
weierophinney added a commit that referenced this pull request Oct 23, 2013
- Updated docblocks to use better grammatical phrasing.
weierophinney added a commit that referenced this pull request Oct 23, 2013
@weierophinney
Copy link
Member

Merged to develop for release with 2.3.0.

weierophinney added a commit to zendframework/zend-paginator that referenced this pull request May 15, 2015
…y/paginator_adapter_callback

Create Callback adapter for Zend\Paginator
weierophinney added a commit to zendframework/zend-paginator that referenced this pull request May 15, 2015
- Updated docblocks to use better grammatical phrasing.
weierophinney added a commit to zendframework/zend-paginator that referenced this pull request May 15, 2015
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.

5 participants