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

Dashboard Suggestions #4007

Closed
timo-bes opened this issue Jun 12, 2013 · 7 comments
Closed

Dashboard Suggestions #4007

timo-bes opened this issue Jun 12, 2013 · 7 comments
Assignees
Labels
Bug For errors / faults / flaws / inconsistencies etc.

Comments

@timo-bes
Copy link
Member

The widget "Visits Overview (with graph)" does not save the selected metrics. "Visits Over Time" on the other hand does. It would be nice if both would save the selected metrics.

Also, is it possible that dashboards can't be deleted? Or is it just me who can't figure out how?

@timo-bes
Copy link
Member Author

I investigated the issue further: When you add a metric in the widget "Visits Overview (with graph)", it stores the column in the table piwik_user_dashboard but it looks like it doesn't read the value when loading the widget.
When avg_time_generation is picked, this is also stored in the column-array in piwik_user_dashboard but when you reload the dashboard it shows the error "The report 'avg_time_generation' was requested but it is not available at this stage. You may also disable the related plugin (avg) to avoid this error.". The only way to fix this is to edit the JSON in piwik_user_dashboard, which has brought me to understand the issue the way I describe it now.

Stefan, I set you as the owner of the ticket. Could you please have a look?

@sgiehl
Copy link
Member

sgiehl commented Jun 15, 2013

In 16fb8c9: refs #4007 sparklines can't handle columns parameter

@sgiehl
Copy link
Member

sgiehl commented Jun 15, 2013

That should solve this issue. Small explenation:
The saved columns are attached to the complete request for the visitsummary index action (when changing the selected columns only the evolution graph is refreshed).
The method throwing the error is: https://github.com/piwik/piwik/blob/master/plugins/VisitsSummary/Controller.php#L89
The called api method can't handle the request for avg_time_generation

@timo-bes
Copy link
Member Author

In 84fb1bf: fixes #4007

  • don't override columns parameter when it is set from the dashboard widget "Visits Overview (with graph)"
  • remove columns parameter when requesting VisitsSummary.get

@timo-bes
Copy link
Member Author

In 9f7b164: refs #4007 in branch 1.X, an alternative fix for this issue has been committed. revert this one in order avoid merge confusion.

@timo-bes
Copy link
Member Author

In 3174917: refs #4007 this should have been in 84fb1bf

  • don't override columns parameter when it is set from the dashboard widget "Visits Overview (with graph)"
  • remove columns parameter when requesting VisitsSummary.get

what i did commit in 84fb1bf was the first step towards making piwik work again in ie8. sorry for the mix up.

@timo-bes
Copy link
Member Author

Thanks for investigating, Stefan. I pushed the fix down to the spot where you identified the issue. I hope that's OK.
Also, I fixed the other issue that selected columns are not restored.
See 3174917

I reverted the fix in master and pushed everything to 1.X - that's what Matt said would be the best way to do it if we want to release quickly (he said that in general, not related to this issue). We will later merge 1.X into master in order to make the fix available there.

sabl0r pushed a commit to sabl0r/piwik that referenced this issue Sep 23, 2014
sabl0r pushed a commit to sabl0r/piwik that referenced this issue Sep 23, 2014
… has been committed. revert this one in order avoid merge confusion.
This issue was closed.
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

No branches or pull requests

2 participants