Skip to content

Commit

Permalink
Some fixes for merge repeat formatting in lists
Browse files Browse the repository at this point in the history
  • Loading branch information
cheesegrits committed Apr 6, 2016
1 parent 5cbc18d commit 16321f6
Show file tree
Hide file tree
Showing 4 changed files with 16 additions and 4 deletions.
9 changes: 6 additions & 3 deletions components/com_fabrik/helpers/html.php
Expand Up @@ -2292,11 +2292,13 @@ public static function canvasSupport()
public static function runContentPlugins(&$text)
{
$app = JFactory::getApplication();
$input = $app->input;
$opt = $input->get('option');
$view = $input->get('view');
$input = $app->input;
$opt = $input->get('option');
$view = $input->get('view');
$format = $input->get('format');
$input->set('option', 'com_content');
$input->set('view', 'article');
$input->set('format', 'html');
jimport('joomla.html.html.content');

/**
Expand All @@ -2322,6 +2324,7 @@ public static function runContentPlugins(&$text)

$input->set('option', $opt);
$input->set('view', $view);
$input->set('format', $format);
}

/**
Expand Down
5 changes: 5 additions & 0 deletions components/com_fabrik/models/element.php
Expand Up @@ -5951,6 +5951,11 @@ public function renderListData($data, stdClass &$thisRow, $opts = array())

foreach ($data as $i => &$d)
{
if (empty($d))
{
$d = ' ';
}

if ($params->get('icon_folder') == '1' && ArrayHelper::getValue($opts, 'icon', 1))
{
// $$$ rob was returning here but that stopped us being able to use links and icons together
Expand Down
4 changes: 4 additions & 0 deletions components/com_fabrik/models/elementlist.php
Expand Up @@ -655,6 +655,10 @@ public function renderListData($data, stdClass &$thisRow, $opts = array())
{
$lis[] = $l;
}
else
{
$lis[] = ' ';
}
}

if (!empty($lis))
Expand Down
2 changes: 1 addition & 1 deletion plugins/fabrik_element/date/date.php
Expand Up @@ -70,7 +70,7 @@ public function renderListData($data, stdClass &$thisRow, $opts = array())
{
if ($data == '')
{
return '';
return parent::renderListData($data, $thisRow, $opts);
}

// @TODO: deal with time options (currently can be defined in date_table_format param).
Expand Down

0 comments on commit 16321f6

Please sign in to comment.