WordPress.org

Ready to get started?Download WordPress

Forums

Simple Staff List
Patch: Fix compatibility issue with CryptX (5 posts)

  1. Nick Ciske
    Member
    Posted 11 months ago #

    The antispambot() function was breaking CryptX's antispam features. I put my patch in a gist. Hope it can make it into a future version!

    Changed lines are 103-108.

    Gist
    https://gist.github.com/nciske/6023318

    CryptX
    http://wordpress.org/plugins/cryptx/

    http://wordpress.org/extend/plugins/simple-staff-list/

  2. Brett Shumaker
    Member
    Plugin Author

    Posted 11 months ago #

    Hey Nick,

    antispambot() is a function in WordPress core and I don't think a plugin should have to check if a specific plugin is installed to be able to use it. If anything, the plugin that breaks if a core WordPress function is used, should be patched to handle that situation. I'm open to debating this thinking, though.

    That being said, I don't think this will be merged into SSL, though I appreciate your willingness to submit a potential fix.

  3. Nick Ciske
    Member
    Posted 11 months ago #

    Ok, making it check for CryptX was probably a bit narrow minded ;-)

    How about allowing me to turn off anitspambot encoding via an option, filter, or constant?

  4. Nick Ciske
    Member
    Posted 11 months ago #

    Gist updated:
    https://gist.github.com/nciske/6023318

    Usage:
    add_filter('sslp_encode_emails','__return_false');

  5. Brett Shumaker
    Member
    Plugin Author

    Posted 11 months ago #

    Hey Nick,
    I think the filter option is a good compromise for this issue. I'll roll that change into the next version.

Reply

You must log in to post.

About this Plugin

About this Topic

Tags

No tags yet.