After upgrading to alpha2 the mailchimp component is not shown in the correct order. See the attached screenshots.

CommentFileSizeAuthor
#3 weight_fix-1499258.patch359 bytesrich.3po
Skærmbillede 2012-03-25 kl. 10.43.59.png1.31 MBlsolesen
Skærmbillede 2012-03-25 kl. 10.44.13.png136.09 KBlsolesen

Comments

beezer75’s picture

beezer75 commented
Title: Mailchimp outputted in incorrect order » not ideal, but a way to fix

I am having the same issue. Through some trial and error I figured out that you could set the order by adjusting the form element weights and using negative values for everything you want to appear above the newsletter sign up.

lsolesen’s picture

lsolesen commented
Title: not ideal, but a way to fix » Mailchimp outputted in incorrect order

Please, do not change the title of issues, if it does not clarify the problem.

You suggested approach is a temporary workaround, but the issue still needs to be fixed.

rich.3po’s picture

rich.3po commented
Status: Active » Needs review
StatusFileSize
new weight_fix-1499258.patch359 bytes

The problem seems to be in the _webform_render_mailchimp() function, where an array of elements is returned rather than an element itself, and no weight is set at the top level.

I've attached a patch which seems to fix the problem for me

-Rich

lsolesen’s picture

lsolesen commented
Status: Needs review » Reviewed & tested by the community

The patch applies cleanly and fixes the problem, so maybe a new release should be rolled :)

blackdog’s picture

blackdog commented
Status: Reviewed & tested by the community » Fixed

Committed, thanks!
New release coming up.

Status: Fixed » Closed (fixed)

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