[Kde-pim] Re: Review Request: kmail: "Local folders" are renamed to "Kmail Folders" after migration

[Thomas McGuire]

> On April 12, 2011, 5:44 p.m., Thomas McGuire wrote:
> > Looks good to me.
> > Kevin, what do you think?
> 
> Kevin Krammer wrote:
>     I am afraid I don't get the problem.
>
> 
> Thomas McGuire wrote:
>     There are two problems in the bug report:
>     - Reporter is confused that the folder name is now "KMail Folders" instead of "Local Folders", like in KMail 1. I agree and think "Local Folders" is a better name.
>     - For the reporter, the top-level folder can't be renamed, probably because it is read-only or has the wrong ACLs. This is not covered by this review request, though.
> 
> Kevin Krammer wrote:
>     I used "KMail Folders" as the name for the migrated KMail mail folders to have a clearly distinguishable user visible name from the one the default maildir resource is using.
>     
>     Ideally the migration process removes that one when it is determined that the mixedmaildir resource can take over as default resource for special mail collections. This, however, can only be done if the "Local Folders" resource is completely empty, otherwise it stays the default special collection resource. 
>     
>     In the latter case the user would end up with two "accounts" named "Local Folders" which, IMHO, could be quite confusing.
>     
>     So IMHO doing a rename at migration time has to take into account the outcome of the special collection take over.
>     
>     As far as I can remember renaming the mixedmaildir resource's top level collection at any later point should be possible and the resource would change the resource name accordingly.
>

Ok, discarding the review request then.
BTW there is a bug that apparently sometimes the top-level folder can't be renamed because it is read-only.


- Thomas


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/101107/#review2594
-----------------------------------------------------------


On April 12, 2011, 5:11 p.m., Antonis Tsiapaliokas wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/101107/
> -----------------------------------------------------------
> 
> (Updated April 12, 2011, 5:11 p.m.)
> 
> 
> Review request for KDEPIM.
> 
> 
> Summary
> -------
> 
> Hello
> 
> After a migration the local folders are renamed to kmail folders. So with this patch the issue is solved. 
> 
> 
> This addresses bug 260572.
>     http://bugs.kde.org/show_bug.cgi?id=260572
> 
> 
> Diffs
> -----
> 
>   migration/kmail/kmailmigrator.cpp 7adeef8 
> 
> Diff: http://git.reviewboard.kde.org/r/101107/diff
> 
> 
> Testing
> -------
> 
> kdepim and kdepim-runtime compile without any issue. Also kmail works fine.
> 
> 
> Thanks,
> 
> Antonis
> 
>

_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/