-
-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
[4.0] check for libxml #31735
[4.0] check for libxml #31735
Conversation
PR for joomla#29425 This is a real edge case discovered by @Fedik joomla#29425 (comment) libxml is enabled by default when compiling php and it cannot be disabled etc in a php.ini The check I have put in place is the same one that myBB uses for the same issue. The only questions are 1. Did I put the check early enough 2. Do we really need it for such an edge case
@brianteeman although this would work I think that the check needs to be in the Initial checks (compatibility layer):
Also that would result to a better error view |
@brianteeman can you fix the cs plz |
maybe better to place it in with some nice message like: joomla-cms/installation/index.php Lines 20 to 26 in 77bcd71
The problem the App does not even start if upd: the text in |
Since all the checks from the Model could end up being foobar if any of these fail I guess beefing up these files is a more realistic approach. Yeah I know the code will end up being god awful but functional code is better than beautiful non functional code. My 2c |
Co-authored-by: Dimitris Grammatikogiannis <[email protected]>
Co-authored-by: Dimitris Grammatikogiannis <[email protected]>
PR for #29425
This is a real edge case discovered by @Fedik #29425 (comment)
libxml is enabled by default when compiling php and it cannot be disabled etc in a php.ini
The check I have put in place is the same one that myBB uses for the same issue.
The only questions are