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

Show Site Search reports in Scheduled email reports and Piwik Mobile #4869

Closed
tsteur opened this issue Mar 17, 2014 · 2 comments
Closed

Show Site Search reports in Scheduled email reports and Piwik Mobile #4869

tsteur opened this issue Mar 17, 2014 · 2 comments
Assignees
@tsteur
Labels
Critical Indicates the severity of an issue is very critical and the issue has a very high priority. Enhancement For new feature suggestions that enhance Matomo's capabilities or add a new report, new API etc.
Milestone
2.2.1

Comments

@tsteur
Copy link
Member

tsteur commented Mar 17, 2014

A user reported the Site Search report is not available in the mobile app and I noticed the report is not included in API.getReportMetadata.

The reason for this is a check for the idSite parameter: https://github.com/piwik/piwik/blob/master/plugins/Actions/Actions.php#L503

This does not work:
http://demo.piwik.org/index.php?module=API&method=API.getReportMetadata&idSites=3&period=day&date=yesterday&format=JSON&token_auth=anonymous

This does work:
http://demo.piwik.org/index.php?module=API&method=API.getReportMetadata&idSites=3&period=day&date=yesterday&format=JSON&token_auth=anonymous&idSite=3

It should work without the idSite parameter as well.
@tsteur
Copy link
Member Author

tsteur commented Mar 17, 2014

In 2da1fc7: fixes #4869 site search was not included in report metadata

@tsteur
Copy link
Member Author

tsteur commented Mar 17, 2014

In f3a53e2: refs #4869 fix tests

@tsteur tsteur added this to the 2.2.1 - Piwik 2.2.1 milestone Jul 8, 2014
@tsteur tsteur self-assigned this Jul 8, 2014
@tsteur tsteur modified the milestones: 2.4.0 - Piwik 2.4.0, 2.2.1 - Piwik 2.2.1 Jul 11, 2014
sabl0r pushed a commit to sabl0r/piwik that referenced this issue Sep 23, 2014
@tsteur
fixes matomo-org#4869 site search was not included in report metadata
2da1fc7
sabl0r pushed a commit to sabl0r/piwik that referenced this issue Sep 23, 2014
@tsteur
refs matomo-org#4869 fix tests
f3a53e2
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tsteur tsteur

Labels
Critical Indicates the severity of an issue is very critical and the issue has a very high priority. Enhancement For new feature suggestions that enhance Matomo's capabilities or add a new report, new API etc.
Projects
None yet
Milestone
2.2.1
Development

No branches or pull requests
1 participant
@tsteur