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

Enhance data cleanup logic in cleaner.php #2032

Merged
merged 5 commits into from
Jan 21, 2025
Merged

Conversation

KytHub
Copy link
Collaborator

@KytHub KytHub commented Jan 15, 2025

Added a new condition to exclude 'wpcode' post type from deletion in the database cleanup query. This ensures better compatibility and prevents unintended removal of relevant data. Adjusted code formatting for consistency and readability. https://github.com/bagrinsergiu/blox-editor/issues/25526

Added a new condition to exclude 'wpcode' post type from deletion in the database cleanup query. This ensures better compatibility and prevents unintended removal of relevant data. Adjusted code formatting for consistency and readability. https://github.com/bagrinsergiu/blox-editor/issues/25526
@KytHub KytHub self-assigned this Jan 15, 2025
import/cleaner.php Outdated Show resolved Hide resolved
import/cleaner.php Outdated Show resolved Hide resolved
Reorganized the `clean` method to better modularize and streamline post-cleanup operations. Introduced dedicated methods for handling post relationships, comments, and media deletion, and improved maintainability by centralizing post type references. https://github.com/bagrinsergiu/blox-editor/issues/25526
@KytHub KytHub requested a review from ViorelEremia January 15, 2025 12:09
import/cleaner.php Show resolved Hide resolved
Nichita added 3 commits January 17, 2025 12:22
Introduce a new method `deletePostRevisions` to remove associated revisions for specified post types, including metadata and relationships. Additionally, correct a typo in the method name `deletePostsComments` to `deletePostComments` for consistency.
@alecszaharia alecszaharia changed the base branch from develop to hotfix-21-01 January 21, 2025 13:55
@alecszaharia alecszaharia merged commit 84c6303 into hotfix-21-01 Jan 21, 2025
@alecszaharia alecszaharia deleted the issue-25526 branch January 21, 2025 13:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants