Opened 6 years ago

Closed 6 years ago

#1484309 closed Feature Patches (fixed)

[PATCH]: roundcube badly handles case when subscription exists but folder doesn't physically exists

Reported by: przemyslaw.plewa@… Owned by:
Priority: 5 Milestone: 0.1-rc1
Component: Core functionality Version: git-master
Severity: normal Keywords:
Cc: arekm@…

Description

When we have subscription for a system default folder but the folder doesn't physically exists then roundcube is foolded and doesn't autocreate missing folder (yes, you can have subscription for not existing folder left; it's verified on dovecot imap server).

Proposed patch checks if the folder for which we have subscription exists and if it doesn't then we create it.

--- program/include/rcube_imap.inc (wersja 528)
+++ program/include/rcube_imap.inc (kopia robocza)
@@ -1722,6 +1722,11 @@

else

$this->subscribe($folder);

}

+ else
+ {
+ if (!in_array_nocase($abs_name, $a_folders))
+ $this->create_mailbox($folder, False);
+ }

}

}

Change History (4)

comment:1 Changed 6 years ago by pepe

  • Cc arekm@… added

comment:2 Changed 6 years ago by pepe

  • Component changed from Client to Core functionality
  • Version changed from 0.1-beta to svn-trunk

comment:3 Changed 6 years ago by mtu

  • Milestone set to 0.1-rc1
  • Type changed from Bugs to Patches

comment:4 Changed 6 years ago by robin

  • Resolution set to fixed
  • Status changed from new to closed

Thanks, fixed in [0f23198e].

Note: See TracTickets for help on using tickets.