[Serializer] SerializedName based on groups

[davidschmidt]

Description
It would be helpful to be able to change the serialized name based on the serialization group.

Example

class Person
{
    /**
     * @SerializedName(name="address1", groups={"registration"})
     * @SerializedName(name="address2", groups={"update"})
     * @Groups({"registration", "update", "view"})
     * @var string
     */
    public $address;
}

[jsamouh]

maybe this would be less problematic to change the existing code:

 * @SerializedName({
 *     {"name"="Test1", "groups"={"group1"}},
 *     {"name"="Test2", "groups"={"group2"}},
*     })

[jsamouh]

in any case it is difficult to add this feature without break BC... @dunglas wdyt ?

[karatektus]

Any updates? this would be great :)

[wajihweriemi]

or alternatives ? maybe ?

[wajihweriemi]

using getter and setter could be a solution, but it will be a getter and setter for each group

[dunglas]

PR welcome, it shouldn't be too hard to implement without BC break (@SerializedName("foo", groups="g")).

[jsamouh]

I can try this week end
Should I create a MR with a specific Symfony version ? or shoud I start from master ?

[YaFou]

@jsamouh it's a new feature so you must start from master.

[jsamouh]

Do we agree to use this syntax for annotation ?

 * @SerializedName({
 *     {"name"="Test0"},
 *     {"name"="Test1", "groups"={"group1"}},
 *     {"name"="Test2", "groups"={"group2"}},
 * })

[jsamouh]

First try on annotation.

• Need discussion about the approach (maybe i am totally wrong :-) ) before continuing
• Miss more tests
• Missing support for YAML and XML
• Need your feedback

[carsonbot]

Thank you for this suggestion.
There has not been a lot of activity here for a while. Would you still like to see this feature?

[carsonbot]

Just a quick reminder to make a comment on this. If I don't hear anything I'll close this.

[davidschmidt]

Thank you for this suggestion.
There has not been a lot of activity here for a while. Would you still like to see this feature?

I would still like to see this.

[kniziol]

I would still like to see this

Me too

[derrabus]

There already was a promising PR by @jsamouh. Maybe someone (or @jsamouh himself) wants to pick that up and resume the work?

[carsonbot]

Thank you for this suggestion.
There has not been a lot of activity here for a while. Would you still like to see this feature?

[hafkenscheid]

Still relevant in my opinion

[BenWaNH]

It would be nice !

[p-lorenzo]

Really nice!

[jsamouh]

will dot it next week

[jotaseme]

any update?

[kniziol]

will dot it next week

@jsamouh Did you switch week in your calendar? 😉

[SebaGnich]

Hey guys :) Do you have any news on this?

[carsonbot]

Thank you for this suggestion.
There has not been a lot of activity here for a while. Would you still like to see this feature?

[martinbertinat]

Yes. It would be awesome to have this.

[kniziol]

There has not been a lot of activity here for a while. Would you still like to see this feature?

Yes, sir 🙂

[jdflament]

Yes, it would be awesome.

[xabbuh]

Someone will have to do the actual work for this. It doesn't look like that is the case.

[nicolas-grekas]

See #46432 ;)

[carsonbot]

Thank you for this suggestion.
There has not been a lot of activity here for a while. Would you still like to see this feature?

[carsonbot]

Friendly reminder that this issue exists. If I don't hear anything I'll close this.

[pauloafonso]

Hey guys! So glad this feature will be released in 6.3!

And it would also be awesome if serialized_path could be able to be based on groups too! Does anyone agree?

I did a search and didn't find anything related to this.

Thanks!

[haithem-rihane]

Any update on this ?

[kniziol]

Any update on this ?

Please see the #46432 PR mentioned above by @nicolas-grekas