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

Argument 1 passed to Piwik\API\DataTableManipulator\ReportTotalsCalculator::hasDataTableMetric() must be an instance of Piwik\DataTable, instance of Piwik\DataTable\Map given #5373

Closed
mattab opened this issue Jun 23, 2014 · 2 comments
Closed

Argument 1 passed to Piwik\API\DataTableManipulator\ReportTotalsCalculator::hasDataTableMetric() must be an instance of Piwik\DataTable, instance of Piwik\DataTable\Map given #5373

mattab opened this issue Jun 23, 2014 · 2 comments
Assignees
@tsteur
Labels
Bug For errors / faults / flaws / inconsistencies etc.
Milestone
Backlog (Help wan...

Comments

@mattab
Copy link
Member

mattab commented Jun 23, 2014

Reported in forums

Reproduce:

 curl --globoff "http://localhost/piwik-master/?module=API&method=API.getBulkRequest&format=json&token_auth=9b1cefc915ff6180071fb7dcd13ec5a4&urls[0]=method%3DCustomVariables.getCustomVariablesValuesFromNameId%26period%3Dday%26idSubtable%3D1%26idSite%3Dall%26date%3D2014-06-01%2Ctoday"
@mattab mattab added this to the 2.5.0 - Piwik 2.5.0 milestone Jul 8, 2014
tsteur added a commit that referenced this issue Jul 19, 2014
@tsteur
refs #5373 this should really make sure we work on a dataTable and no…
9930752 
…t on a datatable map
tsteur added a commit that referenced this issue Jul 19, 2014
@tsteur
refs #5373 this should really make sure we work on a dataTable and no…
86bce17 
…t on a datatable map. @mattab can you confirm?
@tsteur tsteur self-assigned this Jul 19, 2014
@tsteur
Copy link
Member

tsteur commented Aug 6, 2014

@mattab can you confirm it is fixed?

@mattab
Copy link
Member Author

mattab commented Sep 11, 2014

It now works

@mattab mattab closed this as completed Sep 11, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tsteur tsteur

Labels
Bug For errors / faults / flaws / inconsistencies etc.
Projects
None yet
Milestone
Backlog (Help wanted)
Development

No branches or pull requests
2 participants
@tsteur @mattab