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

pfad zur verwendeten php version anpassbar machen #124

Open
staabm opened this issue Sep 21, 2022 · 1 comment
Open

pfad zur verwendeten php version anpassbar machen #124

staabm opened this issue Sep 21, 2022 · 1 comment
Labels
enhancement New feature or request

Comments

@staabm
Copy link
Member

staabm commented Sep 21, 2022

Feature description / Feature Beschreibung
nicht immer ist das php was im $PATH liegt das "richtige".

überlegen wie man einstellbar machen kann von welchem pfad das php gezogen werden soll.
(insbesondere da ja bereits beim install des addons die php version geprüft wird)

@staabm staabm added the enhancement New feature or request label Oct 1, 2022
@staabm
Copy link
Member Author

staabm commented Oct 13, 2022

dafür müsste man vermutlich den check aus dem install-script entfernen

rexstan/install.php

Lines 11 to 26 in 9774c7a

$cliPhpVersion = RexStan::execCmd(RexStan::phpExecutable().' -r "echo PHP_VERSION_ID;"', $lastError);
if (is_numeric($cliPhpVersion)) {
if ($cliPhpVersion < 70300) {
if (DIRECTORY_SEPARATOR === '\\') {
$cliPhpPath = RexStan::execCmd('where php', $lastError);
} else {
$cliPhpPath = RexStan::execCmd('which php', $lastError);
}
$addon->setProperty('installmsg', 'PHP CLI version '.$cliPhpVersion.' on path "'. $cliPhpPath .'" is too old. Please upgrade to PHP 7.3+.');
return;
}
} else {
$addon->setProperty('installmsg', 'Unable to determine PHP CLI version. Make sure PHP is installed in the CLI and availabe within your system PATH.');
return;
}

stattdessen könnte man den check laufen lassen, wenn die einstellungen seite geöffnet wird.

für den php pfad müsste man dann ein eingabefeld einfügen und ggf. die anderen tabs alle ausblenden o.ä. solange noch kein php binary gewählt, bzw. keiner automatisch erkannt werden kann... müsste man sich im details mal durchprobieren

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant