Grégoire Paris comments

Results 933 comments of


                                            Grégoire Paris

MySQL + deadlock leads to `SAVEPOINT DOCTRINE_x does not exist`

@nikophil I'm not sure I understand the issue. Are you saying: 1. there is a deadlock 2. code inside the `finally` throws another exception about a savepoint that does not...

A new media is created after update an old one from Admin Form

`new_on_update` defaults to `true`, but it looks like you should be able to set it: https://github.com/sonata-project/SonataMediaBundle/blob/fabd14b939267ab9ef622963f14aed1cf26caa4c/Resources/doc/reference/form.rst .

A new media is created after update an old one from Admin Form

Have you checked the contents of [this variable](https://github.com/sonata-project/SonataMediaBundle/blob/3.x/Form/DataTransformer/ProviderDataTransformer.php#L148) at runtime?

A new media is created after update an old one from Admin Form

Yeah but options are not passed on construct, they are passed when calling `getOptions`. You can throw an exception in that method, that will give you a stack trace, helpful...

A new media is created after update an old one from Admin Form

Indeed. Please have a look at the calling function to know why it is empty there, and so on.

A new media is created after update an old one from Admin Form

You're right, reading your stack trace, it looks like `getOptions` is called in the constructor.

A new media is created after update an old one from Admin Form

I think the right fix would be to have a property on the admin to configure that, and expose it in the configuration.

A new media is created after update an old one from Admin Form

Or maybe it worked as you expected before? Can you check previous versions of the media bundle?

A new media is created after update an old one from Admin Form

I don't know or use this bundle, but the answer seems to be in the docs > then the media will be overwritten and **related entities can be affected by...

A new media is created after update an old one from Admin Form

I don't know, but I think it is based on the sandbox, so you can have a look. > I think they're talking about the asset no the entity. I...