Skip to content

Commit

Permalink
Fix for non-SEF'ed URL in list email plugin, plus normalize links if …
Browse files Browse the repository at this point in the history
…WYSIWYG.
  • Loading branch information
cheesegrits committed Mar 1, 2018
1 parent 40826dd commit d85a721
Showing 1 changed file with 12 additions and 1 deletion.
13 changes: 12 additions & 1 deletion plugins/fabrik_list/email/models/email.php
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@
// No direct access
defined('_JEXEC') or die('Restricted access');

use Fabrik\Helpers\Pdf;
use Fabrik\Helpers\StringHelper;
use Joomla\Utilities\ArrayHelper;

Expand Down Expand Up @@ -129,7 +130,7 @@ public function onLoadJavascriptInstance($args)
$url .= '&' . $opts->additionalQS;
}

$opts->popupUrl = JRoute::_($url);
$opts->popupUrl = JRoute::_($url, false);
$opts = json_encode($opts);
$this->jsInstance = "new FbListEmail($opts)";

Expand Down Expand Up @@ -687,6 +688,11 @@ private function _send($row, $mailTo)

$thisMsg = $w->parseMessageForPlaceholder($thisMsg, $row);

if ($params->get('wysiwyg', true))
{
Pdf::fullPaths($thisMsg);
}

if ($sendSMS)
{
return $this->sendSMS($mailTo, $thisMsg, $row);
Expand Down Expand Up @@ -962,6 +968,11 @@ private function mailMerged($firstRow, $mergedMsg, $sent, $notSent)
$mergedMsg = $coverMessage . $mergedMsg;
}

if ($params->get('wysiwyg', true))
{
Pdf::fullPaths($coverMessage);
}

if ($toHow == 'single')
{
foreach ($thisTos as $toKey => $thisTo)
Expand Down

0 comments on commit d85a721

Please sign in to comment.