Discriminator not working #6996

Open
opened 2026-01-22 15:42:48 +01:00 by admin · 5 comments
Owner

Originally created by @Ciloe on GitHub (Jun 27, 2022).

Bug Report

Q A
BC Break no
Version 2.13.x-dev

Summary

I have a problem with discriminator when using KnpLabs repository for translation. I created this issue https://github.com/KnpLabs/DoctrineBehaviors/issues/704. When I insert translation on discriminator entity, the discr field is not set. The column not appear in the insertColumns array

Current behavior

The column discr is null

Expected behavior

When entity translation with discriminator is set, this will insert the discr column with correct value

Originally created by @Ciloe on GitHub (Jun 27, 2022). ### Bug Report | Q | A |------------ | ------ | BC Break | no | Version | 2.13.x-dev #### Summary I have a problem with discriminator when using KnpLabs repository for translation. I created this issue https://github.com/KnpLabs/DoctrineBehaviors/issues/704. When I insert translation on discriminator entity, the `discr` field is not set. The column not appear in the insertColumns array #### Current behavior The column `discr` is `null` #### Expected behavior When entity translation with discriminator is set, this will insert the `discr` column with correct value
Author
Owner

@Ciloe commented on GitHub (Jul 20, 2022):

Any updates ?

@Ciloe commented on GitHub (Jul 20, 2022): Any updates ?
Author
Owner

@greg0ire commented on GitHub (Jul 21, 2022):

I have no idea about your issue, but

abstract class AbstractBookableActivityTranslation extends AbstractEntity implements TranslationInterface appears twice in your issue, I don't think that's normal, and I think it doesn't help understanding your issue.

@greg0ire commented on GitHub (Jul 21, 2022): I have no idea about your issue, but `abstract class AbstractBookableActivityTranslation extends AbstractEntity implements TranslationInterface` appears twice in your issue, I don't think that's normal, and I think it doesn't help understanding your issue.
Author
Owner

@Ciloe commented on GitHub (Jul 21, 2022):

Thanks, that's a mistake. I updated my issue.

@Ciloe commented on GitHub (Jul 21, 2022): Thanks, that's a mistake. I updated my issue.
Author
Owner

@mpdude commented on GitHub (Jan 16, 2023):

Please provide a reproducible example here. Also your code over at the other repo does not show all the classes being used.

@mpdude commented on GitHub (Jan 16, 2023): Please provide a reproducible example here. Also your code over at the other repo does not show all the classes being used.
Author
Owner

@Ciloe commented on GitHub (Jan 17, 2023):

It will be difficult. I will check that

@Ciloe commented on GitHub (Jan 17, 2023): It will be difficult. I will check that
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: doctrine/archived-orm#6996