I'm using the Field Formatter Class module and it would be nice to have the core DS fields (the ones that show up when you select a layout) to allow other modules to alter the "field's" settings. Maybe just adding them somehow to the list when field_info_instance_settings() is called might be appropriate?
I'll also mention that we're seeing some some weird warnings if using DS core fields: #1616426: Compatibility with Display Suite
The only other issue I could find that might somewhat resemble this issue: #1867828: Conditions on Display Suite fields are not behaving nicely + obsolete saving of field settings
Comment | File | Size | Author |
---|---|---|---|
#4 | error msg.png | 150.25 KB | Gyver06 |
#2 | 1875750-ds-field_settings-2.patch | 952 bytes | markhalliwell |
Comments
Comment #1
swentel CreditAttribution: swentel commentedHmm interesting, never used one of those field formatter modules on DS fields, besides field formatter conditions, which I maintain myself and where I had to write some workaround code for it.
Getting DS fields into field_info_instance_settings() is as good as impossible as I can't turn DS fields into 'real' Field API fields - at least, not in this stage without having a nearly impossible upgrade path.
However, I'll investigate as we'll probably start using more and more of those modules that depend on field formatter settings api.
Comment #2
markhalliwellOk, not entirely sure, but took a crack at it. Let me know if this patch makes sense, it did to me. If DS is calling
field_formatter_settings_summary()
when viewing a specific view mode, shouldn't that view mode's name be used in instance settings instead of just plain ol' default?Comment #3
markhalliwellNever mind, that just silences the error.
Comment #4
Gyver06 CreditAttribution: Gyver06 commentedUnfortunately, your patch doesn't solve the problem.
I tested it and no luck. Error still present.
See the joined capture screen.
Comment #5
swentel CreditAttribution: swentel commentedI'm going to mark this one duplicate as I think we have a better patch over at #1900014: Field formatter settings summary alter implementations choke on DS Pseudo fields which probably solves this completely.
Comment #6
markhalliwellActually that doesn't solve this issue. It would silence the errors, but the settings are actually never pulled in when configuring the formatter. It just passes an empty array. I'm not sure they would get saved either, now that I think about it.
Comment #7
mansspams CreditAttribution: mansspams commentedPlease see #1992948: Make DS leaner by outsourcing field formatting features as it could be related and could serve as higher level initiative for good changes.
Comment #8
markhalliwellFixed this in the Linked Field module (see related issue), which essentially does the same thing.
Comment #9
markhalliwellI suppose that means this is really a Field Formatter Class issue then.