Feeling Lucky
?
?
[#32970] - [4] add ext-ldap requirement to composer
- Closed
- 2 Apr 2021
- Medium
- Build: 4.0-dev
- # 32970
- Diff
- PhilETaylor:patch-97
User tests: Successful: Unsuccessful:
Summary of Changes
As composer.json is going to install joomla/ldap and symfonty/ldap it is a requirement of the server to also have the LDAP PHP extension installed and this should be defined as a requirement in the composer.json
Also phpStorm moans if its not defined in the composer.json when viewing the LDAP Packages.
Testing Instructions
Code review.
Actual result BEFORE applying this Pull Request
phpStorm moans:
Composer doesn't moan enough about the missing requirements (like it correctly moans about GD)
Expected result AFTER applying this Pull Request
No Moaning.
Documentation Changes Required
None
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | External Library Composer Change |
| Labels |
Added:
?
?
|
||
PhilETaylor
- comment
- 2 Apr 2021
The docs are wrong.
This is literally the point of constraints in composer.json
PhilETaylor
- comment
- 2 Apr 2021
brianteeman
- comment
- 2 Apr 2021
agree to disagree then
| Status | Pending | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2021-04-02 22:28:12 |
| Closed_By | ⇒ | PhilETaylor |
This is not correct. See https://docs.joomla.org/J4.x:Setting_Up_Your_Local_Environment#Possible_Issues