WordPress.org

Plugin Directory

akismet

Opened 2 years ago

Closed 17 months 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 2 years ago.
Use AJAX to paginate through comments for "Check for Spam" button.

Download all attachments as: .zip

Change History (5)

@jondavidjohn2 years ago

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

comment:1 @tellyworth2 years ago

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

comment:2 @jondavidjohn2 years ago

  • Cc jondavidjohn added

comment:3 @devesine2 years ago

  • Cc devesine added

comment:4 @eoigal17 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.