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 |
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
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 #12058imp/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
Assigned to Michael Slusarz
State ⇒ Resolved
Version ⇒ 6.0.4
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
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
Priority ⇒ 1. Low
Type ⇒ Bug
Summary ⇒ Deleting multiple attachments successively causes an error in Dynamic view
Queue ⇒ IMP
Milestone ⇒
Patch ⇒ No
State ⇒ Unconfirmed
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.