6.0.0-beta1
7/14/25

[#12058] Deleting multiple attachments successively causes an error in Dynamic view
Summary Deleting multiple attachments successively causes an error in Dynamic view
Queue IMP
Queue Version 6.0.4
Type Bug
State Resolved
Priority 1. Low
Owners slusarz (at) horde (dot) org
Requester aarno.sandvik (at) helsinki (dot) fi
Created 02/19/2013 (4528 days ago)
Due
Updated 04/08/2013 (4480 days ago)
Assigned
Resolved 02/19/2013 (4528 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch No

History
04/08/2013 10:06:38 AM Git Commit Comment #6 Reply to this comment
Changes have been made in Git (master):

commit b48ace5042184452b0f1e1d0f3496351e478e92e
Author: Michael M Slusarz <slusarz@horde.org>
Date:   Tue Feb 19 18:33:09 2013 -0700

     Need to do even more housekeeping in IMP 6.1 when stripping a 
part (Bug #12058)

  imp/lib/Ajax/Application/Handler/Dynamic.php |   16 +++++++++-----
  imp/lib/Basic/Message.php                    |   13 +++++++----
  imp/lib/Dynamic/Message.php                  |    8 ++++--
  imp/lib/Indices/Mailbox.php                  |   29 ++++++++++++++++++-------
  imp/lib/Message.php                          |    5 +++-
  5 files changed, 48 insertions(+), 23 deletions(-)

http://git.horde.org/horde-git/-/commit/b48ace5042184452b0f1e1d0f3496351e478e92e
02/20/2013 01:17:16 AM Git Commit Comment #5 Reply to this comment
Changes have been made in Git (master):

commit 325ad376a3b18b210f69652e2a67a65574188437
Author: Michael M Slusarz <slusarz@horde.org>
Date:   Tue Feb 19 18:14:10 2013 -0700

     Move selfUrl w/ params generation to Horde_Core

     Since we already need Horde_Core 2.2.0+ to fix the bug, might as well
     move this code over to Horde_Core now, since it is globally useful.

     See Bug #12058

  imp/attachment.php            |    2 +-
  imp/lib/Contents.php          |    2 +-
  imp/lib/IMP.php               |   22 ----------------------
  imp/lib/Mime/Viewer/Pgp.php   |    2 +-
  imp/lib/Mime/Viewer/Smime.php |    2 +-
  imp/message.php               |    4 ++--
  imp/package.xml               |    2 +-
  imp/thread.php                |    4 ++--
  8 files changed, 9 insertions(+), 31 deletions(-)

http://git.horde.org/horde-git/-/commit/325ad376a3b18b210f69652e2a67a65574188437
02/19/2013 11:56:27 PM Michael Slusarz Comment #4
Assigned to Michael Slusarz
State ⇒ Resolved
Version ⇒ 6.0.4
Reply to this comment
IMP 6.0.5 (requires a version of Horde_Core > 2.2.0 also).
02/19/2013 11:55:48 PM Git Commit Comment #3 Reply to this comment
Changes have been made in Git (master):

commit 9b7f13b875ec0c65a7b3ebc5a5242fbc35d7f957
Author: Michael M Slusarz <slusarz@horde.org>
Date:   Tue Feb 19 16:55:05 2013 -0700

     [mms] Correctly generate self URLs if a URL parameter is altered 
by the current message action (Bug #12058).

  imp/attachment.php   |    3 +--
  imp/docs/CHANGES     |    2 ++
  imp/dynamic.php      |    2 +-
  imp/lib/Contents.php |   17 +++++++++++++++--
  imp/lib/IMP.php      |   29 +++++++++++++++--------------
  imp/lib/Message.php  |   14 +++++++++-----
  imp/message.php      |    8 +++++---
  imp/minimal.php      |    2 +-
  imp/package.xml      |    2 ++
  imp/thread.php       |    4 ++--
  10 files changed, 53 insertions(+), 30 deletions(-)

http://git.horde.org/horde-git/-/commit/9b7f13b875ec0c65a7b3ebc5a5242fbc35d7f957
02/19/2013 11:55:42 PM Git Commit Comment #2 Reply to this comment
Changes have been made in Git (master):

commit d0b73b92b8e44c662e4e2a318336a26f4c2b3789
Author: Michael M Slusarz <slusarz@horde.org>
Date:   Tue Feb 19 16:54:21 2013 -0700

     [mms] Fix order of adding URL parameters in HordeCore.addURLParam 
(Bug #12058).

  framework/Core/js/hordecore.js |   15 ++++++++-------
  framework/Core/package.xml     |    2 ++
  2 files changed, 10 insertions(+), 7 deletions(-)

http://git.horde.org/horde-git/-/commit/d0b73b92b8e44c662e4e2a318336a26f4c2b3789
02/19/2013 11:27:12 AM aarno (dot) sandvik (at) helsinki (dot) fi Comment #1
Priority ⇒ 1. Low
Type ⇒ Bug
Summary ⇒ Deleting multiple attachments successively causes an error in Dynamic view
Queue ⇒ IMP
Milestone ⇒
Patch ⇒ No
State ⇒ Unconfirmed
Reply to this comment
When a user opens a message in a new window in Dynamic view and 
deletes more than one attachment it causes an error:

Fatal error: Call to a member function getHeaderText() on a non-object 
in /usr/share/pear/Horde/Imap/Client/Socket/Catenate.php on line 126

Steps to reproduce. Open a message with multiple attachments in a new 
window -> View all parts -> Delete first attachment -> View all parts 
-> Delete second attachment -> Error

Preview doesn't seem to be affected though.





Saved Queries