Plugin Directory

akismet

Opened 11 years ago

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

Download all attachments as: .zip

Change History (5)

@jondavidjohn
11 years ago

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

#1 @tellyworth
11 years ago

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

#2 @jondavidjohn
11 years ago

  • Cc jondavidjohn added

#3 @devesine
11 years ago

  • Cc devesine added

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