Steps to reproduce (using Atoms):

  1. Add two new entity references one by one, without saving the form.
  2. Click "Show row weights" to switch from drag & drop mode to manual mode, you should see that `treeable_parent` element is only shown for the first item.
  3. Reorder items or organize them in a tree.
  4. Save.

Current result: Empty of broken tree of entity references.

Expected result: Proper tree.

As I figured out this happens because when you adding a second entity reference the form is not complete — `treeable_parent` element is missing.

CommentFileSizeAuthor
#5 adding_form_state_implementation-2605876-5.patch936 bytesmohit_aghera
#3 2605876-3-edit.png55.61 KBanton-staroverov
#3 2605876-2-result.png22.63 KBanton-staroverov
#3 2605876-1-create.png49.87 KBanton-staroverov

Comments

tonystar created an issue. See original summary.

anton-staroverov’s picture

anton-staroverov commented
Issue summary: View changes
anton-staroverov’s picture

anton-staroverov commented
Issue summary: View changes
StatusFileSize
new 2605876-1-create.png49.87 KB
new 2605876-2-result.png22.63 KB
new 2605876-3-edit.png55.61 KB
anton-staroverov’s picture

anton-staroverov commented
Issue summary: View changes
mohit_aghera’s picture

mohit_aghera commented
Assigned: Unassigned » mohit_aghera
StatusFileSize
new adding_form_state_implementation-2605876-5.patch936 bytes

From debugging in detail it seems that form_state was not giving appropriate value.
Updating field value count retrieval code.

mohit_aghera’s picture

mohit_aghera commented
Status: Active » Needs review
anton-staroverov’s picture

anton-staroverov commented

Works for me! Awesome!

  • tonystar committed a46d923 on 7.x-1.x 
    Issue #2605876 by mohit_aghera, tonystar: Adding two entity references...
anton-staroverov’s picture

anton-staroverov commented
Status: Needs review » Reviewed & tested by the community
anton-staroverov’s picture

anton-staroverov commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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