WordPress.org

Forums

The Support Forums will be in read-only mode for a scheduled maintenance window on 01 September 2016 14:00 UTC - 20:00 UTC. More information.

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

  1. Nick Ciske
    Member
    Posted 3 years 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 3 years 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 3 years 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 3 years ago #

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

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

  5. Brett Shumaker
    Member
    Plugin Author

    Posted 3 years ago #

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

Topic Closed

This topic has been closed to new replies.

About this Plugin

  • Simple Staff List
  • Frequently Asked Questions
  • Support Threads
  • Reviews

About this Topic

Tags

No tags yet.