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

Add MongoCollection Uniqueness compatibility #744

Merged
merged 1 commit into from
Mar 14, 2017

Conversation

Jurigag
Copy link
Contributor

@Jurigag Jurigag commented Mar 9, 2017

Hello!

This pull request affects the following components: (please check boxes)

  • Library
  • Code Style
  • Documentation
  • Testing

In raising this pull request, I confirm the following (please check boxes):

  • I have read and understood the Contributing Guidelines?
  • I have checked that another pull request for this purpose does not exist.
  • I wrote some tests for this PR.

Small description of change: this is adding compability for MongoCollection to work with Phalcon\Validation\Validator\Uniqueness since from phalcon 3.1.x Collection can use Phalcon\Validation component.

Thanks

* User: User
* Date: 2017-03-09
* Time: 17:11
*/
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you please remove this

* @copyright (c) 2011-2017 Phalcon Team
* @link https://www.phalconphp.com
* @author Wojciech Ślawski <[email protected]>
* @package Phalcon\Test\Mvc
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Phalcon\Test\Mvc\Collection

@Jurigag
Copy link
Contributor Author

Jurigag commented Mar 9, 2017

I have no idea why tests are failing.

PHP Fatal error: Cannot use Phalcon\Validation\Exception as Exception because the name is already in use in /home/travis/build/phalcon/incubator/Library/Phalcon/Validation/Validator/CardNumber.php on line 25

Any idea what this mean?

* User: User
* Date: 2017-03-09
* Time: 17:07
*/
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you please remove this

* User: User
* Date: 2017-03-09
* Time: 17:06
*/
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you please remove this

@Jurigag Jurigag force-pushed the 3.1.x-mongo-collection branch 6 times, most recently from 7aee6fc to 69a7fe0 Compare March 9, 2017 22:36
@Jurigag Jurigag changed the base branch from 3.0.x to 3.1.x March 9, 2017 22:52
@Jurigag Jurigag closed this Mar 9, 2017
@Jurigag Jurigag reopened this Mar 9, 2017
@Jurigag Jurigag force-pushed the 3.1.x-mongo-collection branch 6 times, most recently from e9a9eef to ee121b9 Compare March 14, 2017 12:34
@sergeyklay sergeyklay merged commit 691bd90 into phalcon:3.1.x Mar 14, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants