Feeling Lucky
[#1594] - Grouped uppercase class name letters
- Closed
- 7 Dec 2013
- Medium
- Build: master
- # 1594
- Diff
- bembelimen:patch-4
- Success default The Travis CI build passed Details
User tests: Successful: Unsuccessful:
Refering to the patch 0414f0f the JViewLegacy name loader was changed for the "new MVC". This breaks classes like: ExampleViewSEO. The getName() returns "s" instead of "seo". This patch fixed that behavior:
ExampleViewSEO => seo
ExampleViewApplicationHtml => application
matrikular
- 
dongilbert
- comment
- 26 Jul 2013
Test this getName() method in your view and see if it fixes your issue: https://gist.github.com/dongilbert/6089087
Note: it should. :)
phproberto
- comment
- 26 Jul 2013
@dongilbert can you please create a PR for easy testing?
Thanks!
Hackwar
- comment
- 5 Dec 2013
Is this still valid or can this one be closed?
mahagr
- comment
- 7 Dec 2013
Looks like it's fixed.
| Status | New | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2013-12-07 17:04:46 |
Looks like I wasn't as fast as you are:
http://joomlacode.org/gf/project/joomla/tracker/?action=TrackerItemEdit&tracker_item_id=31565
BTW: The fix doesn't fix the issue with CamelCased view names as discussed in here:
http://forum.joomla.org/viewtopic.php?f=728&t=811158
ExampleViewFooBar tries still to look template files from views/foo/tmpl
I would revert the original change (referenced above) and use derived class/function instead of JViewLegacy::getName() in the new HMVC.