Opened 5 years ago

Closed 5 years ago

#1484681 closed Bugs (fixed)

Creating a new folder w/a comma in its name confuses RC

Reported by: nickveys Owned by: robin
Priority: 5 Milestone: 0.1.1
Component: Client Scripts Version: 0.1-rc2
Severity: minor Keywords: folder create comma name
Cc:

Description

Roundcube 0.1RC2, to reproduce:

  1. Personal Settings -> Folders
  2. Create a new folder named "Comma, Folder" (w/o quotes)
    • It will appear to have created a folder correctly
  3. Return to Email screen
    • You will have folders "Comma" and " Folder".

Looking @ the IMAP store, I have a .Comma, Folder/ directory, so it appears to have created correctly, but accessing it is another story.

Attachments (1)

#1484681.200711301604.patch​ (4.1 KB) - added by robin 5 years ago.
Always use array when subscribing folders.

Download all attachments as: .zip

Change History (6)

comment:1 Changed 5 years ago by nickveys

Poking around, appears to be broken down in the IlohaMail IMAP library's iil_C_ListSubscribed().

comment:2 Changed 5 years ago by nickveys

Nope, happens before that... Seems to be at creation time, as soon as the page refreshes with the new folder there is a correctly created folder on the server and a subscription on the client to 2 bad folders.

Changed 5 years ago by robin

Always use array when subscribing folders.

comment:3 Changed 5 years ago by robin

  • Owner set to robin

Try the patch in file #1484681.200711301604.patch.

comment:4 Changed 5 years ago by seansan

  • Milestone set to 0.1.1

Review in 0.1.1

comment:5 Changed 5 years ago by alec

  • Resolution set to fixed
  • Status changed from new to closed

Fixed in [c5097c63] (with little fix for that patch)

Note: See TracTickets for help on using tickets.