WordPress.org

Plugin Directory

akismet

#1789 closed defect (fixed)

Use AJAX pagination for the "Check for Spam" button.

Reported by: jondavidjohn Owned by: tellyworth
Priority: high Severity: major
Plugin: akismet Keywords: has-patch
Cc: jondavidjohn, devesine

Description

We've run into issues where we have a massive amount of comments. The current "Check for Spam" button's implementation times out because there is no restriction on how many comments to process.

So once a user of this plugin hits an arbitrary number of unapproved comments (depending on server configuration), the "Check for Spam" button breaks and stops working as expected

This patch runs the "Check for Spam" routine via ajax pagination and reloads once complete. This seems like a more bounded and controlled approach.

Attachments (1)

ajax-paginate-checkforspam.diff (7.4 KB) - added by jondavidjohn 22 months ago.
Use AJAX to paginate through comments for "Check for Spam" button.

Download all attachments as: .zip

Change History (5)

@jondavidjohn22 months ago

Use AJAX to paginate through comments for "Check for Spam" button.

comment:1 @tellyworth22 months ago

  • Owner changed from matt to tellyworth
  • Status changed from new to assigned

comment:2 @jondavidjohn22 months ago

  • Cc jondavidjohn added

comment:3 @devesine22 months ago

  • Cc devesine added

comment:4 @eoigal13 months ago

  • Resolution set to fixed
  • Status changed from assigned to closed

Check for Spam button now uses ajax pagination

Note: See TracTickets for help on using tickets.