Tickets :: [#11253] error messages for max_timelimit, max_recipient don't show the real values

6.0.0-beta1

7/20/25

Summary	error messages for max_timelimit, max_recipient don't show the real values
Queue	IMP
Queue Version	5.0.22
Type	Bug
State	Resolved
Priority	2. Medium
Owners	jan (at) horde (dot) org
Requester	dirk.dettmering (at) med (dot) uni-marburg (dot) de
Created	07/09/2012 (4759 days ago)
Due
Updated	10/15/2012 (4661 days ago)
Assigned
Resolved	07/09/2012 (4759 days ago)
Github Issue Link
Github Pull Request
Milestone	5.0.23
Patch	No

10/15/2012 08:35:17 PM	Git Commit	Comment #6	Reply to this comment
Changes have been made in Git (develop): commit a2db518ef0890dd2ae08ec41980bd6a751729b4d Author: Jan Schneider <jan@horde.org> Date: Mon Jul 9 19:43:16 2012 +0200 Fix permission checking in IMP (~~Bug #11253~~). IMP's hasPermission() worked completely different than any other application's hasPermission() method. Fix that and create a wrapper method in the IMP class instead. The method name is still misleading, but at least it works consistently again, and the error message are correct now. imp/docs/CHANGES \| 1 + imp/lib/Application.php \| 40 +----------------------------------- imp/lib/Compose.php \| 10 ++++---- imp/lib/IMP.php \| 51 +++++++++++++++++++++++++++++++++++++++++++++++ imp/lib/Mailbox.php \| 8 +++--- imp/package.xml \| 4 +- 6 files changed, 64 insertions(+), 50 deletions(-) http://git.horde.org/horde-git/-/commit/a2db518ef0890dd2ae08ec41980bd6a751729b4d

08/29/2012 12:41:46 PM	Git Commit	Comment #5	Reply to this comment
Changes have been made in Git (master): commit 8810240d970ad7754fbb8e307b1bda37d335f471 Author: Jan Schneider <jan@horde.org> Date: Mon Jul 9 19:43:16 2012 +0200 Fix permission checking in IMP (~~Bug #11253~~). IMP's hasPermission() worked completely different than any other application's hasPermission() method. Fix that and create a wrapper method in the IMP class instead. The method name is still misleading, but at least it works consistently again, and the error message are correct now. Conflicts: imp/lib/Application.php imp/lib/Compose.php imp/lib/Mailbox.php imp/package.xml imp/docs/CHANGES \| 1 + imp/lib/Application.php \| 40 +----------------------------------- imp/lib/Compose.php \| 10 ++++---- imp/lib/IMP.php \| 51 +++++++++++++++++++++++++++++++++++++++++++++++ imp/lib/Mailbox.php \| 8 +++--- imp/package.xml \| 2 +- 6 files changed, 63 insertions(+), 49 deletions(-) http://git.horde.org/horde-git/-/commit/8810240d970ad7754fbb8e307b1bda37d335f471

07/11/2012 09:45:34 AM	Git Commit	Comment #4	Reply to this comment
Changes have been made in Git (develop): commit 8810240d970ad7754fbb8e307b1bda37d335f471 Author: Jan Schneider <jan@horde.org> Date: Mon Jul 9 19:43:16 2012 +0200 Fix permission checking in IMP (~~Bug #11253~~). IMP's hasPermission() worked completely different than any other application's hasPermission() method. Fix that and create a wrapper method in the IMP class instead. The method name is still misleading, but at least it works consistently again, and the error message are correct now. Conflicts: imp/lib/Application.php imp/lib/Compose.php imp/lib/Mailbox.php imp/package.xml imp/docs/CHANGES \| 1 + imp/lib/Application.php \| 40 +----------------------------------- imp/lib/Compose.php \| 10 ++++---- imp/lib/IMP.php \| 51 +++++++++++++++++++++++++++++++++++++++++++++++ imp/lib/Mailbox.php \| 8 +++--- imp/package.xml \| 2 +- 6 files changed, 63 insertions(+), 49 deletions(-) http://git.horde.org/horde-git/-/commit/8810240d970ad7754fbb8e307b1bda37d335f471

07/09/2012 05:46:12 PM	Jan Schneider	Assigned to Jan Schneider State ⇒ Resolved Milestone ⇒ 5.0.23

07/09/2012 05:46:02 PM	Git Commit	Comment #3	Reply to this comment
Changes have been made in Git (master): commit a2db518ef0890dd2ae08ec41980bd6a751729b4d Author: Jan Schneider <jan@horde.org> Date: Mon Jul 9 19:43:16 2012 +0200 Fix permission checking in IMP (~~Bug #11253~~). IMP's hasPermission() worked completely different than any other application's hasPermission() method. Fix that and create a wrapper method in the IMP class instead. The method name is still misleading, but at least it works consistently again, and the error message are correct now. imp/docs/CHANGES \| 1 + imp/lib/Application.php \| 40 +----------------------------------- imp/lib/Compose.php \| 10 ++++---- imp/lib/IMP.php \| 51 +++++++++++++++++++++++++++++++++++++++++++++++ imp/lib/Mailbox.php \| 8 +++--- imp/package.xml \| 4 +- 6 files changed, 64 insertions(+), 50 deletions(-) http://git.horde.org/horde-git/-/commit/a2db518ef0890dd2ae08ec41980bd6a751729b4d

07/09/2012 03:23:37 PM	dirk (dot) dettmering (at) med (dot) uni-marburg (dot) de	Comment #2 New Attachment: max_recipients_error.png	Reply to this comment
The second error message is attached to this comment.

07/09/2012 03:21:39 PM	dirk (dot) dettmering (at) med (dot) uni-marburg (dot) de	Comment #1 Priority ⇒ 2. Medium Type ⇒ Bug Summary ⇒ error messages for max_timelimit, max_recipient don't show the real values Queue ⇒ IMP Milestone ⇒ Patch ⇒ No New Attachment: max_timelimit_error.png State ⇒ Unconfirmed	Reply to this comment
The error messages corresponding to max_timelimit and max_recipient always talk about 1 recipient per 24h or per mail (see attached screenshots: Empfänger is german for recipient) no matter what's the actual value for max_timelimit and max_recipient.