We are going to have several provider implementations, we need to be able to select them in every treefield.

CommentFileSizeAuthor
#6 0001-Issue-1531170-by-fgm-Allow-selecting-the-provider-fo.patch3.06 KBfgm
#3 0001-Issue-1531170-Allow-selecting-the-provider-per-treef.patch5.96 KBfgm
#2 0001-Issue-1531170-Allow-selecting-the-tree-provider-per-.patch6.84 KBfgm
#1 1531170-provider-pluggeable.patch5.4 KBdamien tournoud

Comments

damien tournoud’s picture

damien tournoud commented
Status: Active » Needs review
StatusFileSize
new 1531170-provider-pluggeable.patch5.4 KB

First shot at this. Not really tested.

fgm’s picture

fgm
Primary language French
Location Paris, France
commented
StatusFileSize
new 0001-Issue-1531170-Allow-selecting-the-tree-provider-per-.patch6.84 KB

Rerolled against latest dev version.

fgm’s picture

fgm
Primary language French
Location Paris, France
commented
StatusFileSize
new 0001-Issue-1531170-Allow-selecting-the-provider-per-treef.patch5.96 KB

Rerolled with module-relative paths. Applies on top of #1543226: Tree_Provider_Simple test fails for lack of postUpdate: refactor.

fgm’s picture

fgm
Primary language French
Location Paris, France
commented
Status: Needs review » Fixed

Committed to 7.x-1.x.

fgm’s picture

fgm
Primary language French
Location Paris, France
commented
Status: Fixed » Needs work

Extra changes needed: draggable views handler incorrectly selects the tree behavior.

fgm’s picture

fgm
Primary language French
Location Paris, France
commented
Status: Needs work » Needs review
StatusFileSize
new 0001-Issue-1531170-by-fgm-Allow-selecting-the-provider-fo.patch3.06 KB

Patch attached

damien tournoud’s picture

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

Ooops. Thanks for the fix.

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.