Stop execution in case the script is used for an upgrade #11
Conversation
Also includes new dependencies Fixes #9
LukasReschke
force-pushed
the
addUpdateWarning
branch
from
84e5c09
to
c57ba86
Compare
|
👍 Also thanks for the other improvements |
Some code that I also used for owncloud/administration#11
Some code that I also used for owncloud/administration#11
|
@karlitschek Can we merge this and put live? We get again a lot of those bug reports, just now: owncloud/core#14317 |
|
|
||
| // do we have PHP 5.3.3 or newer? | ||
| if(version_compare(PHP_VERSION, '5.3.3', '<')) { | ||
| $error.='PHP 5.3.3 is required. Please ask your server administrator to update PHP to version 5.3.3 or higher.<br/>'; |
There was a problem hiding this comment.
We still ship 7.0 with the installer as far I know
There was a problem hiding this comment.
Hi
could you provide the instructions to compile owncloud client for Windows on sure 13?
Sent from Yahoo Mail on Android
From:"Lukas Reschke" notifications@github.com
Date:Wed, Feb 18, 2015 at 12:28 PM
Subject:Re: [administration] Stop execution in case the script is used for an upgrade (#11)
In web-installer/setup-owncloud.php:
- @return string with error messages > */ > static public function checkdependencies() { > - $error=''; > - > - // do we have PHP 5.3.2 or newer? > - if(version_compare(PHP_VERSION, '5.3.2', '<')) { > - $error.='PHP 5.3.2 is required. Please ask your server administrator to update PHP to version 5.3.2 or higher. PHP 5.2 is no longer supported by ownCloud and the PHP community.'; > + $error = ''; > + $missingDependencies = array();; > + > + // do we have PHP 5.3.3 or newer? > + if(version_compare(PHP_VERSION, '5.3.3', '<')) { > + $error.='PHP 5.3.3 is required. Please ask your server administrator to update PHP to version 5.3.3 or higher.
';
We still ship 7.0 with the installer as far I know
—
Reply to this email directly or view it on GitHub.
There was a problem hiding this comment.
could you provide the instructions to compile owncloud client for Windows on sure 13?
Not really… This discussion is about something completely different, see #11 (comment)
|
👍 |
Stop execution in case the script is used for an upgrade
|
@karlitschek @tomneedham how do we update this script on https://owncloud.org/install/#instructions-web ??? THX |
|
So somebody (most likely @karlitschek) has to replace |
|
hmm - what about pointing the link to the admin repo? |
I'd avoid that as about 280 people can commit directly here… Would be way too easy to lure them into a botnet that way :) |
|
i will replace it in our server later. thanks guys |
|
This is now life. Thank you guys. |
Also includes new dependencies
Fixes #9
@karlitschek @PVince81