New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Packaging] Delete content type step #330

Closed
tusmester opened this Issue Apr 8, 2018 · 0 comments

Comments

Projects
None yet
2 participants
@tusmester
Member

tusmester commented Apr 8, 2018

It would be useful to have an SnAdmin step that is able to delete one content types from the system.

<DeleteContentTypes names="Car" />
<DeleteContentTypes>Car</DeleteContentTypes>

This step should

  • delete contents of this type
  • delete content templates of this type
  • delete applications for this type
  • delete content views from /Root/Global/contenetviews and /Root/Skins/.../contentviews related to this type
  • remove this type from the repository where this is an allowed type:
    • other CTDs
    • any reference field
    • any content
  • delete the CTD
  • execute recursively
  • add to built-in steps docs

Extra:

  • default execution shows information.
  • deletion is triggered by an extra parameter.
    • only information: <DeleteContentType name="ContentType1" />
    • delete anyway: <DeleteContentType name="ContentType1" delete="force" />
    • delete if not instantiated: <DeleteContentType name="ContentType1" delete="ifNotUsed" />

Maybe the main logic should be developed in the repository layer (instead of directly in a step) so that an action can call it later if needed.

(the code will be backported into an sn6 custom step library anyway, but we should add it to the official codebase too for future use)

@tusmester tusmester added this to the Sprint 156 milestone Apr 8, 2018

@kavics kavics self-assigned this Apr 10, 2018

@tusmester tusmester modified the milestones: Sprint 156, Sprint 157 Apr 18, 2018

@tusmester tusmester removed the discussion label May 2, 2018

@tusmester tusmester closed this May 2, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment