• Resolved stevem

    (@stevem)


    In version 2.6.12 if I preview a yet to be published newsletter I get
    Warning: Illegal string offset ‘color’ in /home/sarcsout/public_html/wp-content/plugins/wysija-newsletters/helpers/render_engine.php on line 109
    repeated 21 times and then the preview is shown correctly.
    Emailing the preview is fine though.

    https://wordpress.org/plugins/wysija-newsletters/

Viewing 10 replies - 1 through 10 (of 10 total)
  • Hi Stevem,

    I’ve tested MailPoet with our debug on and with wp_debug on and I couldn’t see these Warnings you’re talking about. I’ve also tried on a automatic newsletter but couldn’t see the warning.

    Can you please elaborate a little more what are you trying to do, this way you’ll help us find this issue.

    Thanks for your feedback by the way 😀

    Thread Starter stevem

    (@stevem)

    I have a weekly automatic newsletter which currently doesn’t have any posts in it, so it just contains the header and footer. If I go to
    Mailpoet, Newsletters, then hover over the automatic newsletter and choose Preview (which is next to Edit, Duplicate, Delete) then a new tab opens with 21 lines repeating the error, followed by the correct preview of the newsletter.

    Does that help?

    It does, I just tried with the exact same setup and couldn’t see any Warnings.

    Can you please get in touch with us here at http://support.mailpoet.com/ we’ll need to ask you private details about your website.

    Please, don’t forget to refer this topic on the contact. Thanks!

    Thread Starter stevem

    (@stevem)

    OK, I have done that.

    Hello guys, I’m seeing the same behaviour in my website. I’m seeing it in premium MailPoet but not in free versions.

    Can you share the solution here if you find something? Please?
    Thanks in advance.

    Thread Starter stevem

    (@stevem)

    Mailpoet support have solved this problem with a change to line 109 in render_engine.php. They say the error only occurs in PHP 5.4 and that the next update will correct it.

    You can manually change it before then. This is their solution.
    Change line 109 render_engine.php

    from

    if (isset($vars['border'])) $result[] = 'border: ' . $vars['border']['size'] . ' ' . $vars['border']['style'] . ' ' . $vars['border']['color'] . ';';

    to

    if (isset($vars['border']) && is_array($vars['border'])) $result[] = 'border: ' . $vars['border']['size'] . ' ' . $vars['border']['style'] . ' ' . $vars['border']['color'] . ';';

    Thanks for the heads-up Steve. Did they told you about any ETA for the next update?

    Thread Starter stevem

    (@stevem)

    I’m afraid not. They said “We’ll publish a new version soon which will include this fix.

    Closing…

Viewing 10 replies - 1 through 10 (of 10 total)
  • The topic ‘Preview error’ is closed to new replies.