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

Fix: Inline variable #7180

Merged
merged 1 commit into from
Feb 3, 2015
Merged

Fix: Inline variable #7180

merged 1 commit into from
Feb 3, 2015

Conversation

localheinz
Copy link
Member

This PR

  • returns the newly created instance of a module class directly, instead of assigning it to a variable first and then returning it

@Ocramius Ocramius added this to the 2.3.5 milestone Feb 2, 2015
@Ocramius Ocramius self-assigned this Feb 2, 2015
@Ocramius Ocramius merged commit 4249daf into zendframework:master Feb 3, 2015
Ocramius added a commit that referenced this pull request Feb 3, 2015
@Ocramius
Merge branch 'cs/#7180-module-manager-direct-value-return'
de98f63 
Close #7180
Ocramius added a commit that referenced this pull request Feb 3, 2015
@Ocramius
Merge branch 'cs/#7180-module-manager-direct-value-return' into develop
be0b349 
Close #7180
Forward port #7180
@Ocramius
Copy link
Member

Ocramius commented Feb 3, 2015

Merged, thanks @localheinz!

master: de98f63
develop: be0b349

gianarb pushed a commit to zendframework/zend-modulemanager that referenced this pull request May 15, 2015
@Ocramius
Merge branch 'cs/zendframework/zendframework#7180-module-manager-dire…
b817c63 
…ct-value-return'

Close zendframework/zendframework#7180
gianarb pushed a commit to zendframework/zend-modulemanager that referenced this pull request May 15, 2015
@Ocramius
Merge branch 'cs/zendframework/zendframework#7180-module-manager-dire…
ebac38f 
…ct-value-return' into develop

Close zendframework/zendframework#7180
Forward port zendframework/zendframework#7180
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees

@Ocramius Ocramius

Labels
enhancement ModuleManager
Projects
None yet
Milestone
2.3.5
Development

Successfully merging this pull request may close these issues.
2 participants
@localheinz @Ocramius