Closed (fixed)
Project:
Webform
Version:
7.x-3.x-dev
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
3 Jun 2010 at 13:29 UTC
Updated:
2 Jul 2010 at 03:20 UTC
Jump to comment: Most recent file
Comments
Comment #1
Stevel commentedThis patch seems to solve the issue, though I'm not sure the semantics are correct.
Comment #2
Stevel commentedComment #3
quicksketchThanks Stevel, I think we might be able to remove the dynamic weight entirely and just set it to 10 or something. The function http://api.lullabot.com/_field_extra_fields_pre_render/7 should fix the problem for us no?
Comment #4
quicksketchHmm, currently HEAD seems pretty broken. I can't find where "extra" field settings show up at all. I've got both Webform and Comments enabled on a content type but neither of them show up under the "Manage Display" tab.
The comment_node_page_additions() function doesn't seem to set a weight *at all*, which I believe indicates that Field module should set the weight for it. Unfortunately since you can't change the weight through the UI, it's a bit hard to test.
For the time being I'm just going to set the weight to 10 and assume that Field module will eventually start working again. To prevent bug reports saying Webform's weights don't work in Alpha 5, we'll use a conditional for the next few months until D7 stabilizes.