solve version check issue

This commit is contained in:
jayvirsinh_gohil
2021-06-25 15:43:30 +05:30
parent 7721bf6f93
commit d53e227b08
2 changed files with 11 additions and 8 deletions

View File

@ -6,6 +6,7 @@ use Crater\Http\Requests\DatabaseEnvironmentRequest;
use Crater\Http\Requests\DiskEnvironmentRequest; use Crater\Http\Requests\DiskEnvironmentRequest;
use Crater\Http\Requests\DomainEnvironmentRequest; use Crater\Http\Requests\DomainEnvironmentRequest;
use Crater\Http\Requests\MailEnvironmentRequest; use Crater\Http\Requests\MailEnvironmentRequest;
use Crater\Http\Requests\PaymentSettingsRequest;
use Exception; use Exception;
use Illuminate\Http\Request; use Illuminate\Http\Request;
use Illuminate\Support\Facades\DB; use Illuminate\Support\Facades\DB;
@ -76,9 +77,9 @@ class EnvironmentManager
} }
try { try {
$this->checkDatabaseConnection($request); $conn = $this->checkDatabaseConnection($request);
$requirement = $this->checkVersionRequirements($request); $requirement = $this->checkVersionRequirements($request, $conn);
if ($requirement) { if ($requirement) {
return [ return [
@ -176,7 +177,7 @@ class EnvironmentManager
* @param DatabaseEnvironmentRequest $request * @param DatabaseEnvironmentRequest $request
* @return bool * @return bool
*/ */
private function checkVersionRequirements(DatabaseEnvironmentRequest $request) private function checkVersionRequirements(DatabaseEnvironmentRequest $request, $conn)
{ {
$connection = $request->database_connection; $connection = $request->database_connection;
@ -195,6 +196,7 @@ class EnvironmentManager
switch ($connection) { switch ($connection) {
case 'mysql': case 'mysql':
$dbSupportInfo = $checker->checkMysqlVersion( $dbSupportInfo = $checker->checkMysqlVersion(
$conn,
config('crater.min_mysql_version') config('crater.min_mysql_version')
); );

View File

@ -2,6 +2,7 @@
namespace Crater\Space; namespace Crater\Space;
use PDO;
use SQLite3; use SQLite3;
class RequirementsChecker class RequirementsChecker
@ -121,10 +122,10 @@ class RequirementsChecker
* *
* @return array * @return array
*/ */
public function checkMysqlVersion(string $minMysqlVersion = null) public function checkMysqlVersion($conn, string $minMysqlVersion = null)
{ {
$minVersionMysql = $minMysqlVersion; $minVersionMysql = $minMysqlVersion;
$currentMysqlVersion = $this->getMysqlVersionInfo(); $currentMysqlVersion = $this->getMysqlVersionInfo($conn);
$supported = false; $supported = false;
if (version_compare($currentMysqlVersion, $minVersionMysql) >= 0) { if (version_compare($currentMysqlVersion, $minVersionMysql) >= 0) {
@ -145,11 +146,11 @@ class RequirementsChecker
* *
* @return string * @return string
*/ */
private static function getMysqlVersionInfo() private static function getMysqlVersionInfo($pdo)
{ {
$currentVersion = explode(' ', mysqli_get_client_info()); $currentVersion = $pdo->getAttribute(PDO::ATTR_SERVER_VERSION);
return $currentVersion[1]; return $currentVersion;
} }
/** /**