Discussion: directly set object property is probably not a good idea

[patrickli]

While easy and working, I don't like the idea that this bundle sets the vich uploader path included filename back to the object itself. Because if the entity manager is flushed after serialization, it will save the path-prepended file field into the db.

A better way is to ignore both the filename and file object fields, then use the addData() method on post-serialize to add the fully qualified file path to the serialized result. However this requires the user to manually mark the fields as excluded and only works for json since addData() is only available for json type.

Any thoughts?

[draco1989]

I created the bundle in order to be able to get the image url at serialization time, that means that all the logic should be done already

The correct way to output the serialized object should be with annotation @View() of FOS/RestBundle, serialize filter of twig or by returning a Response with the serialized object, so do not think the case to be common.

But if you have any idea valid for json as for xml, i'll be pleased to discuss it.