diff options
author | Christoph Wurst <christoph@winzerhof-wurst.at> | 2020-07-10 12:52:22 +0300 |
---|---|---|
committer | Christoph Wurst <christoph@winzerhof-wurst.at> | 2020-07-10 15:13:55 +0300 |
commit | 4772760b1046c8ad4eef0faa0e7720eadd42e32a (patch) | |
tree | c4b25d91426938b4bb1c2c274fc81855992416ee /lib/IMAP | |
parent | 02984c83d5d23b08aa9ca8111c727e943fbe67b7 (diff) |
Do not merge two threads that have the same subject
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
Diffstat (limited to 'lib/IMAP')
-rw-r--r-- | lib/IMAP/Threading/ThreadBuilder.php | 11 |
1 files changed, 5 insertions, 6 deletions
diff --git a/lib/IMAP/Threading/ThreadBuilder.php b/lib/IMAP/Threading/ThreadBuilder.php index 97e787adb..72283423f 100644 --- a/lib/IMAP/Threading/ThreadBuilder.php +++ b/lib/IMAP/Threading/ThreadBuilder.php @@ -211,13 +211,12 @@ class ThreadBuilder { && $container->hasMessage() && $container->getMessage()->hasReSubject()) { $container->setParent($subjectContainer); $subjectTable[$subject]; - } else { - $new = Container::empty(); - $container->setParent($new); - $subjectContainer->setParent($new); - $new->setParent($root); - $subjectTable[$subject] = $new; } + /* + * According to RFC5256 we would have to combine two messages with the same subject + * to a thread. But this will also group unrelated messages, so we deliberately omit + * this, just like most other clients do. + */ } } |