Success

User tests: Successful: Unsuccessful:

avatar bembelimen
bembelimen
26 Jul 2013

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

avatar bembelimen bembelimen - open - 26 Jul 2013
avatar matrikular matrikular - reference | - 26 Jul 13
avatar mahagr
mahagr - comment - 26 Jul 2013

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.

avatar dongilbert
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. :)

avatar phproberto
phproberto - comment - 26 Jul 2013

@dongilbert can you please create a PR for easy testing?

Thanks!

avatar dongilbert
dongilbert - comment - 26 Jul 2013

Here you go: #1595

avatar Hackwar
Hackwar - comment - 5 Dec 2013

Is this still valid or can this one be closed?

avatar mahagr
mahagr - comment - 7 Dec 2013

Looks like it's fixed.

avatar Bakual
Bakual - comment - 7 Dec 2013

Closing as fixed with c6ea14f

avatar Bakual Bakual - change - 7 Dec 2013
Status New Closed
Closed_Date 0000-00-00 00:00:00 2013-12-07 17:04:46
avatar Bakual Bakual - close - 7 Dec 2013
avatar Bakual Bakual - close - 7 Dec 2013

Add a Comment

Login with GitHub to post a comment