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

in General Settings dashboard, getting "Invalid PHP version" #10560

Closed
mattab opened this issue Sep 26, 2016 · 4 comments · Fixed by #10573
Closed

in General Settings dashboard, getting "Invalid PHP version" #10560

mattab opened this issue Sep 26, 2016 · 4 comments · Fixed by #10573
Assignees
@tsteur
Labels
not-in-changelog For issues or pull requests that should not be included in our release changelog on matomo.org. Regression Indicates a feature used to work in a certain way but it no longer does even though it should.
Milestone
3.0.0-b1

Comments

@mattab
Copy link
Member

mattab commented Sep 26, 2016

Reproduce URL is: index.php?module=CoreAdminHome&action=home&showaddsite=1&period=day&date=yesterday&idSite=1

Marketplace request URL is http://plugins.piwik.org/api/2.0/plugins?keywords=&php=7.0.9-1%2Bdeb.sury.org%7Etrusty%2B1&piwik=3.0.0-b1&prefer_stable=1&query=&sort=newest

My PHP version is PHP 7.0.9-1+deb.sury.org~trusty+1

Response from marketplace was: Invalid PHP Version

Screenshot:

invalid php version
@mattab mattab added not-in-changelog For issues or pull requests that should not be included in our release changelog on matomo.org. Regression Indicates a feature used to work in a certain way but it no longer does even though it should. labels Sep 26, 2016
@mattab mattab added this to the 3.0.0-b1 milestone Sep 26, 2016
@mattab mattab changed the title in General Settings dashboard, getting "Invalid PHP version" when php version is in General Settings dashboard, getting "Invalid PHP version" Sep 26, 2016
@tsteur
Copy link
Member

tsteur commented Sep 26, 2016

what is the output of PHP_VERSION vs phpversion()

@mattab
Copy link
Member Author

mattab commented Sep 26, 2016

same output:

$ php -r 'echo PHP_VERSION . "\n" . phpversion() . "\n"; '
7.0.9-1+deb.sury.org~trusty+1
7.0.9-1+deb.sury.org~trusty+1

@tsteur tsteur mentioned this issue Sep 27, 2016
Merged
@tsteur
Copy link
Member

tsteur commented Sep 27, 2016

issued PR in #10573

Can you test output of php -r "echo PHP_MAJOR_VERSION . '.' . PHP_MINOR_VERSION . '.' . PHP_RELEASE_VERSION;"

if it works I will fix it for Piwik 2 as well

@tsteur tsteur self-assigned this Sep 27, 2016
@mattab
Copy link
Member Author

mattab commented Sep 27, 2016 

$ php -r "echo PHP_MAJOR_VERSION . '.' . PHP_MINOR_VERSION . '.' . PHP_RELEASE_VERSION;"
7.0.9

@mattab mattab closed this as completed Oct 3, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tsteur tsteur

Labels
not-in-changelog For issues or pull requests that should not be included in our release changelog on matomo.org. Regression Indicates a feature used to work in a certain way but it no longer does even though it should.
Projects
None yet
Milestone
3.0.0-b1
Development

Successfully merging a pull request may close this issue.

fix in General Settings dashboard, getting "Invalid PHP version" matomo-org/matomo
2 participants
@tsteur @mattab