Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow adapter as configuration key when using StoragePluginFactoryInterface #182

Merged

Conversation

boesing
Copy link
Member

@boesing boesing commented Nov 17, 2021

Q A
Bugfix no
BC Break no
New Feature yes

Description

In v2.12.0, we provided a normalized array structure along with a configuration check command. When a project is being migrated from latest v2 version to v3, it will still break due to an incompatible configuration. This was changed in v3 and was not backported to v2 before releasing v3.

With this patch, the name configuration key will be deprecated in favor of the adapter configuration key to ensure better backwards compatibility. adapter is also used in documentation as outlined in #179 and thus, this could also be treated as a bugfix.

@boesing boesing added Bug Something isn't working Enhancement labels Nov 17, 2021
@boesing boesing added this to the 3.1.0 milestone Nov 17, 2021
…inFactoryInterface`

In v2.12.0, we provided a normalized array structure along with a configuration check command. When a project is being migrated from latest v2 version to v3, it will still break due to an incompatible configuration. This was changed in v3 and was not backported to v2 before releasing v3.

With this patch, the `name` configuration key will be deprecated in favor of the `adapter` configuration key to ensure better backwards compatibility.

Signed-off-by: Maximilian Bösing <2189546+boesing@users.noreply.github.com>
@boesing boesing force-pushed the feature/adapter-configuration-array-key branch from a17e612 to 823776a Compare November 17, 2021 15:37
@boesing boesing merged commit d021354 into laminas:3.1.x Nov 17, 2021
@boesing boesing deleted the feature/adapter-configuration-array-key branch November 17, 2021 15:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working Enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant