Essentially this provides a backward compatibility layer so that we can continue to use existing plugins, but using the core system. It includes a custom discovery class and a generic manager class.

Patch forthcoming

CommentFileSizeAuthor
#13 ctools-1704824-13.patch32.52 KBdawehner
#10 ctools-1704824-10.patch32.5 KBtim.plunkett
#8 1704824-retrofit-plugin-system.patch32.91 KBmerlinofchaos
#5 ctools-1704824-5.patch52.44 KBtim.plunkett
#1 1704824-retrofit-plugin-system.patch64.52 KBmerlinofchaos

Comments

merlinofchaos’s picture

merlinofchaos commented
Status: Active » Needs work
Issue tags: +VDC
StatusFileSize
new 1704824-retrofit-plugin-system.patch64.52 KB

This patch is somewhat dirty and still has a bunch of TODOs, but it does work with the export_ui plugin exactly as is. However that's still using the registry which is going to go away, so we'll still have to move some stuff around later.

tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
commented
Status: Needs work » Needs review

Status: Needs review » Needs work

The last submitted patch, 1704824-retrofit-plugin-system.patch, failed testing.

tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
commented
Status: Needs work » Needs review
StatusFileSize
new ctools-1704824-5.patch52.44 KB

Uploading without the patch without the inception patch.

Status: Needs review » Needs work

The last submitted patch, ctools-1704824-5.patch, failed testing.

merlinofchaos’s picture

merlinofchaos commented

Aw nuts. This patch also includes the config exportable patch. I'll have to extricate.

merlinofchaos’s picture

merlinofchaos commented
Status: Needs work » Needs review
StatusFileSize
new 1704824-retrofit-plugin-system.patch32.91 KB

There, this patch has less cruft.

Status: Needs review » Needs work

The last submitted patch, 1704824-retrofit-plugin-system.patch, failed testing.

tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
commented
Status: Needs work » Needs review
StatusFileSize
new ctools-1704824-10.patch32.5 KB

Without windows line endings.

Status: Needs review » Needs work

The last submitted patch, ctools-1704824-10.patch, failed testing.

tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
commented
Issue tags: +CTools dependency

Tagging.

dawehner’s picture

dawehner
Primary language German
commented
Status: Needs work » Needs review
StatusFileSize
new ctools-1704824-13.patch32.52 KB

Just updated the patch, the todos are still left.

Status: Needs review » Needs work

The last submitted patch, ctools-1704824-13.patch, failed testing.

dawehner’s picture

dawehner
Primary language German
commented

Link the PSR0/Annotation Plugin conversion for the export ui: #1764194: Convert export_ui to annotations

tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
commented
Issue tags: -VDC, -CTools dependency

We don't actually need this anymore.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Issue summary: View changes
Status: Needs work » Closed (won't fix)

The 8.x-1.x branch has been abandoned, 8.x-2.x is being rewritten from scratch, so this is no longer relevant.

Feel free to reopen if there's a project it's still appropriate for.