Steps to reproduce:

1) edit a node containing at least one field collection item.

2) upload one or several files using the bulk upload widget.

Result: the last FC item listed in step 1 has been overwritten by the first FC item created in step 2.

-------

My setup:
- Field Collection Bulk Upload 7.x-1.0-alpha1
- Plupload integration 7.x-2.x-dev
- File Field Sources 7.x-1.8

CommentFileSizeAuthor
#6 field_collection_bulkupload-fix_last_item_disappearing-2098649-6.patch1.2 KBethan.han777
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

rombapa’s picture

rombapa CreditAttribution: rombapa commented

Has anyone had a chance to reproduce this?

rombapa’s picture

rombapa CreditAttribution: rombapa commented
Issue summary: View changes
rombapa’s picture

rombapa CreditAttribution: rombapa commented

My first description of the issue was incorrect: field collection items must already exist in the node in order to reproduce the problem (i.e. the node must be saved with one or more FC items then re-edited).

Mackee’s picture

Mackee CreditAttribution: Mackee commented

Same here

abrar_arshad’s picture

abrar_arshad CreditAttribution: abrar_arshad commented

Adding the below line before #195 solves the problem for me

$upload_delta = $upload_delta + 1;
ethan.han777’s picture

ethan.han777 CreditAttribution: ethan.han777 commented
FileSize
field_collection_bulkupload-fix_last_item_disappearing-2098649-6.patch1.2 KB

Hi,

I made a patch for this issue.

ethan.han777’s picture

ethan.han777 CreditAttribution: ethan.han777 commented
Status: Active » Needs review
rombapa’s picture

rombapa CreditAttribution: rombapa commented

The patch works for me. Thanks ethan.han777.

joseph11’s picture

joseph11 CreditAttribution: joseph11 commented

Patch works for me. Thanks.

dxx’s picture

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

Patch works for me.

tom.burton’s picture

tom.burton CreditAttribution: tom.burton at Interdruper commented

Patch #6 worked also for me. Thx.

  • ram4nd committed 5fea29f on 7.x-1.x authored by ethan.han777 
    Issue #2098649 by ethan.han777, rombapa, dxx: Last existing field...
ram4nd’s picture

ram4nd CreditAttribution: ram4nd as a volunteer commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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