Dublicate Template Styles #93

Closed
wants to merge 6 commits into
from

2 participants

@hieblmedia

Dublicate template styles if other type of extension exists with the same name.

I have added a filter with type=template for the query.

Also affected is the form field templatestyles, see:
joomla/joomla-platform#761

See http://joomlacode.org/gf/project/joomla/tracker/?action=TrackerItemEdit&tracker_item_id=27780

@infograf768
Joomla! member

Please post issue in the tracker.

@chdemko chdemko commented on an outdated diff Jan 20, 2012
administrator/components/com_templates/models/styles.php
@@ -123,6 +123,7 @@ protected function getListQuery()
$query->select('extension_id AS e_id');
$query->join('LEFT', '#__extensions AS e ON e.element = a.template');
$query->where('e.enabled = 1');
+ $query->where('e.type=' . $db->quote('template'));
@chdemko
chdemko added a line comment Jan 20, 2012

could you use

$query->where($db->quoteName('e.type') . '='  . $db->quote('template'));

because 'type' could be a reserved word in some db engine

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
@hieblmedia

Moved templatestyle form field, from plattform to cms library.
See: joomla/joomla-platform#761 (comment)

@hieblmedia

I have reverted the commit to add the templatestyle form field. Because it is handled by the platform repository.

joomla/joomla-platform#761 (comment)
joomla/joomla-platform#814

@hieblmedia

Fxied in 4631a2d

@hieblmedia hieblmedia closed this Mar 5, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment