[#8474] - support Multi path for controller
- Closed
- 22 Jun 2017
- Medium
- Build: staging
- # 8474
- Diff
- mvanvu:patch-2
- Success continuous-integration/travis-ci/pr The Travis CI build passed Details
User tests: Successful: Unsuccessful:
Thanks for great Joomla! MVC structure. The suggest for support multi base path for JControllerLegacy.
My solutions are create a method addIncludePath and custom the method getInstance.
Here is the example for test. https://github.com/mvanvu/com_hello
description: com_hello just has a view hello in the backend and frontend. So the plugin comment will provide new views comment list and add new comment for the both frontend and backend.
P/s: com_hello just test for this request.
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Labels |
Added:
?
|
||
zero-24
- | Category | ⇒ | Libraries |
@mvanvu Can you please provide steps to test this PR?
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/8474.
@RonakParmar Please see com_hello https://github.com/mvanvu/com_hello. As default the com_hello has two views (hellos and hello) and the plugin comment (https://github.com/mvanvu/com_hello/tree/master/admin/plugin/comment) which provides more views by use JController::addIncludePath.
Is this PR actually necessary to achieve what you want.
Can you not use
JControllerLegacy::getInstance("Content", array("model_prefix" => "yourmodelname", "model_path" => DIR . "/models/"));
or if you have n instance already
$controller->addViewPath( " your new path ");
or
JModelLegacy::addIncludePath
etc.
@nibra I would suggest closing this in 1 week if there is no response
| Status | Pending | ⇒ | Information Required |
set Status on "Information Required".
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/8474.
If this PR get no Response, it will be closed at 23th July 2017.
| Status | Information Required | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2017-06-22 10:14:37 |
| Closed_By | ⇒ | mvanvu |
@mvanvu See mvanvu#1