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

Multisite upgrade notice showing to child-blog admins? #259

Closed
raamdev opened this issue Aug 4, 2014 · 7 comments
Closed

Multisite upgrade notice showing to child-blog admins? #259

raamdev opened this issue Aug 4, 2014 · 7 comments

Comments

@raamdev
Copy link
Contributor

raamdev commented Aug 4, 2014

A user reported that the Quick Cache Pro upgrade notice was appearing even for child-blog admins, who don't have permission to upgrade Quick Cache Pro at the network level.

I haven't verified this bug yet, so this needs testing.


Reported here: https://websharks.zendesk.com/agent/#/tickets/3342

@raamdev raamdev added this to the Next Release milestone Aug 5, 2014
@jaswrks jaswrks self-assigned this Aug 5, 2014
@jaswrks
Copy link

jaswrks commented Aug 5, 2014

I can take a look at this. Assigning this to myself.

@jaswrks
Copy link

jaswrks commented Aug 5, 2014

Confirmed. This bug is caused by the persistent notice functionality built into Quick Cache. The notice itself is suppressed properly; i.e. the \quick_cache\plugin::$update_cap is required. However, once a persistent notice is generated, it displays in all views with only a permission check against the basic \quick_cache\plugin::$cap property. That's a bug.

@jaswrks
Copy link

jaswrks commented Aug 5, 2014

I should note that it's only the link being displayed that is the issue. I'm not seeing a real security threat here, since the functionality itself is still off limits as it should be in this scenario.

jaswrks pushed a commit to wpsharks/comet-cache-pro that referenced this issue Aug 5, 2014
jaswrks pushed a commit to wpsharks/comet-cache-pro that referenced this issue Aug 5, 2014
@jaswrks
Copy link

jaswrks commented Aug 5, 2014

Resolved in the feature/259 branch.

@raamdev
Copy link
Contributor Author

raamdev commented Aug 6, 2014

@jaswsinc Thanks! Can you submit a Pull Request for the feature/259 branch when you get a chance?

@raamdev
Copy link
Contributor Author

raamdev commented Aug 6, 2014

@jaswsinc Ha, never mind. I was looking for the PR on the Quick Cache repo. I see the PR on the Quick Cache Pro repo (and how I missed the reference to it before your last comment I'll never know).

@raamdev
Copy link
Contributor Author

raamdev commented Aug 6, 2014

@jaswsinc Thanks for taking care of this!

Closed by Pull Request wpsharks/comet-cache-pro#67

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

No branches or pull requests

2 participants