Closed (fixed)
Project:
Image Class
Version:
7.x-1.0
Component:
User interface
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
14 Jan 2014 at 22:39 UTC
Updated:
31 Jan 2014 at 15:20 UTC
Jump to comment: Most recent
Comments
Comment #1
yannickooHey Kyle,
this is absolutely not intended but this is not an issue of Image Class module. You can see in code that only the image formatter is affected.
Please open an issue in the Field formatter settings issue queue but I cannot help you with this issue.
Sorry :(
Comment #2
Melissamcewen commentedI'm having this issue as well. Has anyone opened up a bug on Field formatter for this?
Comment #3
kyletaylored commentedI just did.
#2174649: Image Class formatter shows up on all fields
Comment #4
dave reidThe code in this module does not restrict its API alter hook to only the image formatter.
See http://drupalcode.org/project/field_formatter_settings.git/blob/refs/hea... and the context provided. This module should be checking if
$context['instance']['display'][$context['view_mode']]['type'] = 'image'inside image_class_field_formatter_settings_form_alter() and image_class_field_formatter_settings_summary_alter().@yannickoo: I have no clue why you'd say the issue is in the Field Formatter Settings module, considering you know the code more than others. It is an alter hook. Alter hooks get run on every module that implements the hook. You cannot assume that the alter hook only gets invoked in your module when you care, and not all times.
Comment #5
yannickooSorry Dave for blaming your module D:
This was fixed with ba9626e now and everything is fine.
Dave, why you suggested to add an
ifinside the summary alter? The settings should not be available at that point, right?Comment #6
yannickooComment #7
dave reidBecause the summary alter runs on *all* modules, so you must restrict adding or altering the summary *only* if the formatter you care about is being used. It looks like both hooks are fixed now though, so good!
Comment #8
yannickooThanks again! Did not need that before because Linked Field is available for every field :)