Skip to content
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

prereq and dependency checks on post trash #263

Open
7 of 18 tasks
thomasplevy opened this issue Dec 1, 2016 · 3 comments
Open
7 of 18 tasks

prereq and dependency checks on post trash #263

thomasplevy opened this issue Dec 1, 2016 · 3 comments
Labels
Severity: Normal Type: Enhancement Improvements existing features or code
Milestone

Comments

@thomasplevy
Copy link
Contributor

thomasplevy commented Dec 1, 2016

use action wp_trash_post (https://core.trac.wordpress.org/browser/tags/4.5.3/src/wp-includes/post.php#L2656) to perform lookups before trashing

Also check into untrash actions

course

sections

  • unset lesson parent meta data

lesson

  • remove prerequisites from any other lessons
  • warnings on builder when lessons deleted (empty node on syllabus)

quiz

  • remove from any lessons
  • remove quiz attempts

questions

  • remove from any quizzes

track

  • remove prerequisite from any courses

membership

user

  • postmeta cleanup
  • order anonymization
  • cancel recurring orders

User cleanup moved to #940

orders

  • related transactions

WIP: https://github.com/gocodebox/lifterlms/blob/master/includes/class.llms.post.relationships.php

@thomasplevy
Copy link
Contributor Author

Transactions should be deleted when an order is deleted

eri-trabiccolo added a commit to eri-trabiccolo/lifterlms-rest that referenced this issue Jul 31, 2019
Also prevent possible fatal errors trying to get siblings of a section with no parent course:
actually we should delete child sections when deleting courses see gocodebox/lifterlms#263
@eri-trabiccolo
Copy link
Collaborator

eri-trabiccolo commented Jan 8, 2021

@thomasplevy
we should remove access plans when "parent" course/membership is permanently deleted, as we don't have the concept of an "orphan" access plan.
Should we add this to the list above or should we open a new issue?

@thomasplevy
Copy link
Contributor Author

@eri-trabiccolo added above

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Severity: Normal Type: Enhancement Improvements existing features or code
Projects
None yet
Development

No branches or pull requests

3 participants