Plugin Directory


Opened 2 years ago

Closed 2 years ago

#1792 closed defect (fixed)

Omnisearch causing a PHP Error

Reported by: markjaquith Owned by: georgestephanis
Priority: normal Severity: normal
Plugin: jetpack Keywords: omnisearch has-patch
Cc: jeremy+wp@…


Error: Declaration of Jetpack_Omnisearch_Comments::pagination() should be compatible with WP_List_Table::pagination($which)

If you extend a class and override a method, the arguments and access levels have to match.

Attachments (1)

1792.diff (746 bytes) - added by markjaquith 2 years ago.

Download all attachments as: .zip

Change History (4)

@markjaquith2 years ago

comment:1 @markjaquith2 years ago

Untested, but matches the signatures in two instances I found.

comment:2 @jeherve2 years ago

  • Cc jeremy+wp@… added
  • Keywords omnisearch has-patch added
  • Owner changed from tmoorewp to georgestephanis

comment:3 @georgestephanis2 years ago

  • Resolution set to fixed
  • Status changed from new to closed

In 730887:

Jetpack: Omnisearch: Add parameter to overridden pagination() function. Fixes #1792 props @markjaquith

Note: See TracTickets for help on using tickets.