Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Issue #3222 - Added suport for multi-level nested ini config variables #3236

Closed
wants to merge 2 commits into
from

Conversation

Projects
None yet
3 participants
Contributor

robertboloc commented Dec 16, 2012

Multi-level ini config variables can now be parsed

Issue #3222

Contributor

robertboloc commented Dec 16, 2012

The Travis build fail seems to be unrelated to this fix.

@samsonasik samsonasik commented on an outdated diff Dec 17, 2012

library/Zend/Config/Reader/Ini.php
@@ -145,6 +145,27 @@ protected function process(array $data)
}
/**
+ * Process a nested section
+ *
+ * @param array $sections
+ * @param mixed $value
+ * @return array
+ */
+ private function buildNestedSection($sections, $value)
+ {
+ if(count($sections) == 0) {
+ return $this->processSection($value);
+ } else {
@samsonasik

samsonasik Dec 17, 2012

Contributor

remove 'else' because already return early.

@weierophinney weierophinney added a commit that referenced this pull request Dec 18, 2012

@weierophinney weierophinney Merge branch 'hotfix/3236' into develop
Forward port #3236
0de645c

@weierophinney weierophinney added a commit to zendframework/zend-config that referenced this pull request May 15, 2015

@weierophinney weierophinney Merge branch 'hotfix/3236' 5cdfa08

@weierophinney weierophinney added a commit to zendframework/zend-config that referenced this pull request May 15, 2015

@weierophinney weierophinney Merge branch 'hotfix/3236' into develop 251b79e
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment