6.0.0-beta1
7/8/25

[#11382] folder listing fails due to empty prefs-value for spam_folder
Summary folder listing fails due to empty prefs-value for spam_folder
Queue IMP
Queue Version 5.0.23
Type Bug
State Resolved
Priority 3. High
Owners slusarz (at) horde (dot) org
Requester piper (at) hrz (dot) uni-marburg (dot) de
Created 08/29/2012 (4696 days ago)
Due
Updated 08/29/2012 (4696 days ago)
Assigned
Resolved 08/29/2012 (4696 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch No

History
08/29/2012 11:17:57 PM Michael Slusarz Assigned to Michael Slusarz
State ⇒ Resolved
 
08/29/2012 11:17:51 PM Git Commit Comment #2 Reply to this comment
Changes have been made in Git (FRAMEWORK_4):

commit e05a0e59510ea3f46b84c6053b8df109bc172abe
Author: Michael M Slusarz <slusarz@horde.org>
Date:   Wed Aug 29 17:17:31 2012 -0600

     Bug #11382: spam folder pref might be empty

  imp/mailbox.php                        |    8 ++++++--
  imp/message-dimp.php                   |   10 ++++++----
  imp/message-mimp.php                   |   10 ++++++----
  imp/message.php                        |   10 ++++++----
  imp/templates/dimp/javascript_defs.php |    3 ++-
  5 files changed, 26 insertions(+), 15 deletions(-)

http://git.horde.org/horde-git/-/commit/e05a0e59510ea3f46b84c6053b8df109bc172abe
08/29/2012 01:07:42 PM piper (at) hrz (dot) uni-marburg (dot) de Comment #1
Priority ⇒ 3. High
Patch ⇒ No
Milestone ⇒
Queue ⇒ IMP
Due ⇒ 08/29/2012
Summary ⇒ folder listing fails due to empty prefs-value for spam_folder
Type ⇒ Bug
State ⇒ Unconfirmed
Reply to this comment
In traditional view of imp (version 5.0.23 with horde 4.0.15) no 
folder content is displayed if the user has stored an empty 
prefs-value for 'spam_folder'. Opening the folder-listing yields to 
this fatal PHP error:

Call to a member function equals() on a non-object in 
/var/www/horde/imp/mailbox.php on line 606

Line 606 in imp/mailbox.php reads:
  !IMP_Mailbox::getPref('spam_folder')->equals(IMP::$mailbox)

This expression is also used in some other places in imp, the error 
should therefore be not restricted to the folder listing.

Thanks for any solution,
Andreas

Saved Queries