Plugin Directory

akismet

Opened 12 years ago

Closed 12 years ago

#1789 closed defect (fixed)

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

Reported by: jondavidjohn's profile jondavidjohn's profile jondavidjohn Owned by: tellyworth's profile tellyworth's profile 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 12 years ago.
Use AJAX to paginate through comments for "Check for Spam" button.

Download all attachments as: .zip

Change History (5)

@jondavidjohn
12 years ago

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

#1 @tellyworth
12 years ago

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

#2 @jondavidjohn
12 years ago

  • Cc jondavidjohn added

#3 @devesine
12 years ago

  • Cc devesine added

#4 @eoigal
12 years 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.