Skip to content
New issue

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

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Phalcon\Collection #13886

Closed
niden opened this issue Mar 10, 2019 · 5 comments

Comments

Projects
2 participants
@niden
Copy link
Member

commented Mar 10, 2019

Phalcon\Collection is a collection is a supercharged OO array. It implements Countable, ArrayAccess, Serializable and JsonSerializable.

It is used as a base for collections through the application, especially when in need to access globals such as $_POST, $_GET etc.

@niden niden added the Feature - NFR label Mar 10, 2019

@niden niden self-assigned this Mar 10, 2019

@niden niden added this to To do in 4.0 Release via automation Mar 10, 2019

@JABirchall

This comment has been minimized.

Copy link

commented Mar 10, 2019

Implement a zephir version of Illuminate/Support/Collection

@niden

This comment has been minimized.

Copy link
Member Author

commented Mar 10, 2019

@JABirchall never seen that before lol. For now we will create it with phalcon -> zephir. Maybe later we can speed it up using a more native C/Zephir implementation

@JABirchall

This comment has been minimized.

Copy link

commented Mar 10, 2019

@niden Im just saying, Illuminate's collection is very verbose and useful, I use it on a ton of things and even pull it into phalcon projects to collect database, api or stream results.

@niden

This comment has been minimized.

Copy link
Member Author

commented Mar 10, 2019

I will check it out bud thanks.

@niden niden moved this from To do to In progress in 4.0 Release Mar 12, 2019

@niden niden referenced this issue Mar 12, 2019

Merged

T13886 collection #13891

4 of 4 tasks complete

niden added a commit that referenced this issue Mar 12, 2019

T13886 collection (#13891)
* [4.0.x] - Added unicode flag for email filter

* [4.0.x] - Updating the ext folder

* [4.0.x] - Added Arr helper class

* Revert "[4.0.x] - Added Arr helper class"

This reverts commit f8153d3.

* Revert "[4.0.x] - Updating the ext folder"

This reverts commit 655eafe.

* [#13886] - Initial implementation and test stubs

* [#13886] - Fixing interface

* [#13886] - Added constructor

* [#13886] - Implementation of the collection

* [#13886] - Collection tests

* [#13886] - Updated the CHANGELOG
@niden

This comment has been minimized.

Copy link
Member Author

commented Mar 12, 2019

Implemented

@niden niden closed this Mar 12, 2019

4.0 Release automation moved this from In progress to Done Mar 12, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.