When attempting to use this module with Automatic Entity Label, the Title field instead of the Label field is incorrectly hidden. I've attempted to get it to work properly to no avail. Therefore, I started working on adding an option to Bean itself. Here is my patch thus far.
The patch adds a row for the bean_type table
Adds an option on the edit bean page
Saves the value (but doesn't properly pull it up later)
However, I can't get the Bean Entity to recognize my new field. I don't have a solid grasp on Entities, so I'm sure I'm missing something obvious.
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | bean-auto-entitylabel-fix-1995770-3.patch | 404 bytes | zd370 |
| #2 | 1995770-autolabel-2.patch | 4.83 KB | albert volkman |
| #2 | interdiff.txt | 732 bytes | albert volkman |
| autolabel.patch | 4.12 KB | albert volkman |
Comments
Comment #1
damienmckennaComment #2
albert volkman commentedThis module now actually changes the label, but I still can't figure out how to verify the bean's auto label setting.
Comment #3
zd370 commentedI understand this thread is little old, but the issue is still not resolved.
@Albert Volkman, thanks for the patch. It may have solved your need, but it seems unnecessary to do all that work. The issue here is with the way Bean exposes its entity info via hook bean_entity_info().
Under entity keys, label is being mapped to the title field. Since, auto_entitylabel module relies on this information, it tried to automate the title field instead of label. The attached patch should solve the issue with Entity Auto Label module.
Comment #4
zd370 commentedComment #6
joelpittetWow that seems to make everything better, thank you @kris84
Comment #7
sylus commentedI was curious since this was left in the queue for so long is there a chance for any sort of regressions by switching from title to label? The fix seems appropriate for me but would be great if could get a bean maintainer to chime in :)
Comment #8
indytechcook commentedAny thoughts about how this would effect the other title/label related isseus?
Comment #9
indytechcook commentedComment #10
indytechcook commentedComment #11
joelpittetI'm not quite sure how those issues would react. Though that change really made a world of difference for the auto entity labeling.
Maybe cross post to them to see if this helps their issues as well?
Comment #12
BarisW commentedThanks for the patch. Autocompleted entity reference fields do work with this patch and other widgets still work too. Please commit!
Comment #13
joelpittetBump, this patch is awesome:)
Comment #14
damienmckennaCommitted. Lets see how the fallout goes.
Comment #16
joelpittetThank you for taking the plunge! Crosses fingers for no-fallout!