WordPress.org

Plugin Directory

akismet

Opened 10 months ago

Closed 5 weeks ago

#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 10 months ago.
Use AJAX to paginate through comments for "Check for Spam" button.

Download all attachments as: .zip

Change History (5)

jondavidjohn10 months ago

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

comment:1 tellyworth10 months ago

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

comment:2 jondavidjohn10 months ago

  • Cc jondavidjohn added

comment:3 devesine10 months ago

  • Cc devesine added

comment:4 eoigal5 weeks 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.