6.0.0-beta1
7/5/25

[#11959] Stripping attachments in dynamic view causes an error
Summary Stripping attachments in dynamic view causes an error
Queue IMP
Queue Version 6.0.3
Type Bug
State Resolved
Priority 2. Medium
Owners jan (at) horde (dot) org
Requester aarno.sandvik (at) helsinki (dot) fi
Created 01/15/2013 (4554 days ago)
Due
Updated 01/15/2013 (4554 days ago)
Assigned
Resolved 01/15/2013 (4554 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch Yes

History
01/15/2013 05:18:57 PM Jan Schneider Assigned to Jan Schneider
State ⇒ Resolved
 
01/15/2013 05:13:53 PM Git Commit Comment #2 Reply to this comment
Changes have been made in Git (master):

commit 1fea596ab4a1b42e0b2cb9de15b9fe2c2ca264b7
Author: Jan Schneider <jan@horde.org>
Date:   Tue Jan 15 18:13:04 2013 +0100

     [jan] Fix stripping attachments in dynamic view 
(aarno.sandvik@helsinki.fi, Bug #11959).

  imp/docs/CHANGES            |    2 ++
  imp/lib/Dynamic/Message.php |    2 +-
  imp/package.xml             |    2 ++
  3 files changed, 5 insertions(+), 1 deletions(-)

http://git.horde.org/horde-git/-/commit/1fea596ab4a1b42e0b2cb9de15b9fe2c2ca264b7
01/15/2013 02:41:16 PM aarno (dot) sandvik (at) helsinki (dot) fi Comment #1
Priority ⇒ 2. Medium
New Attachment: hy_fix_broken_strip_attachment.patch Download
Patch ⇒ Yes
Milestone ⇒
Queue ⇒ IMP
Summary ⇒ Stripping attachments in dynamic view causes an error
Type ⇒ Bug
State ⇒ Unconfirmed
Reply to this comment
When a user tries to remove attachments from a message it causes an error:

PHP Fatal error:  Call to a member function push() on a non-object in 
/horde/imp/lib/Dynamic/Message.php on line 51

Line 51 contains the following code:

$notification->push(_("Attachment successfully stripped."), 'horde.success');

$notification hasn't been declared before it's called and that causes 
the error.

Saved Queries