Skip to content

Commit

Permalink
Composer: Add pimple/pimple as dependency ILIAS-eLearning#6733
Browse files Browse the repository at this point in the history
This PR adds `pimple/pimple` as composer dependency.

Usage:
* Provides the dependency injection container that is used as a service locator
  throughout the system.
* Will most likely be used as the DIC solution for the new initialisation scheme
  introduced with the [component revision](docs/development/components-and-directories.md).

Wrapped By:
* ILIAS\DI\Container

Reasoning:
* Pimple provides a light weight and simple dependency injection container, which
  is yet quite powerful. The interface is modelled along \ArrayAccess, hence it
  can be easily mocked for testing.
* Dependency injection is ILIAS' future way to express dependencies between
  services and components.

Maintenance:
* The library has been stable for a long time and didn't receive updates since 2021.
* The actual code is only about 150 LOC and hence could be easily maintained by us,
  if necessary.

Links:
* Packagist: https://packagist.org/packages/pimple/pimple
* GitHub: https://github.com/silexphp/Pimple
  • Loading branch information
klees committed Dec 7, 2023
1 parent 666df52 commit 1f5216d
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions composer_new.json
Original file line number Diff line number Diff line change
Expand Up @@ -44,6 +44,7 @@
"ext-xml": "*",
"ext-zip": "*",
"ext-imagick": "*",
"pimple/pimple" : "^3.0",
},
"require-dev": {
},
Expand Down

0 comments on commit 1f5216d

Please sign in to comment.