Closed (fixed)
Project:
Views Crosstab
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
26 Oct 2014 at 18:21 UTC
Updated:
7 Nov 2017 at 20:44 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
joel_guesclin commentedComment #2
scotwith1tI was getting similar messages and changing
to
in plugins/views_crosstab_table.inc, line 463 got rid of the warnings for me.
Comment #3
danchadwick commented@scotself: Doing so will disable the ability to add a summary row. There is an issue outstanding with the views module to fix the definition in views, but I'm sure it will never get committed. Ignore the warning if you want the feature.
Comment #4
scotwith1tHmmm. Indeed. The summary row is then incorrect. Can you point us to the views fix if someone has come up with a patch? Having settings.php turn off strict warnings seems a bit over the top and risks missing some other potentially important warnings?
When i put the '&' back i have more errors than just the one above, I assume they are all related since they all go away when I remove the '&'?
Comment #6
danchadwick commentedI won't call this "fixed" but rather mitigated. Instead of honoring the argument passed to ::pre_render(), I put the summary row right into the view result. This would break any calls to pre_render that aren't passed ($this->result), but at least it gets rid of the log-clogging warnings, which you can't easily suppress with PHP7.