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

Adds preserve defined order as a factory option#7276

Merged
weierophinney merged 1 commit intozendframework:developfrom
carnage:add-options-to-annotation-builder-factory
Mar 10, 2015
Merged

Adds preserve defined order as a factory option#7276
weierophinney merged 1 commit intozendframework:developfrom
carnage:add-options-to-annotation-builder-factory

Conversation

@carnage
Copy link
Contributor

@carnage carnage commented Feb 28, 2015

No description provided.

Copy link
Member

Choose a reason for hiding this comment

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

Why are you removing these?

Copy link
Member

Choose a reason for hiding this comment

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

Granted, tests pass without them, so likely they aren't needed.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

They were required by a previous implementation of the factory; I improved it but hadn't removed them from the test

@weierophinney weierophinney added this to the 2.4.0 milestone Mar 10, 2015
@weierophinney weierophinney merged commit e9c90f9 into zendframework:develop Mar 10, 2015
weierophinney added a commit that referenced this pull request Mar 10, 2015
…er-factory

Adds preserve defined order as a factory option
weierophinney added a commit that referenced this pull request Mar 10, 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.

2 participants