Skip to:
Content

BuddyPress.org

Opened 13 years ago

Closed 13 years ago

Last modified 12 years ago

#5068 closed defect (bug) (fixed)

A Notice error is thrown if replying to an activity in Activity Administration

Reported by: imath Owned by: boonebgorges
Priority: low Milestone: 1.9
Component: Activity Version: 1.8
Severity: minor Keywords: has-patch
Cc:

Description

In the administration of the activity, if i ajax reply to an activity, then it throws a notice error about the Undefined index: hook_suffix in /wp-admin/includes/screen.php at line 402. The problem seems to be that the screen argument is not defined in the construct method of BP_Activity_List_Table.

So i suggest the diff attached to this ticket which also strips the call to get_current_screen() in the prepare_items() function and replaces $screen->id by $this->screen->id in the $per_page var.

Attachments (1)

5068.diff​ (980 bytes ) - added by imath 13 years ago.

Download all attachments as: .zip

Change History (4)

@imath
13 years ago

#1 @boonebgorges
13 years ago

  • Keywords 2nd-opinion removed
  • Milestone Awaiting Review → 1.9
  • Priority normal → low

Looks good at a glance, but I'd like to wait until 1.9, as the current notice isn't hurting anything and I haven't dug deep enough into the root issues.

#2 @boonebgorges
13 years ago

  • Owner set to boonebgorges
  • Resolution → fixed
  • Status new → closed

In 7473:

Avoid PHP notice when leaving activity comments in Activity Admin.

Fixes #5068

Props imath

#3 @johnjamesjacoby
12 years ago

  • Version 1.8-beta → 1.8
Note: See TracTickets for help on using tickets.