WordPress.org

Ready to get started?Download WordPress

Forums

Campaign Monitor Ajax Forms
please use new createsend API version (5 posts)

  1. Carlo Roosen
    Member
    Posted 4 months ago #

    Hi Lee,

    We have two Campaign Monitor plugins here on wordpress.org (campaign-monitor-synchronization and campaign-monitor-dual-registration), and there seems to be a conflict between your plugin and ours.

    The problem lies in duplicate loading of the Createsend API. The newer versions of this API prevent duplicate loading, but it seems your code uses an older version. Could you have a look please?

    carloroosen.com
    Carlo Roosen

    https://wordpress.org/plugins/ajax-campaign-monitor-forms/

  2. Lee Willis
    Member
    Plugin Author

    Posted 4 months ago #

    Hi Carlo,

    My plugin already checks if the class exists before loading the API (See https://plugins.trac.wordpress.org/browser/ajax-campaign-monitor-forms/trunk/cm-ajax.php#L14). If you're seeing duplicate inclusions then that suggests that your plugin is being loaded after mine, and *doesn't* include those same checks.

    I can't see anything in the latest version of the CM API that would prevent this (https://github.com/campaignmonitor/createsend-php/search?q=class_exists&ref=cmdform)

    Can you provide more info on why you think picking up the latest version would help here?

  3. Lee Willis
    Member
    Plugin Author

    Posted 4 months ago #

    PS. Details of the actual error you're seeing would help assess the situation a bit better

  4. Elena Mukhina
    Member
    Posted 4 months ago #

    Hi Lee,

    Our plugin performs this check too. The issue is slightly different. We use the latest Createsend API version, and everything works properly. But our plugin generates the following error

    Campaign Monitor synchronization error. Must supply a valid HTTP Basic Authorization header

    when your plugin is installed and the old API files are used. When I replace API files in your plugin folder everything works perfectly again.
    You can read in more details here http://wordpress.org/support/topic/sync-error?replies=11.

  5. Lee Willis
    Member
    Plugin Author

    Posted 4 months ago #

    Hi,

    I've updated my plugin to use the latest version of the CM API. Let me know how you get on.

Reply

You must log in to post.

About this Plugin

About this Topic

Tags

No tags yet.