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

Fix for Collection extract method updates targetElement object #3081

Closed
wants to merge 5 commits into from
Closed

Fix for Collection extract method updates targetElement object #3081

wants to merge 5 commits into from

Conversation

xoob
Copy link
Contributor

@xoob xoob commented Nov 27, 2012

Fixes #2574

@weierophinney
Copy link
Member

@xoob Can you please provide some tests?

@ghost ghost assigned weierophinney Dec 10, 2012
@weierophinney
Copy link
Member

@xoob I'd like to merge this, but I need some tests first, please.

@xoob
Copy link
Contributor Author

xoob commented Jan 3, 2013

@weierophinney Tests added. Build breaks for 5.3 because composer can't download something but the tests run ok for 5.3.3 and 5.4.

[Composer\Downloader\TransportException]                                     
The 'https://api.github.com/repos/sebastianbergmann/php-code-coverage/zipball/1.2.7' URL
could not be accessed: HTTP/1.1 403 Forbidden

weierophinney added a commit that referenced this pull request Jan 3, 2013
@weierophinney
Merge branch 'hotfix/3081' into develop
a7a1f6c 
Forward port #3081

Conflicts:
	library/Zend/Form/Element/Collection.php
	tests/ZendTest/Form/Element/CollectionTest.php
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees

@weierophinney weierophinney

Labels
None yet
Projects
None yet
Milestone
2.0.7
Development

Successfully merging this pull request may close these issues.

Collection extract method updates targetElement object to an existing value
2 participants
@xoob @weierophinney