Skip to content

Conversation

fdellwing
Copy link
Contributor

fixes #13039

@fdellwing
Copy link
Contributor Author

I do not understand why this reusing broke stuff, but this fixxes it.

@sgiehl
Copy link
Member

sgiehl commented Jun 6, 2018

Good find. Guess that happens as it is used as reference before: https://github.com/matomo-org/matomo/pull/13041/files#diff-efad25243f722b5c59f5a00f11cf78d9R28

@sgiehl sgiehl added the Needs Review PRs that need a code review label Jun 7, 2018
@sgiehl sgiehl added this to the 3.6.0 milestone Jun 7, 2018
@diosmosis diosmosis merged commit 061e936 into matomo-org:3.x-dev Jun 10, 2018
@fdellwing fdellwing deleted the fixxes-logo-staticgraph branch June 13, 2018 09:23
@mattab mattab changed the title Fixxes a problem with preused variable name in graph generating via API Fixes last icon in ImageGraph generated static images shows a wrong icon Aug 28, 2018
@mattab mattab added the Bug For errors / faults / flaws / inconsistencies etc. label Aug 28, 2018
InfinityVoid pushed a commit to InfinityVoid/matomo that referenced this pull request Oct 11, 2018
…PI (matomo-org#13041)

* Fixxes a problem with preused variable name in graph generating via API

* Update expected system test files.

* Update expected files again.
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. Needs Review PRs that need a code review
Development

Successfully merging this pull request may close these issues.

Last Country in ImageGraph shows a wrong flag
4 participants