Closed (fixed)
Project:
Wysiwyg Fields
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
19 Jan 2012 at 13:40 UTC
Updated:
8 Feb 2012 at 09:20 UTC
Jump to comment: Most recent file
Comments
Comment #1
chester_martin commentedI haven't yet figured out exactly what this function does, but it seems that it is trying to populate 'format' even in textfields that don't use 'format'. ie plaintext-fields. Can the solution be to check if current field uses 'format'? Maybe there's a better solution but I will try to submit a patch.
Comment #2
chester_martin commentedhere's a simple patch. There's a comment in the code saying "// @TODO - Use field_info_instances() for more efficiency." so this is probably just a quick solution. It helped me anyway.
Comment #3
decipheredIt's actually a left over from the D6 version to clear the field cache. It may or may not be needed in the future, so for the moment I will comment the line out until that can be determined. I can confirm that it isn't in use at all at the moment.
Comment #4
decipheredThe infringing line has been commented out for the moment and committed to 7.x-1.x, will re-visit it at a later stage.
Comment #5
chester_martin commentedThats great! Thanks!