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

Installation failed on step 7 #1326

Closed
anonymous-matomo-user opened this issue May 5, 2010 · 7 comments
Closed

Installation failed on step 7 #1326

anonymous-matomo-user opened this issue May 5, 2010 · 7 comments
Labels
Bug For errors / faults / flaws / inconsistencies etc. Critical Indicates the severity of an issue is very critical and the issue has a very high priority.
Milestone

Comments

@anonymous-matomo-user
Copy link

On 7 step I've got following error message "- You can't access this resource as it requires an 'admin' access for at least one website."

Even if I try to use "example.org" or my own domain.
Tried to use svn version and lates.zip without result, same error.

@robocoder
Copy link
Contributor

This appears to be the problem behind the failing webtest.

@mattab
Copy link
Member

mattab commented May 5, 2010

I broke that.. let me fix it

@mattab
Copy link
Member

mattab commented May 5, 2010

(In [2138]) Fixes #1326
checkUserHasSomeAdminAccess() fails when there is no website yet. Making it work if user is super user.

@mattab
Copy link
Member

mattab commented May 5, 2010

(In [2140]) 0.6 with fix #1326

@mattab
Copy link
Member

mattab commented May 5, 2010

This was introduced in [2124]

I republished 0.6 with the fix in it, hopefully not many users did have time to download 0.6

@robocoder
Copy link
Contributor

Thanks Matt.

For consistency, can we similarly patch checkUserHasAdminAccess()?

@mattab
Copy link
Member

mattab commented May 5, 2010

checkUserHasAdminAccess is slightly different as it is used to actually check also that the idsites provided are correct and exist (eg. in SitesManager.updateSite).

If we change checkUserHasAdminAccess we would need to check that all methods would not fail or misbehave when give one or a list of fake website IDs. but agreed, this would be consistent and good todo :)

@anonymous-matomo-user anonymous-matomo-user added this to the Piwik 0.6 milestone Jul 8, 2014
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. Critical Indicates the severity of an issue is very critical and the issue has a very high priority.
Projects
None yet
Development

No branches or pull requests

3 participants