- π©πͺGermany gngn
I know "Minimally maintained" ... but there are still a lot of installs and I would like to have the status message...
#3 modifies
swiftmailer_requirements($phase)
toif ($phase == 'install') { ... } elseif ($phase === 'runtime') { ... + else { + try { + ... + } + }
(at least with current 8.x-2.x).
According to API: hook_requirements $phase can have the following values:
- Checking installation requirements ($phase == 'install').
- Checking update requirements ($phase == 'update').
- Status reporting ($phase == 'runtime').
The above codes will only run in phase 'update'.I think we need the 'runtime' phase, so we should get rid of the 'else' - or what I am not getting?
P.S. Changing to current 8.x-2.x
- πΊπΈUnited States dave reid Nebraska USA
You can see the patch was originally provided for the 8.x-1.x version which does not currently have an if check for $phase == 'runtime' so It needed to add it. You are correct that it needed to be adjusted for the 8.x-2.x version which does have that already.