Contact info field to short (255) + Image links in contact info get broken

Hi,
I tried to use to contact information field in order to add some extra footer content including image logo.
But it is too short (255) characters.
I managed to solve this by ALTERING in database
1. the wp_enewsletter_settings COLUMN value to longtext and
2. the wp_enewsletter_newsletters COLUMN contact_info to longtext

I think it would be nice if you fix this in the next version, by changing the default table CREATEs in the plugin setup.

Also my images urls in the contact_info are getting broken.
I type
<img src="http://MYSITE/wp-content/themes/MYTHEME/_inc/images/MYIMAGE.jpg">
but after I in the newsletter mail it gets:
<img src="http:/MYSITE/wp-content/themes/MYTHEME/_inc/http://MYSITE/wp-content/plugins/e-newsletter/email-newsletter-files/templates/MYNEWSLETTERTEMPLATE/images/MYIMAGE.jpg

How can I fix it?
Thanks in advance
Lena

    stergatu

    Hi KimberlyL.
    I don't want to have the images in many locations (theme/images/, plugin1/images, plugin2/images/ etc), but to use the default location.
    I found why the rewrite happens of the image location and a fix for it.
    Please let the developer known.

    In /e-newsletter.php replace the 2 occurancies of
    $contents = str_replace( "images/", $this->plugin_url . "email-newsletter-files/templates/" . $_REQUEST['template'] . "/images/", $contents );

    with
    $contents = str_replace( 'src="images/', 'src="'.$this->plugin_url . 'email-newsletter-files/templates/' . $_REQUEST['template'] . '/images/', $contents );

    and the 1 occurance of
    $contents = str_replace( "images/", $this->plugin_url . "email-newsletter-files/templates/" . $newsletter_data['template'] . "/images/", $contents );

    with
    $contents = str_replace( 'src="images/', 'src="'.$this->plugin_url . 'email-newsletter-files/templates/' . $newsletter_data['template'] . '/images/', $contents );

    so the function won't change the src of external images

    Best
    Lena

    Cole

    Hey stergatu

    I can understand why in your particular case you would want to have that column changed to longtext. Have you looked at modifying the template footer directly instead?

    Nice snippets for the str_replace(). That seems like a pretty good solution for external images. I will need to make sure there are no complications with the default themes by doing that but I think it should be alright. I will include the image fix in the next version.

    The database column changing to longtext won't make it I'm afraid. This is the first case I have seen where someones content exceeded the table column and I would be wary to force a bunch of upgrading functions that cause havoc down the road.

    What I will do though is allow eNewsletter themes to be installed from a location outside of the eNewsletter plugin folder. This should make any templates you have update safe in the future making it a more viable solution for the footer adjustments you are wanting to do..

    Thanks so much. Please let me know if you have any other questions.

    Cole