[#9409] - Fix support for autoloadable model path
- Fixed in Code Base
- 15 Apr 2016
- Medium
- Build: staging
- # 9409
- Diff
- izharaazmi:patch-7
- 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:
Currently the "controllers", "models", "tables" etc folder names are used in many components. However we are supposed to start using singular forms "controller", "model", "table" etc so as to support autoloading of these classes.
Summary of Changes
The model prefix is being calculated from the controller name which is always in lowercase. This causes evaluating the model prefix as sampleModel instead of SampleModel assuming component as com_sample. This breaks autoload for the model.
Testing Instructions
Create a small empty component using "model" as folder name for models. (or rename the "models" folder) and try to get some data from the model like $this->get('Data'); which should get the data from SampleModelSample::getData() and display, ideally!
I created a sample component from component-creator.com and modified it for this demo. Use it if you want. See attachment below.
Open the URL: index.php?option=com_sample in your front-end.
Result
Without this patch: Nothing is shown
After this patch: You see "Hello Joomla!"
Attachment
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Labels |
Added:
?
|
||
| Category | ⇒ | Libraries |
I have tested this item 
successfully on 52bcb7d
Patch works
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9409.
| Status | Pending | ⇒ | Ready to Commit |
RTC - thanks
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9409.
| Labels |
Added:
?
|
||
| Milestone |
Added: |
||
rdeutz
- | Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2016-04-15 12:14:44 |
| Closed_By | ⇒ | rdeutz |
| Labels |
Removed:
?
|
||
| Milestone |
Removed: |
||
| Milestone |
Added: |
||
| Milestone |
Added: |
||
| Milestone |
Removed: |
||
I have tested this item
successfully on 52bcb7d
works here thanks.
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9409.