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

update for new zend\hydrator component #149

Closed
wants to merge 6 commits into from
Closed

Conversation

prolic
Copy link
Contributor

@prolic prolic commented Oct 1, 2015

No description provided.

@gianarb gianarb added the Enhancement New feature or request label Oct 1, 2015
@prolic
Copy link
Contributor Author

prolic commented Oct 4, 2015

Question: Do you also want to rename DoctrineModule\Stdlib\Hydrator\DoctrineObject => DoctrineModule\Hydrator\DoctrineObject ?

@prolic
Copy link
Contributor Author

prolic commented Oct 4, 2015

Needed to drop 5.4 support to get the tests running again. PHP 5.4 is also not support by PHP-Internals anymore.

@Ocramius
Copy link
Member

Ocramius commented Oct 4, 2015

Question: Do you also want to rename DoctrineModule\Stdlib\Hydrator\DoctrineObject => DoctrineModule\Hydrator\DoctrineObject ?

Not a good idea

@prolic
Copy link
Contributor Author

prolic commented Oct 9, 2015

weird that the build succeeded. zend-mvc 2.6 required zend-hydrator 1.0, should be incompatible with 2.0.

@@ -1,7 +1,7 @@
<?php
chdir(dirname(__DIR__));

$loader = require_once('vendor/autoload.php');
$loader = require ('vendor/autoload.php');
Copy link
Contributor

Choose a reason for hiding this comment

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

Why?

@alcaeus alcaeus added this to the 1.0.0 milestone Feb 12, 2017
@alcaeus
Copy link
Member

alcaeus commented May 13, 2017

Closing since the pull request is rather old and the patch doesn't apply cleanly. I've checked whether the class in question is still used, which it isn't, so I also assume that this was cleaned up in the meantime. If not, please let me know.

@alcaeus alcaeus closed this May 13, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BC Break Enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants