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

Jump to bottom

Error message CpChart triggered with specific payload #13505

Closed
mattab opened this issue Sep 29, 2018 · 1 comment · Fixed by #13509
Closed

Error message CpChart triggered with specific payload #13505

mattab opened this issue Sep 29, 2018 · 1 comment · Fixed by #13509
Assignees
@sgiehl
Labels
Bug For errors / faults / flaws / inconsistencies etc.
Milestone
3.6.1

Comments

@mattab
Copy link
Member

mattab commented Sep 29, 2018 •
edited

As seen on production:

First message (more below)

Error: {"message":"Argument 4 passed to CpChart\Chart\Draw::drawText() must be of the type array, boolean given, called in plugins/ImageGraph/StaticGraph/Exception.php on line 75","file":"/vendor/szymach/c-pchart/src/Chart/Draw.php","line":1186}

URL: index.php?module=API&method=ImageGraph.get&idSite=1&period=day&date=last10&apiModule=UserCountry&apiAction=getCountry&outputType=0&showLegend=1&fontSize=9&aliasedGraph=1&textColor=pfpszr9gse&backgroundColor=FFFFFF&gridColor=CCCCCC&legendAppendMetric=1&format=rss&token_auth=XYZANONYMIZED&translateColumnNames=1

FYI We received hundreds like this, so there may be othe payload that trigger it
@mattab mattab added this to the 3.6.1 milestone Sep 29, 2018
@mattab mattab added the Bug For errors / faults / flaws / inconsistencies etc. label Sep 29, 2018
@mattab
Copy link
Member Author

mattab commented Sep 29, 2018

Let's also fix another similar one too:

Argument 5 passed to CpChart\Chart\Draw::drawFilledRectangle() must be of the type array, null given, called in /plugins/ImageGraph/StaticGraph.php on line 358","file":"/vendor/szymach/c-pchart/src/Chart/Draw.php","line":444}

URL:index.php?module=API&method=ImageGraph.get&idSite=1&period=day&date=last10&apiModule=UserCountry&apiAction=getCountry&outputType=0&showLegend=1&fontSize=9&aliasedGraph=1&textColor=222222&backgroundColor=X&gridColor=CCCCCC&legendAppendMetric=1&format=rss&token_auth=XYZANONYMIZED&translateColumnNames=1

@sgiehl sgiehl self-assigned this Sep 30, 2018
@sgiehl sgiehl mentioned this issue Sep 30, 2018
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sgiehl sgiehl

Labels
Bug For errors / faults / flaws / inconsistencies etc.
Projects
None yet
Milestone
3.6.1
Development

Successfully merging a pull request may close this issue.

Use empty array as default value for cpchart colors matomo-org/matomo
2 participants
@mattab @sgiehl