• Resolved Kylen Downs

    (@kdowns)


    Version 5.187 appears to have added a column to the sites screen to display the status of the cleantalk API key. Unfortunately, it appears to be displaying the status in multiple columns instead of limiting it into just the “Cleantalk Status” column. This appears to affect any custom columns.

    Screenshot: https://freeimage.host/i/DjMTMv

Viewing 8 replies - 1 through 8 (of 8 total)
  • Plugin Support sergecleantalk

    (@sergecleantalk)

    Hello,

    Could you try to deactivate the plugin with the “Complete deactivation” option enabled and then activate it again?

    1. Enter the Antispam by Cleantalk plugin settings on your main website. (WordPress Dashboard -> Settings -> Antispam by Cleantalk -> Advanced settings)

    2. Check the last settings “Complete deactivation” and save changes.

    3. Deactivate the plugin on the Network “Plugins” page in WordPess Dashboard.

    4. Then activate it.

    5. Proceed to the plugin’s settings. (See item 1.)

    Did it help?

    Thread Starter Kylen Downs

    (@kdowns)

    Complete deactivation and reactivation did not resolve the issue. It did clear out the status message for all sites except the primary site, however the message is still duplicated across multiple columns. The message also returns for subsites as soon as I logged into them.

    I also reverted the plugin back to 5.186 and the issue is not present in that version, but as soon as the plugin is updated to 5.187 the issue returns.

    Thread Starter Kylen Downs

    (@kdowns)

    Looking into this locally, the apbct__manage_sites_custom_column_action() function on cleantalk-spam-protect/inc/cleantalk-admin.php:1244 is not passing any sort of validation or check on $_column_name which means the output is occurring on all custom columns.

    The cleantalk status column does not have a specific name, but it can be validated against the int(0) with this replacing line 1261:

    if ($_column_name == 0){
            echo $key_status_caption;
        }
    • This reply was modified 1 year, 9 months ago by Kylen Downs.
    Plugin Support eugenecleantalk

    (@eugenecleantalk)

    Thankyou for your tip.

    I passed the information to our developers.

    We will contact you within 3 business days.

    Plugin Support eugenecleantalk

    (@eugenecleantalk)

    Your solution is correct. Thank you.

    We added it to the code of our plugin. The fix version can be downloaded here: https://github.com/CleanTalk/wordpress-antispam/releases/download/fix-version/cleantalk-spam-protect.zip.

    The changes made will also be included in the next release of the plugin.

    Thread Starter Kylen Downs

    (@kdowns)

    Cheers! I appreciate the plugin and support, you guys are awesome!

    As better solution than the one I provided, I might suggest actually naming the custom cleantalk column as I can foresee potential conflicts if the logic is left as targeting the int(0) value. By naming the column, it can ensure the if() statement targets the specific cleantalk column to avoid the echo outputting on a different custom column if that other column also happens to be unnamed.

    Plugin Support eugenecleantalk

    (@eugenecleantalk)

    Thank you for your kind words!

    I’ve passed your details to our programmer staff.

    Plugin Support eugenecleantalk

    (@eugenecleantalk)

    It was these edits that our developer made to the plugin code.

Viewing 8 replies - 1 through 8 (of 8 total)
  • The topic ‘Multisite site column output bug’ is closed to new replies.