ie, on taxonomy tags.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

swentel’s picture

Assigned: Unassigned » swentel
swentel’s picture

Title: Field UI crashes with the cogwheel » Field templates can not be available for certain field

eg, 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

swentel’s picture

Title: Field templates can not be available for certain field » Field templates can not be available for certain field, but the label should be
swentel’s picture

another related #1524128: Ignore fieldgroup for settings - note it's not related to fieldgroup but to custom fields apparently (quoting aspilicious)

aspilicious’s picture

In ds_extras_ds_field_settings_alter

We do this:

    // Field styles.
    if ($function != 'theme_ds_field_expert' && $function != 'theme_ds_field_reset') {
      $classes = implode(' ', $values['classes']);
      if (!empty($classes)) {
        $field_settings[$key]['ft']['classes'] = $classes;
      }
    }

In those cases swentel desribes $values['classes'] doesn't exist. So if we check for $values['classes'] that problem is solved.

swentel’s picture

While 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.

swentel’s picture

Title: Field templates can not be available for certain field, but the label should be » Field templates can not be available for certain fields, but the label should be (and other problems)
aspilicious’s picture

We 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...

swentel’s picture

We can't override the fieldgroup handlers - especially on their formatter select and summary.

swentel’s picture

The fieldgroup handlers don't work either in 1.5, I'll let zuuperman or stalski try and get a fix for it.

swentel’s picture

#1543428: Blog display does not match DS Manage Display settings. - troubles with the ds 'title' field when updating its settings.

swentel’s picture

swentel’s picture

Experience 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

Stalski’s picture

Status: Active » Needs review
FileSize
789 bytes

The 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.

swentel’s picture

Priority: Critical » Normal

Moving to normal. The critical errors are gone in dev, needs more UI/UX love now.

swentel’s picture

Status: Needs review » Needs work

That patch in #14 got in - also marked #1632390: Minimal field template not rendering field styles as a dupe

swentel’s picture

Title: Field templates can not be available for certain fields, but the label should be (and other problems) » Field templates, field refresh and cogwheel problems
kclarkson’s picture

I 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,

swentel’s picture

Issue tags: +beta blocker

tagging

swentel’s picture

Status: Needs work » Closed (fixed)
Issue tags: -beta blocker
FileSize
6.25 KB

Done, once and for all!

swentel’s picture

Status: Closed (fixed) » Active

God damn, I need to reopen this as it triggers notices in the background, no biggy, but still.

swentel’s picture

Status: Active » Closed (fixed)
FileSize
2.07 KB

Ok, crossing fingers now.

carajito’s picture

Status: Closed (fixed) » Fixed

It is fixed at the moment

Thanks swentel

Christopher Riley’s picture

Pulled 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?

aspilicious’s picture

cmriley some fields don't have any settings so they don't have a cog.

*if I'm correct*

swentel’s picture

Thats' probably related to #1733378: Not able to set the dynamic field settings - will investigate later.

Christopher Riley’s picture

I 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

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

mvonfrie’s picture

Status: Closed (fixed) » Needs work

I 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:

Ein AJAX-HTTP-Fehler ist aufgetreten.
HTTP-Rückgabe-Code: 500
Im Folgenden finden Sie Debugging-Informationen.
Pfad: /system/ajax
Statustext: Service unavailable (with message)
Antworttext: Recoverable fatal error: Argument 1 passed to ds_extras_field_template_settings_form() must be an array, null given, called in /var/www/oeh.aau.at/htdocs/drupal7/sites/all/modules/ds/includes/ds.field_ui.inc on line 2255 and defined in ds_extras_field_template_settings_form() (Zeile 285 von /var/www/oeh.aau.at/htdocs/drupal7/sites/all/modules/ds/modules/ds_extras/includes/ds_extras.admin.inc).

Module versions:
Core: 7.23
DS: 7.x-2.6+1-dev
OG: 7.x-2.3

swentel’s picture

Status: Needs work » Needs review
FileSize
531 bytes

That's very weird, could you try following patch ?

mvonfrie’s picture

The patch works.

aspilicious’s picture

Status: Needs review » Reviewed & tested by the community

Strange... but it's ok to commit this.

aspilicious’s picture

Version: 7.x-2.x-dev » 8.x-2.x-dev
Issue summary: View changes
Status: Reviewed & tested by the community » Patch (to be ported)

Fixed. Needs porting.

aspilicious’s picture

Status: Patch (to be ported) » Fixed

Doesn't need a fix for D8. Everything changed :)

aspilicious’s picture

Status: Fixed » Closed (fixed)