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

Fixes #6274 added proper support for constants #6288

Closed
wants to merge 2 commits into from

Conversation

steverhoades
Copy link
Contributor

This has potential backwards compatibility breaks as getProperty/getProperties will no longer return constants. Methods getConstant and getConstants must be used instead.

addProperty and addProperties are backwards compatible and will proxy to addConstant and addConstants if a constant is added through those methods.

@Ocramius Ocramius added this to the 2.4.0 milestone May 20, 2014
@Ocramius Ocramius self-assigned this May 20, 2014
@Ocramius
Copy link
Member

@steverhoades can you please check https://github.com/zendframework/zf2-documentation on whether this also needs docs changes?

@steverhoades
Copy link
Contributor Author

@Ocramius Thats a great point, i'll update the docs where necessary.

@Ocramius
Copy link
Member

Manually merged at @053237d37c713468dd4e64c7aef1779f1e335903

@Ocramius Ocramius closed this Jul 27, 2014
freax pushed a commit to freax/zf2 that referenced this pull request Nov 27, 2014
freax pushed a commit to freax/zf2 that referenced this pull request Nov 27, 2014
freax pushed a commit to freax/zf2 that referenced this pull request Nov 27, 2014
freax pushed a commit to freax/zf2 that referenced this pull request Nov 27, 2014
freax pushed a commit to freax/zf2 that referenced this pull request Nov 27, 2014
freax pushed a commit to freax/zf2 that referenced this pull request Nov 27, 2014
freax pushed a commit to freax/zf2 that referenced this pull request Nov 27, 2014
freax pushed a commit to freax/zf2 that referenced this pull request Nov 27, 2014
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