When you try to remove multiple mappings at once in the Feeds UI mapping form, only half of the selected mappings are removed.

Fix attached.

CommentFileSizeAuthor
#1 1085194-1-not_all_mappers_removed.patch959 bytespfrenssen
not_all_mappers_removed.patch657 bytespfrenssen
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

pfrenssen’s picture

pfrenssen
Location Sofia
CreditAttribution: pfrenssen commented
FileSize
1085194-1-not_all_mappers_removed.patch959 bytes

Fixed the patch.

johnv’s picture

johnv
Primary language Dutch
CreditAttribution: johnv commented
Status: Needs review » Reviewed & tested by the community

Nice to have this annoying bug gone.

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented
Status: Reviewed & tested by the community » Fixed

Committed to Git.
http://drupalcode.org/project/feeds.git/commit/eee8ae7
http://drupalcode.org/project/feeds.git/commit/9987ed7

Status: Fixed » Closed (fixed)

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