https://bugs.kde.org/show_bug.cgi?id=364314

Jan Kundrát <j...@kde.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|---                         |FIXED
      Latest Commit|                            |http://commits.kde.org/troj
                   |                            |ita/1c1fa01e1de1d6be7e59bda
                   |                            |5d788ee78d90b068a

--- Comment #1 from Jan Kundrát <j...@kde.org> ---
Git commit 1c1fa01e1de1d6be7e59bda5d788ee78d90b068a by Jan Kundrát.
Committed on 15/06/2016 at 07:16.
Pushed by gerrit into branch 'master'.

IMAP: do not misinterpret pending grandchild mailboxes

Under certain circumstances where the code triggered several LIST
commands at once, the LIST interpretation was a bit too eager and simply
looked at all mailboxes whose name matched a correct prefix. This worked
as long as we had a guarantee that mailbox listing for "A" and "A.B" was
never active at any given time, but this guarantee doesn't really hold
anymore.

It could be argued that it's actually a bug to ask for parent's and
a child's children at the same time (because the child data will get
thrown away anyway due to our enforced re-creation of all these
TreeItemMailbox instances), but let's solve this in a pretty quick and
just-a-bit-dirty way. This patch simply ensures that we don't assume
that these grand-children are actually children of the current parent.
Change-Id: I37570a6a5eab3eb15dcdbfa29922cb1b8dd79a7a

M  +9    -2    src/Imap/Model/Model.cpp
M  +66   -0    tests/Imap/test_Imap_Tasks_ListChildMailboxes.cpp
M  +2    -0    tests/Imap/test_Imap_Tasks_ListChildMailboxes.h

http://commits.kde.org/trojita/1c1fa01e1de1d6be7e59bda5d788ee78d90b068a

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to