The Auto Label tab is not displayed for Media Entity Bundles (NOT Media-in-core since Drupal core 8.4). It would be nice to beable to auto label media.
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | auto_entitylabel-2856152-4-Auto-Label-tab-not-displayed-for-Media.patch | 1.92 KB | geek-merlin |
Comments
Comment #2
geek-merlinThis was introduced in #2755445: Auto Entity label does not set entity labels and breaks contrib modules, and i think "only support core entities" was too narrow.
In the long run we should look like devel or field_tools provide any entity with a task.
For now let's just remove the disabling code.
Patch flying in.
Comment #3
geek-merlinOf course this patch does not remedy what was the original reason of the code but it might be a start and fixes the problem for me.
Comment #4
geek-merlinLooks like the condition when a autolabel makes sense was broken in the first place and restricting on core entities was a mere hack.
The right condition is ConfigEntityBundle.
Patch flying in.
Comment #5
gg24 commentedHi @axel.rutz,
I went through the patch and comment mentioned above and can clearly see that initially automatic entity label module was only supporting core modules not contrib modules and I appreciate that you took this initiative here and after applying the patch I am able to see the auto label tab too for media content as well but when I try to set the auto label for media Image I do not it working so my point here is once we are complete with the functionality than only we should move this code to code repo, just enabling the tab will be confusing as of now. Let me know if I am not clear here.
Note:- This patch ensures that auto label tab is visible for media content but functionality is not complete yet.
Thanks!
Comment #6
geek-merlinYes rite! I fixed that bug in #2923876: Broken when provider !=entityTypeId.
Comment #8
colanThanks! (I especially like patches where I can use
git am!!)Comment #9
geek-merlinCool this is in now!
Comment #10
geek-merlin> Thanks! (I especially like patches where I can use git am!!)
Nice you like that too. You can link people to the docs page i created: Making maintainer-friendly "git am" single and multi-patches | Drupal.org