WordPress.org

Ready to get started?Download WordPress

Forums

[Plugin: PixoPoint Email Submit] php implode() error (8 posts)

  1. brian914
    Member
    Posted 1 year ago #

    I modified this plugin a bit but I don't think the issue is from what I did. Basically, the plugin works fine, expect if I take out all the emails from the form field that holds them in the admin.

    I am getting the following error:

    Notice: Undefined variable: email_list in /home/content/70/9724470/html/temp/wp-content/plugins/pixopoint-email-submit-modified/admin_page.php on line 47

    Warning: implode() [function.implode]: Invalid arguments passed in /home/content/70/9724470/html/temp/wp-content/plugins/pixopoint-email-submit-modified/admin_page.php on line 47

    Warning: Cannot modify header information - headers already sent by (output started at /home/content/70/9724470/html/temp/wp-content/plugins/pixopoint-email-submit-modified/admin_page.php:47) in /home/content/70/9724470/html/temp/wp-includes/pluggable.php on line 881

    The area it is referring to is this:

    // Recombine email addresses
    $input = implode( ',', $email_list );

    I don't really know what to with this?

    Any help would awesome!

  2. Ryan Hellyer
    Member
    Posted 1 year ago #

    Thanks for the bug report. Please try the latest version and let me know if that fixes the problem for you.

    Also, you may find you receive faster reports from plugin developers if you use that plugins specific support forum here on WordPress.org ... ie: http://wordpress.org/support/plugin/pixopoint-email-submit. I only discovered this topic as I had a feed of anything with my name or "PixoPoint" in it that is indexed in Google. If this page wasn't indexed, or it was indexed without something I have an alert for, I would not have seen it.

  3. brian914
    Member
    Posted 1 year ago #

    Hi Ryan,
    thank you so much to respond to my issue. I will use the other forum next time!

    I had modified the version of your plugin I had, since I wanted the email field etc to look different. I had changed the form html in the index file starting at line 60.

    I did deactivate my version and activate the latest version of the plugin and got the following error:
    The plugin generated 356 characters of unexpected output during activation. If you notice “headers already sent” messages, problems with syndication feeds or other issues, try deactivating or removing this plugin.

    Is there a way for me to use the plugin as is, so it can be updated regularly and maybe change the from html via my functions.php file?

    Thanks a lor for the help!

  4. Ryan Hellyer
    Member
    Posted 1 year ago #

    Oops, I didn't fix all the debug errors. This is lesson in a why not code plugins after drinking a mixture of wine and beer! I'll have an update out soon which will correct for that bug.

    You can customise the HTML output specific to your site by outputting the HTML directly rather than using the built in function in the plugin. So long as the form is submitting the correct POST data to WordPress the plugin will process it just fine. That way you would never need to edit the plugin files directly. I could add a filter or templating layler to the plugin, but I think this way is probably easier for you (and definitely is for me).

    I think the following should work (completely untested):

    <form class="pixopoint-emailsubmit" method="post" action="">
    	<label>Email</label>
    	<input type="text" value="" name="pixopoint_emailsubmit" onfocus="if (this.value == \'Enter email address\') {this.value = \'\';}" onblur="if (this.value == \'\') {this.value = \'Enter email address\';}" />
    	<input type="submit" value="Submit" />
    </form>
  5. brian914
    Member
    Posted 1 year ago #

    Ha, funny! No problem, I will check for the update in the next few days?

    I am not completely sure I understand what you are saying in terms of the form. So I would add that form to my template, instead of the shortcode call I have now? Here is what I currently have:
    do_shortcode( '[emailsubmit]' );

    Thanks a bunch for the help!

  6. Ryan Hellyer
    Member
    Posted 1 year ago #

    The new version was ready before you even replied.

    And yep, add the HTML to your template directly and don't worry about the shortcode. All the shortcode does is output HTML, so you can just bypass it and spit it out directly.

  7. brian914
    Member
    Posted 1 year ago #

    Okay, awesome!!! I will try that a little bit later!

    Thank you so much!!!

  8. brian914
    Member
    Posted 1 year ago #

    Hey Ryan,
    I just made the changes and am now using the plugin unchanged with the from in my file, instead of the shortcode., but am still getting the same error. I downloaded the latest version.

    I am getting the error when I cut emails from the Settings > Email Submissions box and click the update button. Here is what I get:

    Notice: Undefined variable: email_list in /home/content/70/9724470/html/wp-content/plugins/pixopoint-email-submit/admin_page.php on line 47
    
    Warning: Cannot modify header information - headers already sent by (output started at /home/content/70/9724470/html/wp-content/plugins/pixopoint-email-submit/admin_page.php:47) in /home/content/70/9724470/html/wp-includes/pluggable.php on line 881

    The url that shows where I get this is the options.php file.
    Also, once i get back to that window, it now shows me the following:

    <br />
    <b>Warning</b>:  explode() expects parameter 2 to be string, array given in <b>/home/content/70/9724470/html/wp-content/plugins/pixopoint-email-submit/index.php</b> on line <b>120</b><br />
    <br />
    <b>Warning</b>:  Invalid argument supplied for foreach() in <b>/home/content/70/9724470/html/wp-content/plugins/pixopoint-email-submit/index.php</b> on line <b>123</b><br />

    Is this something I am doing wrong?

    Thanks again for the help!

Topic Closed

This topic has been closed to new replies.

About this Topic