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

Readiness Check does not check for MCrypt Extension #713

Closed
Viveur opened this issue Oct 28, 2014 · 5 comments
Closed

Readiness Check does not check for MCrypt Extension #713

Viveur opened this issue Oct 28, 2014 · 5 comments

Comments

@Viveur
Copy link

Viveur commented Oct 28, 2014

Current Behaviour

During Stage 1 of install I see:

PHP Extensions Check You meet 2 out of 2 PHP extensions requirements. [Less detail] PHP Extension ctype. PHP Extension gd.

However at Stage 6 install fails with:

[ERROR] exception 'Exception' with message 'exception 'Exception' with message 'Module 'Magento_Core' depends on 'mcrypt' PHP extension that is not loaded.' in /Users/[name]/Documents/WWW/magento2/lib/internal/Magento/Framework/Module/DependencyManager.php:56

Suggested Behaviour

Readiness Check should test for whether the PHP Extension MCrypt is loaded as this is a required extension.

@maksek
Copy link
Contributor

maksek commented Oct 28, 2014

Hi @bit32, seems like you have 2 php instances on your server, one of them has mcrypt (the one apache uses), and another one doesn't have it.
I will close the issue, due duplication with #714

@maksek maksek closed this as completed Oct 28, 2014
@Viveur
Copy link
Author

Viveur commented Oct 28, 2014

I actually opened this a separate issue as it's not dependent on this issue described in #714 and is generally applicable to any PHP installation.

I know that the Magento 1 installer definitely checks for MCrypt as part of it's install process

I think people may find it confusing to have a situation where they passed a test for PHP Extensions one minute and then encounter an error (that occurs due to a missing PHP Extension) the next.

@maksek
Copy link
Contributor

maksek commented Nov 3, 2014

@bit32, actually root cause of the issue, is the same as in #714, thats why i closed this one with reference.

@Viveur
Copy link
Author

Viveur commented Nov 3, 2014

Massively confused. Surely if anything this is the root cause of #714 and not the other way around?!

@maksek
Copy link
Contributor

maksek commented Nov 3, 2014

@bit32 if you follow installation process, all steps except 6th, are handled by PHP+Apache. Basically Magento setup wizard is using php which configured with apache.
On last step - there is exec code for console run for reindex, which will use your default php installation, returned by this code \Symfony\Component\Process\PhpExecutableFinder::find(). Which in your case is separate instance of php, and doesn't have all extensions installed.
Let me know if resolve your confusion. :)

okorshenko pushed a commit that referenced this issue Oct 12, 2015
magento-engcom-team pushed a commit that referenced this issue Nov 4, 2019
Added test case Id and indentation fix
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants