pre_render() compatibility [#1427960]

http://localhost/content/sisr-0#overlay=group/node/2/admin/people

Strict warning: Declaration of og_handler_field_user_roles::pre_render() should be compatible with that of views_handler_field::pre_render() in require_once() (line 6 of C:\xampp\htdocs\aegis\sites\all\modules\og\includes\views\og_handler_field_user_roles.inc).

Comment	File	Size	Author
#3	og-handler-field-strict-warning-1427960-3.patch	541 bytes	bulldozer2003
#3

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

amitaibu

Hebrew

Israel

CreditAttribution: amitaibu commented 4 February 2012 at 20:42

Are you using the latest views version?

Comment #2

jaypark CreditAttribution: jaypark commented 4 February 2012 at 20:47

stable versions of
Views 7.x-3.1
Organic groups Views integration 7.x-1.0

Comment #3

bulldozer2003

king/sire

CreditAttribution: bulldozer2003 commented 13 March 2012 at 15:49

Status:

Active

» Needs review

File	Size
og-handler-field-strict-warning-1427960-3.patch	541 bytes

Views expects a reference (&) to $values. Changing pre_render($values) to pre_render(&$values) in og_handler_field_user_roles.inc removes the warning.

I also checked if $values is being altered; it is not.