Skip to content

Get ambiguous entity identifier#3

Open
habashysr wants to merge 1 commit into
LRotherfield:masterfrom
habashysr:master
Open

Get ambiguous entity identifier#3
habashysr wants to merge 1 commit into
LRotherfield:masterfrom
habashysr:master

Conversation

@habashysr

Copy link
Copy Markdown

Regardless the field name is, transformer would still be able to get the identifier value

Regardless the field name is, transformer would still be able to get the identifier value
@stanlemon

Copy link
Copy Markdown

This would be super handy, I have a few weird scenarios where the identifier of an entity is not getId() and I am unable to use this package. The best example I can give is if you are using UUID's for object identifiers and decide to name your identifier field accordingly, so the method is getUuid() this component would not be usable.

Would love to see this merged. 👍

@LRotherfield

Copy link
Copy Markdown
Owner

I need to test this, sorry for taking 2 years to get around to this. I will do it today at some point

@LRotherfield

Copy link
Copy Markdown
Owner

Has anyone tested this on Symfony 3?

@habashysr

Copy link
Copy Markdown
Author

Nop but I don't think we will change much code

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants