Skip to content

Conversation

@soderlind
Copy link
Owner

This pull request introduces a new feature that protects folders with active rules from being deleted, and updates the Norwegian Bokmål (nb_NO) translations to support this feature. It also updates the changelog and language metadata to reflect the new version.

Feature Addition:

  • Folder Deletion Protection
    • Folders that have active rules assigned to them cannot be deleted until those rules are removed first. This prevents accidental loss of rule associations and improves data integrity. [1] [2]

Localization Updates:

  • Norwegian Bokmål Translation
    • Added and updated translation strings in vmfa-rules-engine-nb_NO-dfbff627e6c248bcb3b61d7d06da9ca9.json and vmfa-rules-engine-nb_NO.l10n.php to support new folder creation and deletion messages, including the new folder deletion protection message. [1] [2]

Documentation:

  • Changelog and Versioning
    • Updated CHANGELOG.md to document the new feature and added a link for version 1.2.1. [1] [2]

- Updated version number in package.json, readme.txt, and main plugin file (vmfa-rules-engine.php) from 1.2.0 to 1.2.1 to reflect the new release.
- Added a new feature in Plugin.php to protect folders with active rules from being deleted. This is achieved by introducing a new filter 'vmfo_can_delete_folder' that checks if a folder has associated rules before allowing deletion.
- Enhanced user feedback by returning a WP_Error if a user attempts to delete a folder that has active rules, with a message indicating the folder cannot be deleted until the rules are removed.
- Updated the translation strings in languages/vmfa-rules-engine.pot to include the new error message for folder deletion protection.
- Documented the new functionality in the changelog section of readme.txt, noting the addition of folder deletion protection.
@soderlind soderlind merged commit 7492208 into main Jan 23, 2026
3 checks passed
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.

1 participant