Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Looks like there's method compatibility warnings with the most recent version of views:
Strict warning: Declaration of flag_handler_field_ops::pre_render() should be compatible with that of views_handler_field::pre_render() in _registry_check_code() (line 2811 of includes/bootstrap.inc).
Strict warning: Declaration of flag_handler_relationship_content::ui_name() should be compatible with that of views_handler::ui_name() in require_once() (line 84 of sites/all/modules/flag/includes/flag_handler_relationships.inc).
Strict warning: Declaration of flag_handler_relationship_content::options_validate() should be compatible with that of views_handler::options_validate() in require_once() (line 84 of sites/all/modules/flag/includes/flag_handler_relationships.inc).
Strict warning: Declaration of flag_handler_relationship_counts::ui_name() should be compatible with that of views_handler::ui_name() in require_once() (line 178 of sites/all/modules/flag/includes/flag_handler_relationships.inc).
Comment | File | Size | Author |
---|---|---|---|
#16 | flag-strict-1119842-16.patch | 1.68 KB | rjay |
#14 | 1119842-14-views-strict-6.x-backport.patch | 2.01 KB | joelpittet |
#1 | flag-strict-1-1119842.patch | 1.99 KB | duellj |
Comments
Comment #1
duellj CreditAttribution: duellj commentedPatch attached to fix strict warnings
Comment #2
duellj CreditAttribution: duellj commentedComment #3
TimelessDomain CreditAttribution: TimelessDomain commentedpatch worked for me - thanks.
Comment #4
TimelessDomain CreditAttribution: TimelessDomain commentedNot related to this issue
the exposed filter for "Flags: Flagged" does not have an "-Any-" option so a view w/ this filter can only display flagged or not flagged content (not both). this should be fixedupdate#1: the "-Any-" selection of an exposed filter for "Flags: Flagged" is available when your view the view, but not when you edit the view. therefore you can't set the exposed "Flags: Flagged" filter to "-Any-" by default
update#2: #1126724: -Any- option missing from exposed filter for Flags: Flagged (@duellj - updated accordingly)
Comment #5
duellj CreditAttribution: duellj commented@TimelessDomain:
Thanks or the review. You should open up a new issue for the problem you listed in #4. This issue is just for strict warnings in flags/views integration.
Comment #6
Anonymous (not verified) CreditAttribution: Anonymous commentedPatch works for me as well!
Comment #7
lyricnz CreditAttribution: lyricnz commentedWorks for me too.
Comment #8
kolier CreditAttribution: kolier commentedWorking +1.
Comment #9
Cyberwolf CreditAttribution: Cyberwolf commentedSubscribing.
Comment #10
DamienMcKennaThis resolved the errors for me too.
Comment #11
Ingmar CreditAttribution: Ingmar commentedThank you very much!
Comment #12
quicksketchMost of this patch was included in the fix from #1119842: Strict warnings with Views 3.x, but I added the change to flag_handler_field_ops.inc in this patch, since that was not included in the previous patch.
Comment #14
joelpittetDon't know if this needs a backport to 6.x but here is the patch nonetheless.
Comment #15
stewart.adam CreditAttribution: stewart.adam commentedNote - the 6.x patch above doesn't fix all warnings for options_validate since the signature in Drupal 6 has $form passed by value, not reference.
should read:
Comment #16
rjay CreditAttribution: rjay at North Studio commentedThis is an update of the patch from #14, without the modification to the signature of the options_validate method.
Comment #17
joachim CreditAttribution: joachim commentedThanks!
Committed.