WordPress.org

Ready to get started?Download WordPress

Forums

MailPoet Newsletters
[resolved] Use WP_DEBUG before update your plugin (5 posts)

  1. Sergej Müller
    Member
    Posted 3 months ago #

    The plugin creates several errors.

    Please use WordPress prepare https://codex.wordpress.org/Class_Reference/wpdb#Protect_Queries_Against_SQL_Injection_Attacks und WP_Error functions https://codex.wordpress.org/Class_Reference/WP_Error

    Always check your plugin with https://codex.wordpress.org/WP_DEBUG

    Error lines:
    Warning: mysql_real_escape_string() expects parameter 2 to be resource, object given in /www/htdocs/wp-content/plugins/wysija-newsletters/core/model.php on line 542

    Warning: mysql_error() expects parameter 1 to be resource, object given in /www/htdocs/wp-content/plugins/wysija-newsletters/helpers/install.php on line 485

    Warning: mysql_error() expects parameter 1 to be resource, object given in /www/htdocs/wp-content/plugins/wysija-newsletters/helpers/install.php on line 485

    Warning: mysql_error() expects parameter 1 to be resource, object given in /www/htdocs/wp-content/plugins/wysija-newsletters/helpers/install.php on line 485

    Warning: mysql_error() expects parameter 1 to be resource, object given in /www/htdocs/wp-content/plugins/wysija-newsletters/helpers/install.php on line 485

    Warning: mysql_error() expects parameter 1 to be resource, object given in /www/htdocs/wp-content/plugins/wysija-newsletters/helpers/install.php on line 485

    Warning: mysql_error() expects parameter 1 to be resource, object given in /www/htdocs/wp-content/plugins/wysija-newsletters/helpers/install.php on line 485

    Warning: mysql_error() expects parameter 1 to be resource, object given in /www/htdocs/wp-content/plugins/wysija-newsletters/helpers/install.php on line 485

    Warning: mysql_error() expects parameter 1 to be resource, object given in /www/htdocs/wp-content/plugins/wysija-newsletters/helpers/install.php on line 485

    Warning: mysql_error() expects parameter 1 to be resource, object given in /www/htdocs/wp-content/plugins/wysija-newsletters/helpers/install.php on line 485

    Warning: mysql_error() expects parameter 1 to be resource, object given in /www/htdocs/wp-content/plugins/wysija-newsletters/helpers/install.php on line 485

    Warning: mysql_error() expects parameter 1 to be resource, object given in /www/htdocs/wp-content/plugins/wysija-newsletters/helpers/install.php on line 485

    Warning: mysql_error() expects parameter 1 to be resource, object given in /www/htdocs/wp-content/plugins/wysija-newsletters/helpers/install.php on line 485

    Warning: mysql_error() expects parameter 1 to be resource, object given in /www/htdocs/wp-content/plugins/wysija-newsletters/helpers/install.php on line 485

    Warning: mysql_error() expects parameter 1 to be resource, object given in /www/htdocs/wp-content/plugins/wysija-newsletters/helpers/install.php on line 485

    Warning: mysql_error() expects parameter 1 to be resource, object given in /www/htdocs/wp-content/plugins/wysija-newsletters/helpers/install.php on line 485

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

  2. achensee
    Member
    Posted 3 months ago #

    +1

  3. Note: With WordPress 3.9 and PHP 5.5 WordPress will use mysqli. You shouldn't use any mysql_* functions directly. Use the wpdb API instead.

  4. Ben
    Member
    Plugin Author

    Posted 3 months ago #

    Hi Sergej,

    Thanks for the feedback, it's always good to be reminded of the basics.

    We actually write every single line of code with the WP_DEBUG on, it's a rule in the team.

    This being said, it's difficult for us to reproduce all the possible environments our users are having. (server configurations, WordPress versions, plugins installed, themes installed etc etc...)

    Anyway, I suppose that you're using the version 3.9 of WordPress. Am I right? We're actually preparing our code to be compatible with 3.9, it's not yet ready but we're getting there.

    Let me know if I'm wrong, and please do share your current configuration with us.

    Finally for a better and more intimate support, get in touch with us here:
    http://support.mailpoet.com/feedback/

    Cheers!
    Ben

  5. MailPoet Staff
    Member
    Plugin Author

    Posted 3 months ago #

    Closing this... (we're changing our code to be compliant with WP 3.9, don't worry guys)

Reply

You must log in to post.

About this Plugin

About this Topic

Tags

No tags yet.