Need to add 'format_handlers' to $sort_blacklist array.

I ran into a situation with a combination of addressfield, commerce_addressbook, and a checkbox provided by commerce_shipping. If their custom order wasn't maintained in the features field export, a validation error would occur when using the "use billing address" checkbox.

CommentFileSizeAuthor
#3 features-field-array-order-1532422-3.patch612 bytesnadavoid
#1 features-field-array-order-1532422-1.patch595 bytesnadavoid

Comments

nadavoid’s picture

nadavoid commented
Status: Active » Needs review
StatusFileSize
new features-field-array-order-1532422-1.patch595 bytes

Patch attached for review.

hefox’s picture

hefox commented

array(
'item_1',
'item_2',
);

how about that syntax so next item that needs to be popped in can do it easily

nadavoid’s picture

nadavoid commented
StatusFileSize
new features-field-array-order-1532422-3.patch612 bytes

@hefox You got it. I kinda wondered about that, but opted for the minimal-change route at first. Thank you for the suggestion.

mpotter’s picture

mpotter commented
Version: 7.x-1.x-dev » 6.x-1.x-dev
Status: Needs review » Patch (to be ported)

Looks good. Committed and pushed to 005d00e.

  • mpotter committed 005d00e on 8.x-3.x 
    Issue #1532422 by nadavoid: Fixed If the custom sort order of '...