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

[BUG] PrestaShop Install Script Error: "install" directory is missing #2467

Closed
petrus9 opened this issue Mar 9, 2022 · 15 comments · Fixed by #2503
Closed

[BUG] PrestaShop Install Script Error: "install" directory is missing #2467

petrus9 opened this issue Mar 9, 2022 · 15 comments · Fixed by #2503
Assignees
Labels
bug Something isn't working

Comments

@petrus9
Copy link

petrus9 commented Mar 9, 2022

Describe the bug

Also see: https://forum.hestiacp.com/t/prestashop-quick-install-app-error-install-directory-is-missing/5765?u=petrushka

setup domain with SSL

Install PrestaShop Quick install

see the following message: (Even though the directory is empty)

'Data loss warning!
Your web domain already has files uploaded, the installer will overwrite your files and/or the installation might fail.

Please use the installer only for empty web domains'

When I go to my to my URL I see the following:

Error: "install" directory is missing

Tell us how to replicate the bug

  1. Click on Quick install App on any domain panel
  2. click on setup for the PrestaShop App

Which components are affected by this bug?

Control Panel Web Interface

Hestia Control Panel Version

v1.5.10

Operating system

Ubuntu 20.04

Log capture

No response

@petrus9 petrus9 added the bug Something isn't working label Mar 9, 2022
@petrus9 petrus9 changed the title PrestaShop Install Script Error: "install" directory is missing [BUG] PrestaShop Install Script Error: "install" directory is missing Mar 9, 2022
@jaapmarcus jaapmarcus self-assigned this Mar 9, 2022
@jaapmarcus
Copy link
Member

Haven't been able to "Find" the issue:

We delete the install dir in code so the error doesn't surprise me...

$this->appcontext->runUser('v-run-cli-cmd', [
"/usr/bin/php",
$this->getDocRoot("/install/index_cli.php"),
"--db_user=" . $this->appcontext->user() . '_' .$options['database_user'],
"--db_password=" . $options['database_password'],
"--db_name=" . $this->appcontext->user() . '_' .$options['database_name'],
"--firstname=" . $options['prestashop_account_first_name'],
"--lastname=" . $options['prestashop_account_last_name'],
"--password=" . $options['prestashop_account_password'],
"--email=" . $options['prestashop_account_email'],
"--domain=" . $this->domain,
"--ssl=" . $ssl_enabled,], $status);
// remove install folder
$this->appcontext->runUser('v-delete-fs-directory', [$this->getDocRoot("/install")]);
$this->cleanup();
return ($status->code === 0);

How ever there should be a reason why the code before the deletion fails.

When an install fails the system should be able to return with an error instead of "Nothing" ...

@jaapmarcus
Copy link
Member

jaapmarcus commented Mar 9, 2022

What is the php version do you have? So far I can remember Prestashop has issues with certain php versions...

@jaapmarcus
Copy link
Member

@petrus9 Please run preferable on a test server:

v-update-sys-hestia-git jaapmarcus fix/quick-installers

And try again

@jaapmarcus
Copy link
Member

Your server is running PHP 8.0.16, but PrestaShop requires PHP 7.4 or lower. To install PrestaShop 1.7.8.4 you need to downgrade your server's PHP version.
....

@petrus9
Copy link
Author

petrus9 commented Mar 14, 2022

@petrus9 Please run preferable on a test server:

v-update-sys-hestia-git jaapmarcus fix/quick-installers

And try again

Is this a fix for all the installers or just for PrestaShop?

@jaapmarcus
Copy link
Member

  • Opencart
  • Drual
  • Docuwiki

Currently fixed

Prestashop has still some issues but at least I know how to solve them...

MediaWiki not yet tested ...

@petrus9
Copy link
Author

petrus9 commented Mar 14, 2022

for v-update-sys-hestia-git jaapmarcus fix/quick-installers

It looks like this updates HestiaCP to an Alpha version? Or does it just update the quick installer code? I don't have a test server at the moment, but may be able to spin one up.

@jaapmarcus
Copy link
Member

Yes it updates Hestia to the Alpha version + the changes for Quick installer

@petrus9
Copy link
Author

petrus9 commented Mar 17, 2022

@jaapmarcus I am just getting to testing this now, Is v-update-sys-hestia-git jaapmarcus fix/quick-installers still valid or should I go with the latest build?

@jaapmarcus
Copy link
Member

jaapmarcus fix/quick-installers contains the above listed patches.

It still need some rewrites to have some more options...

@petrus9
Copy link
Author

petrus9 commented Mar 17, 2022

@jaapmarcus Is there anything in particular you want me to test right now?

@jaapmarcus
Copy link
Member

All the installers should work fine...

Have a few more ideas to add some version detection when you select the app you want to install.. (For example 7.4 or lower missing for Prestashop)

@petrus9
Copy link
Author

petrus9 commented Mar 21, 2022

Yes, I noticed this kind of message. Thanks! Perhaps even have a message like that show up under the quick install logo of each app or pop up when you click on it.

@jaapmarcus
Copy link
Member

I think we should "grey" out non supported apps

@petrus9
Copy link
Author

petrus9 commented Mar 21, 2022 via email

@jaapmarcus jaapmarcus linked a pull request Mar 29, 2022 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants