Add all accepted ILIAS 12 composer dependency PRs into composer.json#11255
Add all accepted ILIAS 12 composer dependency PRs into composer.json#11255lscharmer wants to merge 1 commit into
Conversation
thibsy
left a comment
There was a problem hiding this comment.
Hi @lscharmer,
Thx a lot for providing the integration of the dependency PR's and for your detailed summary. Much appreciated.
-
jsonlint: thx for bringing this to my attention. I think this package was used to added user-friendliness for system administrators who write config files for the setup and potentially make mistakes =). I will check with @chfsx if we want to keep this or not. -
phpunit: it's correct that we use this for the component bootstrap mechanism (at the moment). I will check in with @chfsx as well to discuss if we manage to replace this usage or have to request it for 12 as a prod dependency (again) ourselves.
Kind regards,
@thibsy
5687350 to
a5da21b
Compare
a5da21b to
c57b80a
Compare
|
PHP CS Fixer (PR #11408) is now integrated. For transparency: The version for |
af2ebff to
bd02bf2
Compare
|
PHPUnit (PR #11423) is now integrated. I fixed all Unit Tests which broke because of the version change from Because of a security advisory, I also updated the following packages:
And because of a version conflicts I updated:
|
|
quick notice @lscharmer : monolog was just confirmed: #11547 |
716b550 to
f7ab2b1
Compare
f7ab2b1 to
079a665
Compare
Hi @rfalkenstein, Monolog & Bacon QR Code are now included as well :) |
|
Hi @ILIAS-eLearning/technical-board, |
Hi,
this PR merges all accepted ILIAS 12 composer dependencies.
The following PRs are included:
ifsnop/mysqldump-phpas composer Dependency for ILIAS 12 #10653league/commonmarkas composer Dependency for ILIAS 12 #10654league/flysystemas composer Dependency for ILIAS 12 #10655sabre/davas composer Dependency for ILIAS 12 #10657symfony/consoleas composer Dependency for ILIAS 12 #10658guzzlehttp/psr7as composer Dependency for ILIAS 12 #10652dflydev/fig-cookiesas composer Dependency for ILIAS 12 #10651psr/http-messageas composer Dependency for ILIAS 12 #10650pimple/pimpleas composer Dependency for ILIAS 12 #10649ramsey/uuidfor ILIAS 12 #10577jumbojett/openid-connect-phpfor ILIAS 12 #10576simplesamlphp/simplesamlphpfor ILIAS 12 #10575phpmailer/phpmailerfor ILIAS 12 #10574mustache/mustachefor ILIAS 12 #10572ezyang/htmlpurifierfor ILIAS 12 #10571phpstan/phpstanfor ILIAS 12 #10581cmikey179/vfsstreamand for ILIAS 12 #10580captainhook/captainhookandcaptainhook/plugin-composerfor ILIAS 12 #10579captainhook/captainhookandcaptainhook/plugin-composerfor ILIAS 12 #10579flip/whoopsfor ILIAS 12 #11265Missing dependency for ILIAS to work
None. All are included now.
Other changes
JSONLint
As the library JSONLint has not been suggested / accepted, I replaced the library with calls to
json_decode.The library was used to provide better error messages than
json_decodeif theconfig.jsonis not valid JSON.@thibsy is this change fine for you?
PHPMailer
The PR #10574 suggests version
7.0but this is incompatible withsimplesaml's required version of thephpmailerso in agreement with @mjansen I changed this to version6.8.LTI Patch
As the file
./vendor/composer/patches/lti.patchdoesn't exist anymore, I removed the patch from thecomposer.json.Unit Tests
All Unit Tests which broke because of the version change of PHPUnit from
v11tov13have been fixed.Summary
In comparison to ILIAS 11:
New dependency:
Removed dependency: