Renaming folder to same name but different case not allowed

[Murat Gunes]

Binary package hint: mozilla-thunderbird

In 1.5.0.10, renaming the folder "foo" to "foO", "fOo" or "Foo" isn't allowed; the OK button isn't clickable upon typing the new name. However, this can be worked around by renaming the folder "foo" to "bar" first, and then renaming "bar" to "foO".

[Sspitzer (sspitzer)]

myk, this was a local folder, not an imap folder, right?

my guess is we do that because on some operating systems (win32), Foo is the
same file as foo.

we should check what 4.x does (on win32 and linux)

[Naving (naving)]

Seth, this is local folders. As in 4x we do not allow local folders with
different case but same name.

[Myk (myk)]

Yes, this is with local folders. I think the purpose of this bug has been
misconstrued. I'm not saying it should be possible to have two folders with the
same name but different case. I'm saying it should be possible to rename a
folder to its own name with different case.

In other words, if I have a folder called "mozilla", I should be able to rename
that folder to "Mozilla". I shouldn't be told there is already a folder with
that name when the only folder with that name is the one I am renaming.

[Naving (naving)]

4x also does the same thing. marking invalid again.

[Håkan W (hwaara-gmail-deactivatedaccount)]

No, this is not invalid. As myk says, of course the user should be able to
change case on the same folder ("Mozilla" -> "mozilla")!

If it doesn't work in 4xp, it's a bug.

[Sspitzer (sspitzer)]

I'm sure there's a good reason for why 4.x did what it did.

on win32, the OS will not support this:

C:\>touch foo
C:\>mv foo FOO
mv: `foo' and `FOO' are the same file

local folders are just files, so the rename will fail.

4.x linux also doesn't allow you to rename folders to the same name.

the code (in 4.x and in mozilla) is all XP, we didn't do XP_UNIX for that rename
code.

for linux, the underlying os can handle "mv foo FOO", but there might be some
mailnews backend code that would break if we allowed it. (I haven't researched.)

it's a valid bug, since there are ways we *could* deal with this (with folder
pretty names), but marking wontfix.

workaround for myk: exit mozilla, rename that file on disk. but be careful not
to break any existing prefs or filters that point to that folder.

[Myk (myk)]

I was able to work around the problem (modulo bug 64463) by renaming the folder
to a different name ("mozilla" -> "foo") and then renaming it back to the name I
wanted with the correct case ("foo" -> "Mozilla"). The same technique could be
employed in the code to fix this problem.

[Håkan W (hwaara-gmail-deactivatedaccount)]

Other major email apps supports renaming of this. I suggest we reopen this and
enforce myk's suggestion. Seth, Navin?

[Sheelar (sheelar)]

Seth, Navin:

any comments? Is this still won't fix or do you want me to reopen?

[Sspitzer (sspitzer)]

re-opening, myk's idea sounds good. we'll have to add extra code to check fix
new name is the same as the old name, excluding case (using strcasecmp), and if
so, do abc -> temp -> Abc.

[Tom-sneak-dresden (tom-sneak-dresden)]

*** Bug 126992 has been marked as a duplicate of this bug. ***

[Jonathan Stewart (reaper-fudo)]

I just found this bug myself today, when trying to rename a folder.
(Win 2000, 0.9.9)

I think this bug is not so much about functionality as it is about polish. Can
you not imagine lesser computer people being amazed and confused about this
message that it couldn't rename the folder even though they chaged the
capitalization? I can (but then i do tech support, so... :)

So anyways, i think this is the sort of bug that separated the amateur projects
from the pro ones. As a test, i created a folder called "mozilla" in Outlook
Express, and renamed it to "Mozilla" without any difficulty at all. Hopefully a
keen developer can polish up this part of the code.

[Sheelar (sheelar)]

*** Bug 145653 has been marked as a duplicate of this bug. ***

[Drew-devereux (drew-devereux)]

Folks, I think there are two separate but similar bugs here:

1. When renaming folders, the new name should not be duplicate-checked against
the original folder name. Duplicate checking should be against all folders
except the one being renamed, since the name of the folder being renamed will
disappear in the process of renaming anyhow.

Note that this bug is independent of case, as can be seen when you try to
execute a rename that doesn't change anything, neither the name nor the case.
Just click on "rename folder", and then hit ENTER. This rename, although silly,
would not cause a problem since it doesn't change the state of the folders in
any way. So there should not be an error message. But instead the "A folder of
that name already exists" alert is thrown.

Fixing this would fix the problem described in comment 3, without having to
introduce any case-specific stuff.

2. The other issue is, should it be permissable to have two folders whose names
differ only by case. If so, then the rename check should be case-dependent, and
this is a separate bug. If not, then the case-independence of the renaming
check is not a problem, and only problem 1. exists. By all means, discuss the
issue, but I think you can fix and close this bug quite easily by fixing issue 1.

[BrianLane (bcl)]

I too agree that this is a bug. It doesn't matter what the underlying problem
is, the user want to rename their folder to change the case, they ought to be
able to do so without the hassle of naming it something different first. To
argue otherwise shows that you have no understand of user interfaces and user
interaction and should be delegated to writing behind the scenes code that never
sees the light of a terminal screen.

I can't believe this thing has been in here since July 2001!

bcl

[Hskupin (hskupin)]

I'm also nerved to do the extra renaming by changing the case of letters.
Because i see that Seth said that it doesn't work with windows i have to add
following. I tried the steps within the command shell:

c:\>mkdir mozilla
c:\>rename mozilla Mozilla

And it works perfectly.
Why should the same operations not work within Mozilla?

[Jo-hermans (jo-hermans)]

*** Bug 144929 has been marked as a duplicate of this bug. ***

[Jo-hermans (jo-hermans)]

*** Bug 171628 has been marked as a duplicate of this bug. ***

[Jo-hermans (jo-hermans)]

*** Bug 209026 has been marked as a duplicate of this bug. ***

[Jo-hermans (jo-hermans)]

*** Bug 210815 has been marked as a duplicate of this bug. ***

[Malte-6thfloor (malte-6thfloor)]

*** Bug 219179 has been marked as a duplicate of this bug. ***

[Sgautherie-bz (sgautherie-bz)]

[Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.6) Gecko/20040113] (W98SE)

Confirming bug and workaroud.

Updating:
*(OS) Linux -> All, Linux + Windows.
*(S) Normal -> Minor, trivial workaround in comment 7.

[Jigho-caramail (jigho-caramail)]

Same bug in Thunderbird (at least 0.5)... don't know if I have to duplicate the
bug for Thunderbird ?

How much time before a 3 year old bug is being fixed ?
(even if not a vital one, I admit)

[Marcus Proest (marcus-proest)]

Confirming for Mozilla Thunderbird 0.5 (20040207)
It's not vital, but pretty anoying.

[Jomppa (jomppa)]

Confirming for Mozilla (2004050509) (WinXP) and mozilla-1.6 (SunOS).

[Asa (asa)]

*** Bug 236143 has been marked as a duplicate of this bug. ***

[Asa (asa)]

*** Bug 237563 has been marked as a duplicate of this bug. ***

[LohPhat (lohphat)]

As for the "workaround": exit mozilla, rename that file on disk. but be
careful not to break any existing prefs or filters that point to that folder.

Whomever suggested this has never worked at a helpdesk serving non-techie types
who don't give a rat's a** what the underlying technical issues are. The
real-world non-techie user sees a silly, petty restriction -- and as was also
said "It works in Outlook".

If you have to first rename the folder twice, then so be it:
mozilla -> $$temp$$ -> Mozilla

[Owen-interdim (owen-interdim)]

Could someone please tell me why fixing this bug is considered so difficult? It
seems like anyone set up to edit code in mozilla should be able to fix this
faster than they could edit the bug report. Perhaps I'm missing something.

I'm not currently setup to contribute, and I can't commit the time to do so now,
but this seems like it should literally be a 5-15 minute fix...

Any contributor should feel free to send me the relevant code sections, I'll fix
it, and then send it back to them to integrate and they can take the credit!

[Spam-minneboken (spam-minneboken)]

*** Bug 251156 has been marked as a duplicate of this bug. ***

[Jo-hermans (jo-hermans)]

*** Bug 241086 has been marked as a duplicate of this bug. ***

[Jo-hermans (jo-hermans)]

*** Bug 254416 has been marked as a duplicate of this bug. ***

[Francis-uy (francis-uy)]

*** Bug 259601 has been marked as a duplicate of this bug. ***

[Francis-uy (francis-uy)]

3 years? Wow. If this hasn't been fixed by the time the election is over, I'll
dig out my old programming books and take it myself.

And someone please remind me to do so.

[Jerfa (jerfa)]

*** Bug 264865 has been marked as a duplicate of this bug. ***

[Moz-supportware (moz-supportware)]

Regarding comment 29, it *would* be a 5-15 minute fix if not for bug 65303 which
is non-trivial to fix...

Bug 65303 is sinister in its effect(s) as some effects happen in the current
session and some persist across sessions and continue to cause erratic operation
weeks later...

[M-wada (m-wada)]

Bug 104525 is Thunderbird bug for this problem.

[Matti-mversen (matti-mversen)]

*** Bug 292648 has been marked as a duplicate of this bug. ***

[Wurstbrot (wurstbrot)]

Even when you create a new folder with name "mozilla" delete it, so that the folder can still be found in trash, create a new folder named "mozilla" and try to delete it too, you will also get the message "A folder
with that name already exists.".

[Mkmelin+mozilla (mkmelin+mozilla)]

That's bug bug 66763.

[Andreas-kleinert (andreas-kleinert)]

Just wanted to confirm the case-sensitivity (i.e. case-insensitivity) renaming problem problem also for Thunderbird 1.5.0.7 (Windows).

For example: you'd like to track some SuSE-related mailing list and created some filter for it, redirecting mails to the "SuSe-ML" folder. Then you notice the spelling mistake and want to rename it to "SuSE-ML" - does not work, because the folder 'already does exist...'

As mentioned below, the same error of course also occurs, if you just leave the name as it is, and press ENTER in the rename dialog.

[Simon-reinhardt-gmx (simon-reinhardt-gmx)]

Same in Thunderbird 1.5.0.9 on Windows (surprise, surprise ;) ).

[Murat Gunes (mgunes)]

Binary package hint: mozilla-thunderbird

In 1.5.0.10, renaming the folder "foo" to "foO", "fOo" or "Foo" isn't allowed; the OK button isn't clickable upon typing the new name. However, this can be worked around by renaming the folder "foo" to "bar" first, and then renaming "bar" to "foO".

[Hilario J. Montoliu (hjmf) (hmontoliu)]

Assigned to Mozilla Team

[Daan-timmer (daan-timmer)]

Same problem still excists in Thunderbird 1.5.0.10 on Windows XP SP2.

I am also wondering why, (it is said anywhere above) they are using a "mv foo Foo" command and not a rename command.

If I press F2 in explorer on any file name, and change foo to Foo then it works, so why should it not work for Thunderbird?

[Mkmelin+mozilla (mkmelin+mozilla)]

It would not be a problem on XP, only on Win9X. And I suppose to some degree for fat partitions on linux (for multi-boot sharing mail folders). mv is the unix rename command...

[M-wada (m-wada)]

*** Bug 380548 has been marked as a duplicate of this bug. ***

[M-wada (m-wada)]

*** Bug 384788 has been marked as a duplicate of this bug. ***

[Cooler (cooler)]

The problem still exists in Thunderbird 2.0.0.6 (using Win XP Pro).

[Mkmelin+mozilla (mkmelin+mozilla)]

*** Bug 397976 has been marked as a duplicate of this bug. ***

[Forage (forage)]

6 years and still present. What are the objections against fixing this bug?

[Mnyromyr (mnyromyr)]

> 6 years and still present. What are the objections against fixing this bug?

You didn't attach your patch yet.

[Forage (forage)]

(In reply to comment #50)
> > 6 years and still present. What are the objections against fixing this bug?
>
> You didn't attach your patch yet.
>

Hehe, I knew I forgot something ;-)

[Philringnalda (philringnalda)]

*** Bug 419802 has been marked as a duplicate of this bug. ***

[Philringnalda (philringnalda)]

*** Bug 419987 has been marked as a duplicate of this bug. ***

[Philringnalda (philringnalda)]

*** Bug 420321 has been marked as a duplicate of this bug. ***

[Philringnalda (philringnalda)]

*** Bug 425994 has been marked as a duplicate of this bug. ***

[Dayslipper (dayslipper)]

Almost 7 years.

[Joshua Cranmer (jcranmer)]

(In reply to comment #56)
> Almost 7 years.
>

Comments like these are not helpful.

If you want it that badly, you could just fix it up yourself and post a patch. The fact is that there are at least two known workarounds, so this isn't a terribly big deal.

In addition, there are also several more pressing issues currently going on with mailnews core that are competing for time.

[Dayslipper (dayslipper)]

I hope that it'll be OK in TB 3.0.

[Mkmelin+mozilla (mkmelin+mozilla)]

*** Bug 447746 has been marked as a duplicate of this bug. ***

[Matt Joyce (matt-joyce)]

Same for v2.0.0.16 (20080724)

[Philringnalda (philringnalda)]

*** Bug 472940 has been marked as a duplicate of this bug. ***

[Ludovic-mozillamessaging (ludovic-mozillamessaging)]

*** Bug 479760 has been marked as a duplicate of this bug. ***

[Craig-stephen (craig-stephen)]

I am seeing this - or maybe just something similar - in Thunderbird 3.0b2:

create folder "test"
create subfolder "Aaaa"
rename subfolder to "aaaa"

for a while, two subfolders show up - "Aaaa" and "aaaa".
Shortly thereafter, "Aaaa" disappears - so it looks like the operation worked!
But... a bit later... "aaaa" disappears, too.

So now my subfolder - and everything in it - are unfindable.

If I start TB 2.0.0.21, however, "aaaa" is present.

If I restart TB 3.0b2, "aaaa" is still missing.

This seems like a problem, since, unlike the scenarious described above,
there's apparently no workaround from within TB 3.0b2.

[M-wada (m-wada)]

(In reply to comment #62)

Local mail folder? IMAP folder? Tb on Linux? on Mac OS X? or on Win?
(file system of case-sensitive filename or of case-insensitive filename)
If IMAP, see Bug 484329.

[Craig-stephen (craig-stephen)]

Oops - thanks WADA.

IMAP folder, WinXP Pro SP2.

Thanks for the cross-reference to Bug 484329;
I will have a look.

Curiously, a short time after posting this,
the original folder *reappeared* - but my
test case, documented above, is still missing
in action. Perhaps this is some IMAP server
problem.

[M-wada (m-wada)]

(In reply to comment #64)
> Perhaps this is some IMAP server problem.

External phenomenon depends on case sensitivity of file name/IMAP folder name at both IMAP server and local PC. Bug 484329 is probably IMAP server on Unix(case sentisitve) & Tb on Linux case, and Tb mis-handles it then folder is hidden. If IMAP of IIS on MS Win, different phenomenon can occur. (example. IIS on MS Win retrurn OK to create "aaa", but returns folder name of "AAA".)
Is your IMAP server case sensitive on IMAP folder name and filename?

Anyway, this bug is for rename of local mail folder.
If your problem with IMAP is different from Bug 484329, get IMAP log, and open separate bug, please.

[C300331 (c300331)]

I can't believe I started this SEVEN years ago and the bug is still UNFIXED.
Someone's already commented Outlook doesn't have the problem yet you folks piddle around arguing about "work arounds" going outside the OS and renaming, etc. Do you really think the general user does that kind of stuff? All he/she sees is a program that hasn't quite got it!

[Wayne-lydecker (wayne-lydecker)]

Please don't fix this. I found the bug this morning, which quickly led me to this report. I haven't been so amused in a long time. This problem has been around since 2001 and it still is a bug! The amount of time and energy to fix the problem would have been a fraction of that used defending why it shouldn't be fixed. Until silly bugs like this are fixed, its hard taking Thunderbird seriously. Thanks for the laugh. I'll check in every now and then to keep track of the latest developments here.

[Gary-rumblingedge (gary-rumblingedge)]

*** Bug 497317 has been marked as a duplicate of this bug. ***

[Mkmelin+mozilla (mkmelin+mozilla)]

*** Bug 259280 has been marked as a duplicate of this bug. ***

[J-haus (j-haus)]

Hi!

I noticed now in TB 3 B3 the same 'bug':

http://img4.abload.de/img/imgdxtr.gif

So i worked it this way around:
Folder Name is 'Test 2' and TB dont let it me to rename it to 'TEST 2'. So i renamed the Folder to 'BlaBla'. Then i renamed 'BlaBla' to 'TEST 2' -> finish! ;-)

[Micah Gersten (micahg)]

Marking this Triaged as we have an upstream bug.

[Philringnalda (philringnalda)]

*** Bug 540154 has been marked as a duplicate of this bug. ***

[Jim (squibblyflabbetydoo)]

*** Bug 546994 has been marked as a duplicate of this bug. ***

[Dmose (dmose)]

I don't think this is so high-leverage that it qualifies as "wanted", but it's not "wontfix" either since we'd accept a patch if it were to appear, so just resetting the status to nothing.

[DGDanforth (danforth)]

This bug was first reported 9 years ago! 2001-07-24 15:21:46 PDT

Why hasn't it been fixed!!!!!!!!!

Mozilla you get a very, very, poor rating for this.

[C300331 (c300331)]

I've been asking that same question for those nine years!
Even the nasty comment, "You can do it in Outlook" doesn't get a response.
This looks very unprofessional for what s(c)ould be an excellent mail reader.

[Bugzilla-standard8 (bugzilla-standard8)]

*** Bug 578921 has been marked as a duplicate of this bug. ***

[Ludovic-mozillamessaging (ludovic-mozillamessaging)]

*** Bug 603267 has been marked as a duplicate of this bug. ***

[Fabio Napodano (napodano)]

fixing this is as easy as programmatically doing what the user currently does to rename from "folder" to "FOLDER":

- rename to "folder1" (or whatever makes the folder name unique"
- rename to "FOLDER"

[edam (edam)]

Happy 10-year anniversary! :o)

[Mkmelin+mozilla (mkmelin+mozilla)]

*** Bug 718940 has been marked as a duplicate of this bug. ***

[Acelists (acelists)]

*** Bug 589532 has been marked as a duplicate of this bug. ***

[Acelists (acelists)]

David is probably not working on this.

[Steve-si9yrl01qsu4 (steve-si9yrl01qsu4)]

Just wanted to point out that this bug is still valid in 12.0b1 - Just for grins and giggles I've downloaded the source.

- Windows 7 64-bit

[The-invincible (the-invincible)]

rename from "bug" to "Bug" is still not possible in thunderbird 11.0.

[Acelists (acelists)]

This seems very similar to bug 66763. I think a very similar approach and code could be used here.

[bowlingb (bowlingb)]

This appears to still be an issue in 13.0.1.

[Megachromulent (megachromulent)]

This is fixed on Linux Mint 13 with Thunderbird 13.0.1 using IMAP...

[Pshoplifter (pshoplifter)]

This bug has been reported 11 years ago (!), and is still there. It is almost funny...

[Stfkerman (stfkerman)]

I wish I could participate in your generosity of spirit. But it's clear that their priorities are to produce new poorly tested versions with even more blatant bugs that show up within 15 minutes of installation. They can't be bothered fixing the old bugs.

That's why I don't bother reporting bugs anymore. I have learned the bitter lesson that installing a later version can be a big mistake, with the new version, one that has been out for a while and ought to be an improvement, ends up being a total disaster, with things that worked for the last 10 years suddenly broken.

[Stfkerman (stfkerman)]

I wish I could participate in your generosity of spirit. But it's clear that their priorities are to produce new poorly tested versions with even more blatant bugs that show up within 15 minutes of installation. They can't be bothered fixing the old bugs.

That's why I don't bother reporting bugs anymore. I have learned the bitter lesson that installing a later version can be a big mistake, with the new version, one that has been out for a while and ought to be an improvement, ending up being a total disaster, with things that worked for the last 10 years suddenly broken.

[Strawpad4 (strawpad4)]

Oh come on. I know it's annoying that some bugs linger for such a long time, but on the other hand

a) this bug is an extremely minor nuisance with a trivial workaround
b) on the whole, Thunderbird is an excellent piece of software that I've enjoyed using for many years now.

I think it's somewhat disrespectful to the developers to go and post bitter and dismissive comments about Thunderbird because of minor glitches.

[Joshua Cranmer (jcranmer)]

(In reply to Megac from comment #86)
> This is fixed on Linux Mint 13 with Thunderbird 13.0.1 using IMAP...

This bug is only an issue on case-insensitive filesystems, so it pretty much affects only Windows.

To everyone else: it is known that this is an issue. Complaining about this bug will not increase its priority. In fact, it will tend to decrease the likelihood that developers will work on it. Please do not comment on this bug unless you are working on, or commenting on, a proposed solution to the problem. See <https://bugzilla.mozilla.org/page.cgi?id=etiquette.html> and please follow its rules.

[Stfkerman (stfkerman)]

Only affects Windows, huh... Are there statistics somewhere on how many Windows downloads have occurred over time VS other operating environments?

[jpfle (jpfle)]

(In reply to Joshua Cranmer [:jcranmer] from comment #91)
> This bug is only an issue on case-insensitive filesystems, so it pretty much
> affects only Windows.

It doesn't only affect Windows. I have this bug on GNU/Linux, and see the bug description: "Platform: All All".

[Acelists (acelists)]

And this bug is not dead, see comment 84.

[Acelists (acelists)]

Yes, this affects Linux too (with POP3 account).

[M04y68q005s-adam (m04y68q005s-adam)]

Created attachment 645623
Proposed patch

Happy 11th birthday! I have for you a present.

Feedback welcome. The bug has nothing to do with case-insensitive file systems (the underlying file renames are performed with rename(2) on Linux/OS X and MoveFileExW on Windows, which can handle case changes just fine). The bug is entirely in the way Thunderbird was handling names.

I've only tested this on OS X.

[Acelists (acelists)]

Oh, nice present :)
Does mailnews/local/src/nsMsgMaildirStore.cpp need a similar touch like nsMsgBrkMBoxStore.cpp ?

[DGDanforth (danforth)]

Ah, from a 'user' point of view presenting code is not a solution. This needs to be incorporated into the next Fire bird release.

[M04y68q005s-adam (m04y68q005s-adam)]

(In reply to :aceman from comment #97)
> Oh, nice present :)
> Does mailnews/local/src/nsMsgMaildirStore.cpp need a similar touch like
> nsMsgBrkMBoxStore.cpp ?

I don't know, I'm not familiar enough with the code base.

[Acelists (acelists)]

Neither am I:) I just think that the implementation of maildir-lite can have the same problem as the mbox implementation (that you changed).

I'll try the patch soon on Linux.

[Acelists (acelists)]

Comment on attachment 645623
Proposed patch

Great, works for me on mbox store on Linux.

[Acelists (acelists)]

Adam, welcome to bugzilla and we welcome your patches:)
Do you know whom to request review from?

[M04y68q005s-adam (m04y68q005s-adam)]

(In reply to :aceman from comment #102)
> Do you know whom to request review from?

No clue, any suggestions?

[Acelists (acelists)]

You can find reviewers here: https://wiki.mozilla.org/Modules . Just find the module the bug is in (as seen in the Product + Component fields). Of course there are some outdated data and people overlapping modules.

[Joshua Cranmer (jcranmer)]

The patch, at a minimum, needs a test. I don't know if a mozmill test or a xpcshell test would be better here.

[M04y68q005s-adam (m04y68q005s-adam)]

I'll see what I can do for a test, but since I'm doing this in my spare time I can't promise any expediency (not that anyone expects that here given the age of this bug...).

[Mozilla-ex (mozilla-ex)]

maildir might need the same kind of change, if it checks for folder existance in the same way before adding a sub-folder.

[Acelists (acelists)]

We need somebody to test the patch on Windows. On the other OSes it is fine to change the case of a file name (representing the folder) on the filesystem level. Changed case means different name. But what happens on Windows?

[M04y68q005s-adam (m04y68q005s-adam)]

FWIW the default file system option on Mac OS X (which I tested) is case-insensitive, but yes, someone should absolutely test on Windows.

[Acelists (acelists)]

Adam, can you check the maildir code in mailnews/local/src/nsMsgMaildirStore.cpp as David points out too? Otherwise the patch might not pass review.

[M04y68q005s-adam (m04y68q005s-adam)]

I can take a look when I have a chance; do you know how I'd test that? Is it used for IMAP/POP/something else?

[Mozilla-ex (mozilla-ex)]

(In reply to Adam Rosenfield from comment #111)
> I can take a look when I have a chance; do you know how I'd test that? Is
> it used for IMAP/POP/something else?

I'd suggest writing an xpcshell test for this general feature. That would make testing maildir support fairly straightforward.

[R-root-packet (r-root-packet)]

I can confirm the same issue on Thunderbird 15.0.1, Mozilla 5.0 on Debian 6.0.5. Thunderbird is installed from Mozilla and not the Debian repos.

[Kent-caspia (kent-caspia)]

Comment on attachment 645623
Proposed patch

Sorry this review took so long, between critical TB 17 bugs and build problems its been hard to fit this in.

Testing on Windows 7, this patch does not work. When I rename folder "test" to "Test", I end up with both "test" and "Test" in the folder pane. There seems to be 2 dbs open as well, as if I click on one folder, then the other, the db attempts to rebuild.

This really should have a unit test as well.

Also the original patch bit rotted, I'll upload by unbitrotted version.

[Kent-caspia (kent-caspia)]

Created attachment 670441
Unbitrotted patch

[Acelists (acelists)]

*** Bug 810740 has been marked as a duplicate of this bug. ***

[M-wada (m-wada)]

*** Bug 840288 has been marked as a duplicate of this bug. ***

[Acelists (acelists)]

*** Bug 859701 has been marked as a duplicate of this bug. ***

[Klonos (klonos)]

...still an issue in TB 26 :/

[Vseerror (vseerror)]

*** Bug 968646 has been marked as a duplicate of this bug. ***

[Cgiraud (cgiraud)]

Still an issue in TB 24.5.0, Linux x86_84.
Walk around: to rename "abc" to "Abc", first rename "abc" to "xxx" and then "xxx" to "Abc".

Folder to be renamed should be excluded from existing folder list when comparing with new name.

[Euryalus-0 (euryalus-0)]

*** Bug 958572 has been marked as a duplicate of this bug. ***

[Bugzillamozillaorg-serge-20140323 (bugzillamozillaorg-serge-20140323)]

*** Bug 270726 has been marked as a duplicate of this bug. ***

[Glenn-glazer (glenn-glazer)]

Thirteen YEARS and counting. I wish people would spend even half the effort they spend on marking duplicate bugs on actually fixing the bug.

[Vseerror (vseerror)]

updating status to reflect reality, so someone else feels free to take up the draft patch

[Bugzilla-tf (bugzilla-tf)]

*** Bug 1065260 has been marked as a duplicate of this bug. ***

[Lpam (lpam)]

this bug was reported 13 years ago. seems like this is mozillas duke nukem. just wanted to report, that today i was confrontated with that bug too.

[Larry Xiao (xiaodi)]

Same here, on thunderbird 31.3.0

(In reply to spam from comment #127)
> this bug was reported 13 years ago. seems like this is mozillas duke nukem.
> just wanted to report, that today i was confrontated with that bug too.

(In reply to Glenn Glazer from comment #124)
> Thirteen YEARS and counting. I wish people would spend even half the effort
> they spend on marking duplicate bugs on actually fixing the bug.

[Acelists (acelists)]

(In reply to Larry Xiao from comment #128)
> (In reply to Glenn Glazer from comment #124)
> > Thirteen YEARS and counting. I wish people would spend even half the effort
> > they spend on marking duplicate bugs on actually fixing the bug.
The problem is, those are not the same people...

[Glenn-glazer (glenn-glazer)]

(In reply to :aceman from comment #129)
> (In reply to Larry Xiao from comment #128)
> > (In reply to Glenn Glazer from comment #124)
> > > Thirteen YEARS and counting. I wish people would spend even half the effort
> > > they spend on marking duplicate bugs on actually fixing the bug.
> The problem is, those are not the same people...

So? The reply was to two similar comments from two different people, gathered together.

[Bugzilla2007 (bugzilla2007)]

Aceman, given the high popularity of this bug, can't we use the workaround as an interim fix when
renaming foo to Foo:

sourcefolder=foo
targetfolder=Foo
if (lcase(sourcefolder) == lcase(targetfolder)) {
- first rename foo to foo.tmp_random
- then rename foo.tmp_random to Foo }

Renaming folders doesn't happen every day, so taking a split second longer doesn't matter.
That way, it'll work for all types of folders/OSs/POP/Imap no matter what (except for those where the server doesn't allow renaming).
Then, we can spend the next decades looking for the real fix. Or maybe this is the real fix, because even some servers don't accept changing capitalization only, so we'd elegantly trick those.
The only scenario where this might fail are extra long nested folder names at the limit, where we might fail to create the tmp target folder if the tmp path gets too long or such.

[Acelists (acelists)]

I'd think that is would even be the final solution, not a workaround. We can never know what filesystem we are really on (even if we knew, we do not know all the limit or all exotic systems).

We must just try it. On any system, if we detect the new name only changes in case, do the renaming dance comment 131 and comment 7 proposed. On on the high level TB does not allow 2 folders to exist if they only differ in case (in CheckIfFolderExists). So this dance can be already done in nsLocalMailFolder.cpp, I don't think we need to code it in all the backends (mbox/maildir).

[UlfZibis (ulf-zibis)]

(In reply to Thomas D. (currently busy elsewhere; needinfo?me) from comment #131)
> The only scenario where this might fail are extra long nested folder names
> at the limit, where we might fail to create the tmp target folder if the tmp
> path gets too long or such.
To avoid this, you could just try to rename to fo1, fo2, ... foa, fob, ... until success. tmp may be already in use.

> because even some servers don't accept changing capitalization only, so we'd elegantly trick those.
+1

[Realgrouchy (realgrouchy)]

I've noticed a problematic behaviour as a consequence of this bug, which I've described in Bug 104525, but since this thread is related and more active, I thought I should mention it here too:

In TB 38.2.0 on OSX 10.10.5, when I try to rename a folder in a POP account, I do *not* get the error until I dismiss the rename dialog.

As a result, if I click "Rename" X times in the rename dialog (with no response from TB), it is only *after* I dismiss the rename dialog that I get a succession of X error messages.

For the purposes of changing the case, this problem will be redundant when this bug (Bug 92165) and/or Bug 104525 are resolved, but I've filed it as a separate Bug 1207212 since this error also happens in non-bug situations.

- RG>

[Alta88-t (alta88-t)]

*** Bug 1287718 has been marked as a duplicate of this bug. ***

[Vseerror (vseerror)]

*** Bug 1485853 has been marked as a duplicate of this bug. ***

[Sherman-m (sherman-m)]

Postbox would be willing to sponsor the development of this fix if anyone would like to work on this.

Requirements:

* Cannot be structured as an add-on (as those are no longer supported in Postbox)
* Cannot rely on changes or APIs made after 52.8 (as Postbox is currently based on this version)
* Must work on Windows 7, 8, and 10 and macOS 10.11 and higher.

If interested, let me know and we'll work out the details.

[Jorgk-bmo (jorgk-bmo)]

Created attachment 9013294
92165-v2.patch - rebased again

Carrying forward Kent's r-.

This actually works fine for a local folder, but it doesn't work for IMAP. If you rename folder "Tests" to "tests" you end up with two versions in the UI. In the subscribe dialogue you only get "tests" and there's also only one "tests.msf" file. After a restart, the "Tests" folder is gone. So the solution can't be far off. Of course it's missing the maildir part which shouldn't be hard to add.

Since this is one of the TB "papercuts" (https://wiki.mozilla.org/Thunderbird/Papercuts), maybe we can look at this bug one day soon now.

Since Ben is already working a bit on fixing some maildir issues and touching the relevant files, maybe he'd be a good candidate. Or maybe not since this would need Windows to test? Just take it as a suggestions since I can't allocate resources.

[Jorgk-bmo (jorgk-bmo)]

I did a bit of debugging. folderPane.js:2605, folder.rename(aName, msgWindow);, calls nsImapMailFolder::Rename() which calls nsImapService::RenameLeaf() and that opens this URL via GetImapConnectionAndLoadUrl():
  "imap://<email address hidden>@server:143/rename>.INBOX.TestMessages>.INBOX.testMessages"
Later we came into nsImapIncomingServer::OnlineFolderRename() and nsImapMailFolder::RenameClient() which does indeed call nsMsgDBFolder::PropagateDelete() as a comment in the patch suggests.

So that's all very well, but there appears to be some code missing to remove the no longer existing folder from the UI.

[Benc-q (benc-q)]

It does fit in quite nicely with what I've been looking at with the EmptyTrash() implementation, so I'm happy to look at it (and in fact, I just spent a couple of hours poking through the code from where Jorg left off - I'll write up my thoughts).

I can run a profile off a vfat filesystem (usb stick or loopback trickery) for case-insensitive filenames, I should be able to hit the same issues as on Windows.

There is an issue even on Linux: if I rename an imap folder "aaa" -> "Aaa", it looks fine in the GUI, but the old "aaa" directory will still be on the filesystem. So something isn't quite right there.

[Benc-q (benc-q)]

Here are my notes - mostly a braindump for my own reference:

Local Folders

nsMsgLocalMailFolder::Rename() performs these steps:

1) calls nsIMsgPluggableStore::RenameFolder() to rename the raw filesystem artifacts:
     * mbox/maildir
     * any .sbd dir
     * .msf file (actually, I'm not sure this should be done by the msgStore)
2) updates the new folder's pretty name
3) checks up on any filters which might be referencing the folder
4) calls RenameSubFolders() which recursively creates new child folders, checking for filter usage as it goes.
5) detaches the old folder from the parent (and calls progagateDelete() on it) and adds the new one.
6) tells the nsIMsgFolderNotificationService that the folder has been renamed.

IMAP folders

nsImapMailFolder::Rename() just tells the server to rename the folder.
Later on, they'll be a nsImapIncomingServer::OnlineFolderRename() coming in, where the local imap folder renaming is handled.
I haven't picked all the way through this yet, but quick observations are:
1) if it's a virtual folder, it just calls nsMsgDBFolder::Rename() and leaves it at that.
2) the pluggable message store isn't used.
3) it calls both nsImapMailFolder::RenameClient() and nsImapMailFolder::RenameLocal(). RenameLocal() seems to be primarily concerned with renaming the local filesystem artifacts, while RenameClient() handles the folder hierarchy and assort imap-specific stuff.

Quick thoughts:

- The IMAP path should use the pluggable mail store to handle the filesystem artifacts.
- there _should_ be a whole heap of common functionality that can be shared between local and imap (including the nsMsgDBFolder::Rename() that imap virtual folders use).
- There's a whole load of folder deleting and rebuilding going on, which seems unnecessary and overly-complicated.
It seems like it _should_ be possible to just rename folders in place, rather than just ditching them completely and rebuilding them... (ie filesystem rename, then patch up affected URIs etc)
- that said, I'm sure this bug can be addressed with a quick-n-nasty patch for now.

[Jorgk-bmo (jorgk-bmo)]

(In reply to Ben Campbell from comment #140)
> It does fit in quite nicely with what I've been looking at with the
> EmptyTrash() implementation, so I'm happy to look at it ...
Thanks. I was in to minds and almost about to cancel the NI since this is going ((too) far) into IMAP. But then on the other hand, we need to increase our backend knowledge of IMAP.

[Bugzilla2007 (bugzilla2007)]

*** Bug 1708164 has been marked as a duplicate of this bug. ***

[Vseerror (vseerror)]

*** Bug 1739969 has been marked as a duplicate of this bug. ***

[Adrien-rybarczyk (adrien-rybarczyk)]

Perhaps the title should be changed, as it also affects imap folders.

[Mkmelin+mozilla (mkmelin+mozilla)]

*** Bug 1746346 has been marked as a duplicate of this bug. ***