I doubt the current implementation can handle generic directed acyclic graphs. We should make sure a node is not inserted twice in the same tree, and thus forbid cardinality != 1 for treefields.

CommentFileSizeAuthor
#1 0001-Issue-1545368-by-fgm-Forbid-cardinality-1-for-treefi.patch2.14 KBfgm

Comments

fgm’s picture

fgm
Primary language French
Location Paris, France
commented
StatusFileSize
new 0001-Issue-1545368-by-fgm-Forbid-cardinality-1-for-treefi.patch2.14 KB

Difficult to implement a reasonable UI for this, so the most reasonable implementation is likely just hook_field_update_forbid().

fgm’s picture

fgm
Primary language French
Location Paris, France
commented
Status: Active » Needs review
damien tournoud’s picture

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

Sounds good.

fgm’s picture

fgm
Primary language French
Location Paris, France
commented
Status: Reviewed & tested by the community » Fixed

Committed to 7.x-1.x, thanks.

Status: Fixed » Closed (fixed)

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