Custom exporter logic

[gforcada]

Description

This is part of plone/plone.exportimport#86

Where an easy way to customize the default content exporter is added.

[stevepiercy]

@gforcada would you please request a review from whoever can provide a technical review? I've already requested two folks, based on the linked PR and the related training author.

[gforcada]

@stevepiercy thanks for doing that, I will try to remember next time. I will add another one, just to have higher chances of one actually reviewing it

[stevepiercy]

I'll do an editorial review, after the technical parts are verified. Thanks for your collaboration.

[davisagli]

This looks accurate for the proposed implementation in plone/plone.exportimport#86, but I also asked a question there to confirm whether @gforcada has confirmed this does what he needs.

[davisagli]

That's a bit of a limitation. Maybe you want all of the content sometimes, and different subsets at other times. This makes me wonder whether we should have named IObjectsExporter adapters, and make it possible to specify which one to use in the CLI

[gforcada]

I had quite a few ideas to improve what gets exported:

• add a control panel that lists what gets exported (a fancy print of what IObjectsExporter returns)
• add dexterity behavior that allows to mark objects to be always/never exported
  • thinking about privacy or mandatory objects
• on that control panel add a checkbox to use either the full or subset export

I'm not sure if there are enough users for such use cases, that's why we started with first allowing to customize the logic, and then we can expand further if there is a need for it

[gforcada]

@davisagli thanks for the review, all points are either addressed or commented 😄

[davisagli]

Ok from me from a technical perspective.

It should have a {versionadded} notice since it wasn't available in existing releases.