Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

In Row evolution popover may return "Call to a member function toString() on null" #12734

Closed
mattab opened this issue Apr 17, 2018 · 1 comment · Fixed by #12786
Closed

In Row evolution popover may return "Call to a member function toString() on null" #12734

mattab opened this issue Apr 17, 2018 · 1 comment · Fixed by #12786
Assignees
Labels
Bug For errors / faults / flaws / inconsistencies etc.
Milestone

Comments

@mattab
Copy link
Member

mattab commented Apr 17, 2018

As we get emails when there are errors, we want to prevent this error from being triggered. This was only triggered once in production.

Message: Error: {"message":"Call to a member function toString() on null","file":"plugins\/CoreHome\/DataTableRowAction\/RowEvolution.php","line":106}

URL where this was reproduced: index.php?date=2018-04-16&apiMethod=MultiChannelConversionAttribution.getChannelAttribution&label=Campaigns+%3E+%404b551fa251-email_campaign_2018_03_02&disableLink=1&module=CoreHome&action=getRowEvolutionPopover&colors=%7B%22backgroundColor%22%3A%22%23ffffff%22%2C%22lineColor%22%3A%22%23162c4a%22%2C%22minPointColor%22%3A%22%23ff7f7f%22%2C%22maxPointColor%22%3A%22%2375bf7c%22%2C%22lastPointColor%22%3A%22%2355aaff%22%7D&idGoal=0&idSite=1&period=day

@mattab mattab added the Bug For errors / faults / flaws / inconsistencies etc. label Apr 17, 2018
@mattab mattab added this to the 3.5.0 milestone Apr 17, 2018
@sgiehl
Copy link
Member

sgiehl commented Apr 29, 2018

I can provide a fix to prevent that error and throw an exception instead. But I wasn't able to reproduce that error with the given URL.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug For errors / faults / flaws / inconsistencies etc.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants