Summary: | Create Maildir ressource on valid maildir could result in not showing mails | ||
---|---|---|---|
Product: | [Frameworks and Libraries] Akonadi | Reporter: | Christoph Thielecke <crissi99> |
Component: | Maildir Resource | Assignee: | kdepim bugs <kdepim-bugs> |
Status: | REPORTED --- | ||
Severity: | normal | ||
Priority: | NOR | ||
Version: | unspecified | ||
Target Milestone: | --- | ||
Platform: | Ubuntu | ||
OS: | Linux | ||
Latest Commit: | Version Fixed In: | ||
Sentry Crash Report: |
Description
Christoph Thielecke
2019-12-16 13:59:14 UTC
Git commit 4ab1f2068b53d3fabe3cf35c087f28bf10bca672 by Igor Poboiko. Committed on 03/05/2020 at 12:18. Pushed by poboiko into branch 'release/20.04'. [resources/maildir] Reload configuraton on configuration change Summary: When user adds new maildir, a new resource gets created, with the default directory `~/.local/share/local-mail`. Since it's a new resource, with no proper configuration, an `attemptConfigRestoring()` is called, which changes it to `~/.local/share/akonadi_maildir_resource#`. It's stored inside `mSettings->path()`. Then a dialog appears, where user can choose prefered directory. It gets written to the config file; then `configurationChanged` gets called. We call `mSettings->save()`, which overwrites the path provided by user with the default one (`~/.local/share/akonadi_maildir_resource#`), making it impossible to create a new maildir anywhere else. Just use `load()` instead, it makes more sense when configuration was changed. Related: bug 416287, bug 415922 Test Plan: 1) Create a new maildir resource pointing to `/tmp/dummy`. 2) The resource gets created, with the name `dummy`. `/tmp/dummy` gets created. 2) Drop some mails into it via KMail. Mails appear inside `/tmp/dummy` Reviewers: dvratil, mlaurent Reviewed By: dvratil Subscribers: wbauer, kde-pim Tags: #kde_pim Differential Revision: https://phabricator.kde.org/D27905 M +2 -1 resources/maildir/maildirresource.cpp https://commits.kde.org/kdepim-runtime/4ab1f2068b53d3fabe3cf35c087f28bf10bca672 |