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: specify a value for 'reportUniqueId' when exporting Insights reports #6790

Closed
J-Mishima opened this issue Dec 3, 2014 · 4 comments
Closed

Error: specify a value for 'reportUniqueId' when exporting Insights reports #6790

J-Mishima opened this issue Dec 3, 2014 · 4 comments
Assignees
@tsteur
Labels
Bug For errors / faults / flaws / inconsistencies etc.
Milestone
Piwik 2.11.0

Comments

@J-Mishima
Copy link

Hi.

An error written in the title in the following procedures occurs.

1.Display 'Action' -> 'Pages'.
2.Click 'Insight' icon in the bottom of the report.
3.Export as CSV(or other format).
4.Show message "Error: Please specify a value for 'reportUniqueId'." in new window

This issue is reporduced any report that have 'Insight' mode.

I use Windows 7 Pro and Chrome 39.0.2171.71 m.

st0197

st0198

st0199

st0200
@mattab mattab added the Bug For errors / faults / flaws / inconsistencies etc. label Dec 3, 2014
@mattab mattab added this to the Short term milestone Dec 3, 2014
@mattab mattab changed the title Error: Please specify a value for 'reportUniqueId'. Error: specify a value for 'reportUniqueId' when exporting Insights reports Dec 3, 2014
@mattab
Copy link
Member

mattab commented Dec 3, 2014

Hi @J-Mishima thanks for the report.

@tsteur
Copy link
Member

tsteur commented Dec 23, 2014

Would be a solution to disable export for Insights?

Alternatively we need to enrich the export feature (in plugins/CoreHome/templates/_dataTableFooter.twig) by adding further properties to ViewDataTableConfig as it currently only gets the apiMethodToRequestDataTable which is in this case Insights.getInsights meaning it does not read any additional properties like reportUniqueId. See https://github.com/piwik/piwik/blob/2.10.0-rc1/plugins/CoreHome/templates/_dataTableFooter.twig#L61

@mattab
Copy link
Member

mattab commented Dec 28, 2014

Would be a solution to disable export for Insights?

OK for me!

@tsteur
Copy link
Member

tsteur commented Jan 4, 2015

Just noticed it is not easily possible to disable all export links... at least not with introducing a new visualization config so might be worth to fix it properly.

@tsteur tsteur self-assigned this Feb 4, 2015
@tsteur tsteur modified the milestones: Piwik 2.11.0, Short term Feb 4, 2015
tsteur added a commit that referenced this issue Feb 4, 2015
@tsteur
refs #6790 when exporting a report we need to apply the same request …
a9c7845 
…params that were used to render the datatable.

This will not only fix this bug but should also fix potential other bugs
where request params were not forwarded. For example an abandonedCarts
param in Ecommerce, or hideFutureHoursWhenToday in VisitTime.
@tsteur tsteur mentioned this issue Feb 4, 2015
Merged
@tsteur tsteur closed this as completed Feb 9, 2015
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
Piwik 2.11.0
Development

No branches or pull requests
3 participants
@tsteur @mattab @J-Mishima