[PR #203] [MERGED] Added a check in Mailbox preRemove hook call. If the hook is returnin… #300

New issue

Closed

opened 2026-02-26 10:30:41 +03:00 by kerem · 0 comments

kerem commented

2026-02-26 10:30:41 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/opensolutions/ViMbAdmin/pull/203
Author: @mfechner
Created: 8/27/2016
Status: ✅ Merged
Merged: 8/28/2016
Merged by: @barryo

Base: master ← Head: mailbox_preRemove_can_stop_removal

📝 Commits (1)

9d9b0b4 Added a check in Mailbox preRemove hook call. If the hook is returning a false, stop the deletion of the mailbox. The plugin has to add a message why the deletion of the mailbox was canceled.

📊 Changes

1 file changed (+18 additions, -17 deletions)

View changed files

📝 application/controllers/MailboxController.php (+18 -17)

📄 Description

…g a false, stop the deletion of the mailbox. The plugin has to add a message why the deletion of the mailbox was canceled.

This change is required to further implement pull request #179

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/opensolutions/ViMbAdmin/pull/203 **Author:** [@mfechner](https://github.com/mfechner) **Created:** 8/27/2016 **Status:** ✅ Merged **Merged:** 8/28/2016 **Merged by:** [@barryo](https://github.com/barryo) **Base:** `master` ← **Head:** `mailbox_preRemove_can_stop_removal` --- ### 📝 Commits (1) - [`9d9b0b4`](https://github.com/opensolutions/ViMbAdmin/commit/9d9b0b4140fe4b4023bb3a20d9af817a403bbd0b) Added a check in Mailbox preRemove hook call. If the hook is returning a false, stop the deletion of the mailbox. The plugin has to add a message why the deletion of the mailbox was canceled. ### 📊 Changes **1 file changed** (+18 additions, -17 deletions) <details> <summary>View changed files</summary> 📝 `application/controllers/MailboxController.php` (+18 -17) </details> ### 📄 Description …g a false, stop the deletion of the mailbox. The plugin has to add a message why the deletion of the mailbox was canceled. This change is required to further implement pull request #179 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>