Alexei V. Mokeev Reporter
	Description • 24 years ago

Steps to reproduce:
1.Create IMAP subfolder(For example subfoilder of INBOX)
2.Subcribe to it
3.Unsubscribe from it
4.Open "Subscribe" window and note, that you is still subscribed
on this folder.

Reproduced under Intel Solaris 2.7
Reprodusible: Always
Build: Mozilla/5.0 (X11; U; SunOS 5.7 i86pc; en-US; m18) Gecko/20010123 
Netscape6/6.0 
IMAP server: Cyrus IMAP4 v2.0.9

	Alexei V. Mokeev Reporter
	Comment 1 • 24 years ago

Add CC

	(not reading, please use seth@sspitzer.org instead)
	Comment 2 • 24 years ago

what type of imap server?

	(not reading, please use seth@sspitzer.org instead)
	Comment 3 • 24 years ago

IMAP server: Cyrus IMAP4 v2.0.9

I should read more carefully.

	(not reading, please use seth@sspitzer.org instead)
	Comment 4 • 24 years ago

is this on the Netscape OEM branch?

if so, try a trunk build.  a lot of subscribe improvements have happened since
that branch.

	Shirley Liu
	Comment 5 • 24 years ago

It works on Mozilla build 2001040604 for PC win2000.

	Shirley Liu
	Comment 6 • 24 years ago

Tested Netscape6.01A on Sparc Solaris 2.8 and Mozilla build 2001040904 on PC
win2000.  I tried to create IMAP subfolder failed. First of all, I could not
create any subfolder under INBOX, the INBOX has been greyed out. The second
test was creating any subfolder under IMAP (except INBOX) failed with error
message "The current command did not succeed. The mail server responded:
CREATE failed: Can't create mailbox node /home/sl116442/Trash: File exists".
But, I tried to have subfolder name for Trash is "Trash1" which was a different
name. The IMAP server is single.eng.sun.com.

	Alexei V. Mokeev Reporter
	Comment 7 • 24 years ago

Not all IMAP servers allow creation of subfolders.
You may use Cyrus IMAP server for test this operation.
This bug is still reproduced for me with Cysrus IMAP v2.0.9
and Mozilla's build 2001040805 under Linux. 

	Raghu Sakleshpur
	Comment 8 • 24 years ago

Marking the bug OS category to Linux for now based on Alexei's comments.

	scottputterman
	Comment 9 • 23 years ago

adding nsenterprise-

	scottputterman
	Comment 10 • 23 years ago

wrong bug.  adding nsenterprise+

	Karen Huang
	Comment 11 • 23 years ago

Yes. I can reproduce this problem on the 08-02-06-trunk build.

	Cavin Song Assignee
	Comment 12 • 23 years ago

Taking from sspitzer.

	Cavin Song Assignee
	Comment 14 • 23 years ago

Basically, in nsImapProtocol::Subscribe() and nsImapProtocol::Unsubscribe() we
need to restore the folder pathname back to original one (ie, the one sent by 
the server for the list/lsub response). For example, 'INBOX/subfld' should be 
changed/converted to 'INBOX.subfld' before sending it to the Cyrus server, 
otherwise the cmd will fail.

bienvenu and sspitzer can you r and sr it? Thanks.

	Cavin Song Assignee
	Comment 15 • 23 years ago

Forgot to cc bienvenu.

	David :Bienvenu
	Comment 16 • 23 years ago

Not sure about this - I'm thinking maybe the caller should not be passing in
folder names with the hierarchical delimiter, but let me think about it a little
bit.

	David :Bienvenu
	Comment 18 • 23 years ago

looks good, Cavin, thanks for working on this - r=bienvenu

	(not reading, please use seth@sspitzer.org instead)
	Comment 19 • 23 years ago

sr=sspitzer

	Cavin Song Assignee
	Comment 20 • 23 years ago

Fix checked in.

	Bradley Baetz (:bbaetz)
	Comment 21 • 23 years ago

This was checked in with ^Ms in it, which broke some of the ports. timeless
fixed it for you. Check that cygwin is configured correctly.

	Cavin Song Assignee
	Comment 22 • 23 years ago

Sorry about that. This is not the first time that I've checked in stuff and I've
not changed any settings on my development machine so I'm not sure what happened
to it this time.  However, I'll investigate the cause.

	Karen Huang
	Comment 23 • 23 years ago

Verified on 08-23-08-trunk build.
This is working now on the Cyrus server for unsubscribe functionality.
Can I mark as verified for this bug? But I am wondering to know what the impact 
from the previous comments?

	Cavin Song Assignee
	Comment 24 • 23 years ago

Yes Karen, you can mark it as VERIFIED. There was some problem with the code I
checked in but it was fixed by Bradley so it's not a problem anymore.

	Karen Huang
	Comment 25 • 23 years ago

Thanks. 
Good work, Cavin. Marking as verified then.