Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
ie, on taxonomy tags.
Comment | File | Size | Author |
---|---|---|---|
#30 | 1519916-30.patch | 531 bytes | swentel |
#22 | 1519916-22.patch | 2.07 KB | swentel |
#20 | 1519916-20.patch | 6.25 KB | swentel |
#14 | 1519916_14.patch | 789 bytes | Stalski |
Comments
Comment #1
swentel CreditAttribution: swentel commentedComment #2
swentel CreditAttribution: swentel commentedeg, preprocess fields and ignore field types. However, we need to make we print a cogwheel for changing the label though for say extra fields (like EVA).
Issues that are related:
- #1522188: Field settings toggle missing for Views fields added via Eva
- #1522200: Link fields field settings toggle causes js error
- #1247406: Field Display Options Not working
Comment #3
swentel CreditAttribution: swentel commentedComment #4
swentel CreditAttribution: swentel commentedanother related #1524128: Ignore fieldgroup for settings - note it's not related to fieldgroup but to custom fields apparently (quoting aspilicious)
Comment #5
aspilicious CreditAttribution: aspilicious commentedIn ds_extras_ds_field_settings_alter
We do this:
In those cases swentel desribes $values['classes'] doesn't exist. So if we check for $values['classes'] that problem is solved.
Comment #6
swentel CreditAttribution: swentel commentedWhile we're doing this: sometimes there's an extra break in the summary. Let's try to fix that as well.
Also, #1520162: Link to field formatter settings not shown via AJAX went in, but it's not 100% - eg post date from disabled to left, doesn't bring up the field templates gear.
Comment #7
swentel CreditAttribution: swentel commentedComment #8
aspilicious CreditAttribution: aspilicious commentedWe have problems on the view and on the save side. I fixed the "classes" issues which was a save issue.
http://drupalcode.org/project/ds.git/commit/81def2069cab8f3714ce301e7d18...
Comment #9
swentel CreditAttribution: swentel commentedWe can't override the fieldgroup handlers - especially on their formatter select and summary.
Comment #10
swentel CreditAttribution: swentel commentedThe fieldgroup handlers don't work either in 1.5, I'll let zuuperman or stalski try and get a fix for it.
Comment #11
swentel CreditAttribution: swentel commented#1543428: Blog display does not match DS Manage Display settings. - troubles with the ds 'title' field when updating its settings.
Comment #12
swentel CreditAttribution: swentel commented#1548386: Manage Display Field Template = Minimal: Label Element Being Ignored marked as dupe
Comment #13
swentel CreditAttribution: swentel commentedExperience this at a current work project as well, error messages was:
Recoverable error: argument 1 passed to ds_extras_field_template_settings_form must be an array, null given, line 382
Comment #14
Stalski CreditAttribution: Stalski commentedThe error in #13 is caused by a variable $settings_form which is not always set.
Since the output of the function can be handled totally different, so for now, it's babysitting time. It would be nice to be able to deal with a certain array, but in vain.
I patched the most simple solution that I could see. Probably this issue has other sub-issues but this one is ready to be reviewed.
Comment #15
swentel CreditAttribution: swentel commentedMoving to normal. The critical errors are gone in dev, needs more UI/UX love now.
Comment #16
swentel CreditAttribution: swentel commentedThat patch in #14 got in - also marked #1632390: Minimal field template not rendering field styles as a dupe
Comment #17
swentel CreditAttribution: swentel commentedmarking #1669926: Extras field templates and cog disappear in the Field UI as dupe
Comment #18
kclarkson CreditAttribution: kclarkson commentedI am using views display and the field template overrides are not working.
I tried all of the formats; Full Reset, Minimal and Expert and still nothing.
Thinking that it might be theme related I also tried Bartik.
Thanks,
Comment #19
swentel CreditAttribution: swentel commentedtagging
Comment #20
swentel CreditAttribution: swentel commentedDone, once and for all!
Comment #21
swentel CreditAttribution: swentel commentedGod damn, I need to reopen this as it triggers notices in the background, no biggy, but still.
Comment #22
swentel CreditAttribution: swentel commentedOk, crossing fingers now.
Comment #23
carajito CreditAttribution: carajito commentedIt is fixed at the moment
Thanks swentel
Comment #24
Christopher Riley CreditAttribution: Christopher Riley commentedPulled the latest dev version and I am still missing the cog. Was going to apply the patch but it says it has already been applied. Am I missing something here or is this still having issues?
Comment #25
aspilicious CreditAttribution: aspilicious commentedcmriley some fields don't have any settings so they don't have a cog.
*if I'm correct*
Comment #26
swentel CreditAttribution: swentel commentedThats' probably related to #1733378: Not able to set the dynamic field settings - will investigate later.
Comment #27
Christopher Riley CreditAttribution: Christopher Riley commentedI went ahead and downgraded to 7.x-1.5 and it put things back the way they should be. I will worry about debugging this later when I have time, if it is not corrected before then.
Thanks
Comment #29
mvonfrie CreditAttribution: mvonfrie commentedI need to re-open this issue because I get the following error when clicking the cog wheel of an entity reference for an OG reference:
Module versions:
Core: 7.23
DS: 7.x-2.6+1-dev
OG: 7.x-2.3
Comment #30
swentel CreditAttribution: swentel commentedThat's very weird, could you try following patch ?
Comment #31
mvonfrie CreditAttribution: mvonfrie commentedThe patch works.
Comment #32
aspilicious CreditAttribution: aspilicious commentedStrange... but it's ok to commit this.
Comment #33
aspilicious CreditAttribution: aspilicious commentedFixed. Needs porting.
Comment #34
aspilicious CreditAttribution: aspilicious commentedDoesn't need a fix for D8. Everything changed :)
Comment #35
aspilicious CreditAttribution: aspilicious commented