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

Move the transformers into an independant bundle #92

Open
shavounet opened this issue Dec 2, 2019 · 3 comments
Open

Move the transformers into an independant bundle #92

shavounet opened this issue Dec 2, 2019 · 3 comments

Comments

@shavounet
Copy link
Contributor

shavounet commented Dec 2, 2019

Description

Is it a good idea ?

Transformers are a big part of the process bundle, and could be independent with a easier access to them.

@shavounet shavounet added this to the v4.0 milestone Dec 2, 2019
@xaviermarchegay xaviermarchegay removed this from the v4.0 milestone Oct 16, 2023
@xaviermarchegay
Copy link
Contributor

@tonongregory what is your take on it? it seems to be annoying to manage?

@VincentChalnot
Copy link
Contributor

For your information I already kind a did that on a personal project for test purposes and it makes sense but the it breaks backward compatibility completely because it involves moving all transformer in a new namespace.

For example I have a bundle that handles object mapping/conversion a bit like the Serializer but more advanced, Transformers are really helpful and I'm not using anything else from the ProcessBundle.
Ref : https://github.com/VincentChalnot/ConverterBundle/blob/v1.0-dev/reference.yaml

Hey @shavounet are you still using the ProcessBundle ?

@shavounet
Copy link
Contributor Author

Yep, 100% a major BC break, but worth it in some usecase (aka calling transformers without the full process bundle).

Nope, it didn't gain traction here, and now we're switching away from PHP

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants