Skip to content

Symfony Dotenv Component Fixes#1

Merged
jonesiscoding merged 9 commits intodeviscoding:1.8from
econoprint:dotenv-fixes
Nov 14, 2023
Merged

Symfony Dotenv Component Fixes#1
jonesiscoding merged 9 commits intodeviscoding:1.8from
econoprint:dotenv-fixes

Conversation

@aaronj-ep
Copy link
Copy Markdown

Allows the bundle to be used with environment variables in the specified configuration. Previously, runtime variables were not properly resolved in the propel.configuration and propel.build_properties services.

Additionally, normalization performed on configurations using environment variables failed, as at the time of normalization, the data was a placeholder, rather than the real configuration data.

With this pull request, the following was changed to allow for proper behavior:

  • The dbal & build_properties sections of the configuration are saved as parameters.
  • The propel.configuration, propel.build_properties and dependent services are now synthetic services.
  • The synthetic services are injected into the container at runtime.

Additionally, some other minor deprecations were handled.

@jonesiscoding jonesiscoding merged commit f694eda into deviscoding:1.8 Nov 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants