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

Make sure to ignore free plugins when checking for license #10940

Merged
merged 1 commit into from Dec 2, 2016

Conversation

tsteur
Copy link
Member

@tsteur tsteur commented Dec 1, 2016

Some users see a license warning for a free GPL plugin see https://forum.piwik.org/t/license-missing-of-a-free-glp-plugin/21674/6. I have spent hours trying to understand how this could possibly happen over the last weeks but not sure how it happens. It must be some kind of caching issue with a certain PHP version or something. This change might not fix the actual issue but is a good check to have on top to check for invalid licenses only if it is not a free plugin.

This is related to https://forum.piwik.org/t/license-missing-of-a-free-glp-plugin/21674 

Some users see a license warning for a free GPL plugin. I have spent hours trying to understand how this could possibly happen over the last weeks but not sure how it happens. It must be some kind of caching issue with a certain PHP version or something. This change might not fix the actual issue but is a good check to have on top to check for invalid licenses only if it is not a free plugin.
@tsteur tsteur added the Needs Review PRs that need a code review label Dec 1, 2016
@tsteur tsteur added this to the 3.0.0-b4 milestone Dec 1, 2016
@mattab mattab merged commit 32ed53c into 3.x-dev Dec 2, 2016
@mattab mattab deleted the marketplace_license3x branch December 2, 2016 04:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs Review PRs that need a code review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants