From: Duccio Gasparri <duccio.gaspa...@gmail.com>
Date: Thu, 19 Apr 2012 07:19:23 -0700 (PDT)
Local: Thurs, Apr 19 2012 10:19 am
Subject: Re: [jcms] Object oriented components , any plan?
I did pull 193 (about JController::factory) and 194 (componentHelper) The real issue with com_media is from line 43 to 75 of media.php:
$cmd = JRequest::getCmd('task', null);
if (strpos($cmd, '.') != false) {
// Set the name for the controller and instantiate it
if (class_exists($controllerClass)) {
}
else {
JError::raiseError(500, JText::_('JERROR_INVALID_CONTROLLER_CLASS')); }
because it does exactly what the new renderComponent would do. It is about simplifying it rather than recoding it, but before moving forward with all the libraries I'd like to hear if the pulls I did on GitHub are in the right direction. I'm stuck in those two issues: 1) do we want to mantain both com_example/example.php and There are two kinds of stuff that could be done at startup, the general Four options come to my mind:
The more we add methods, the less clear the code will be to understand and I have no clue on what could be the best option...
On Thursday, April 19, 2012 2:49:24 PM UTC+2, elin wrote:
> The scary one to look at is com_media in the front end, someone needs to > In com_banners we have ...
> $controller->execute(JRequest::getVar('task', 'click'));
> Also ... we are requiring all these specific helpers by name but why not > Elin
> On Tuesday, April 17, 2012 4:08:15 PM UTC-4, Rouven Weßling wrote:
>> That has already happened. However I don't think that impacts Duccio's >> What I'm wondering is, if components currently do anything else in their >> Also I wonder if there's a backward compatibility impact.
>> Otherwise I quite like the idea, it could help reduce the amount of >> Rouven
You must Sign in before you can post messages.
To post a message you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
| ||||||||||||||