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

Calling MultiSites.getAll results in "Invalid parameter number" (in Piwik 2.5.0-b1) #5887

Closed

tsteur opened this issue Jul 27, 2014 · 0 comments

Assignees

Labels

Bug

Milestone

Member

tsteur commented Jul 27, 2014

Opening for instance http://demo2.piwik.org/index.php?enhanced=1&period=day&date=yesterday&pattern=test&filter_limit=50&sortOrderColumn=nb_visits&filter_sort_column=nb_visits&module=API&token_auth=XXXXXX&format=json&method=MultiSites.getAll&language=en&

Results in

SQLSTATE[HY093]: Invalid parameter number: number of bound variables does not match number of tokens

Can reproduce it on my local instance if pattern contains a string that matches at least one site. Breaks Piwik Mobile 2

The text was updated successfully, but these errors were encountered:

tsteur added this to the Piwik 2.5.0 milestone

tsteur added bug labels

tsteur added a commit that referenced this issue


          refs #5887 should fix Invalid parameter number in case pattern parame…

a49eb87

…ter is used to filter all websites

tsteur added a commit that referenced this issue


          refs #5887 make sure the user has view access to all sites... we shou…

038771c

…ld really introduce a model for sitesManager to not having directly to expose methods

tsteur added a commit that referenced this issue


          refs #5887 use a model so we do not have to expose the method in the …

652eca3

…API. I tried to write a test for this but was not able to since lots of IntegrationTest magic

tsteur closed this as completed

tsteur added a commit that referenced this issue


          refs #5887 should fix Invalid parameter number in case pattern parame…

399bea3

…ter is used to filter all websites

tsteur added a commit that referenced this issue


          refs #5887 make sure the user has view access to all sites... we shou…

299875d

…ld really introduce a model for sitesManager to not having directly to expose methods

tsteur added a commit that referenced this issue


          refs #5887 use a model so we do not have to expose the method in the …

9b04111

…API. I tried to write a test for this but was not able to since lots of IntegrationTest magic

tsteur self-assigned this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment