-
-
Notifications
You must be signed in to change notification settings - Fork 3.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Admin app - Use the autoloader #10881
Changes from all commits
5f19de3
32bccc4
b7afe6c
f61b29e
f3018d9
ec8218c
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -9,7 +9,7 @@ | |
|
||
defined('_JEXEC') or die; | ||
|
||
JLoader::register('BannersHelper', JPATH_COMPONENT . '/helpers/banners.php'); | ||
JLoader::register('BannersHelper', JPATH_ADMINISTRATOR . '/components/com_banners/helpers/banners.php'); | ||
|
||
/** | ||
* View class for a list of tracks. | ||
|
@@ -66,8 +66,6 @@ public function display($tpl = null) | |
|
||
$this->addToolbar(); | ||
|
||
require_once JPATH_COMPONENT . '/models/fields/bannerclient.php'; | ||
|
||
$this->sidebar = JHtmlSidebar::render(); | ||
|
||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Hi @mbabker There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I could not find a direct use of the class that this was loading in the code. Meaning either it's being loaded as part of a form (where the lookup path should be included in the XML or added with a |
||
return parent::display($tpl); | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -11,7 +11,7 @@ | |
|
||
JFormHelper::loadFieldClass('list'); | ||
|
||
require_once __DIR__ . '/../../helpers/installer.php'; | ||
JLoader::register('InstallerHelper', JPATH_ADMINISTRATOR . '/components/com_installer/helpers/installer.php'); | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This is right, but it is now using JPATH_ADMINISTRATOR+component_folder, it could have kept the usage of __ DIR __
I say this because in other cases you keep the usage of __ DIR __, but in some you do not There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
On Monday, July 18, 2016, Georgios Papadakis notifications@github.com
|
||
|
||
/** | ||
* Status Field class for the Joomla Framework. | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mbabker isn't this supposed to autoload by https://github.com/joomla/joomla-cms/blob/staging/libraries/legacy/model/form.php#L203-L206 ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you only use these forms going through JModelForm objects then sure. If
they load any other way the dependency wouldn't be loaded. I equate it to
relying on something else to load jQuery when you depend on it.
On Monday, June 20, 2016, Dimitri Grammatikogianni notifications@github.com
wrote: