Feeling Lucky
?
[#11838] - App class configuration references
- Fixed in Code Base
- 30 Aug 2016
- Medium
- Build: staging
- # 11838
- Diff
- mbabker:app-config
- Success continuous-integration/travis-ci/pr The Travis CI build passed Details
- Success JTracker/HumanTestResults Human Test Results: 2 Successful 0 Failed. Details
User tests: Successful: Unsuccessful:
Summary of Changes
This changes the default paths that the base CLI and web applications use to lookup configuration files to use the JPATH_CONFIGURATION path constant which is in line with how the rest of the CMS is configured.
This also removes overloading the configuration object created by the application. This actually causes some configuration values set by the parent constructors to be lost.
Testing Instructions
The JConfig object is still correctly read into the application class' $config store and downstream uses continue to read the configuration data from this source correctly.
Documentation Changes Required
N/A
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Libraries |
| Labels |
Added:
?
|
||
truptikagathara
- comment
- 30 Aug 2016
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/11838.
| Status | Pending | ⇒ | Ready to Commit |
zero-24
- comment
- 30 Aug 2016
| Labels |
Added:
?
|
||
rdeutz
- | Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2016-08-30 12:32:01 |
| Closed_By | ⇒ | rdeutz |
| Labels |
Removed:
?
|
||
I have tested this item✅ successfully on 2015e3a
Applied patch, tested frontend and backend and worked fine.
Made a
print_R($app);in isis and protostar index.php and config is there.This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/11838.