Can't select a folder when adding an mh-format / maildir / mbox account

Bug #65167 reported by Chris Rose
18
Affects Status Importance Assigned to Milestone
Evolution
Fix Released
Critical
evolution (Ubuntu)
Fix Released
High
Ubuntu Desktop Bugs

Bug Description

The location option is a file chooser not a folder chooser, preventing the selection of a valid folder.

There is a patch upstream, it would be good to get this in for edgy but I realise its a little late in the cycle.

Related branches

Revision history for this message
Chris Rose (chris-vault5) wrote :
Revision history for this message
Sebastien Bacher (seb128) wrote :

Thanks for your bug. I'm not sure the patch is correct, it changes the selector for the mbox mode too and in that case that's not sure you want to pick a folder

Changed in evolution:
assignee: nobody → desktop-bugs
importance: Undecided → High
status: Unconfirmed → Confirmed
Revision history for this message
Chris Rose (chris-vault5) wrote :

Ah. Well -- it fixes it for maildir at least.

Revision history for this message
Sebastien Bacher (seb128) wrote :

right, fixes an usecase and breaks an another one, not sure if that's a win

Revision history for this message
Chris Rose (chris-vault5) wrote :

I'll write a patch which for this, before I do I'd like to confirm that:

Maildir -> select directory
MH -> select directory
mbox -> select file?

Revision history for this message
Sebastien Bacher (seb128) wrote :

that looks a correct behaviour to me

Changed in evolution:
status: Unknown → Confirmed
Revision history for this message
Chris Rose (chris-vault5) wrote :

This isn't just an evo bug. The problem is that camel tells evo that it needs only a path, it dosen't specify that it specifically wants a file or a folder. I'm not sure if thats an oversight or if its deliberate.

Quickfix would be to replace the folder chooser with a text box, proper fix is to patch camel aswell.

Revision history for this message
Zeedruif (zeedruif-deactivatedaccount) wrote :

I'd say that it's not okay to just select a file when dealing with mbox, because the Evo text states that it "May also be used to read a tree of Elm, Pine or Mutt style folders", which also includes mboxes at least in the case of Mutt.

Changed in evolution:
status: Confirmed → Triaged
Changed in evolution:
status: Confirmed → In Progress
Revision history for this message
Sebastien Bacher (seb128) wrote :

The bug has been fixed upstream now

Changed in evolution:
status: Triaged → Fix Committed
Changed in evolution:
status: In Progress → Fix Released
Revision history for this message
Sebastien Bacher (seb128) wrote :

evolution (2.11.90-0ubuntu1) gutsy; urgency=low

  * New upstream version:
    New in 2.11.90:
    - A new command line option (--disable-preview) to disable loading
      crash-on-selected last mails, tasks, contacts
    Bug fixes:
    - #287394: (BNC) When exitting, close after the message was saved.
    - #313221: Improve parsing of VCARD file with multiple TYPE information
    - #322624: Don't add alarms for types with unknown/none notification method
               (LP: #87148)
    - #327977: Fix a crash when setting password in preferences window
    - #331729: Fixed a crash when using cursor keys after
        expunging selected mail
    - #335566: Fix a crash when scaling an image in preview pane
    - #351552: Fixed critical warning with e_passwords for calendars
        without 'auth-domain' property
    - #352346: Cannot create Maildir account (LP: #65167)
    - #377173: Show category icon correctly (LP: #74888)
    - #377309: "Synchronize now" should work even if periodic checking
        is not enabled
    - #383684: Do not load pending hooks if the corresponding
        plugin is disabled
    - #387312: Auto-collapse of expanded threads has been fixed
    - #401533: Text marked for localization in memos
    - #419690: Fixed a crash during startup after a unclean shutdown
    - #427469: When importing VCARD, if HOME or WORK isn't specified for
        EMAIL, add TYPE=OTHER
    - #428125: Added mnemonic for "Plain Text Mode" in preferences
    - #429234: Fixed a crash when opening an attachment
    - #444882: Fixed configure options beautification
    - #451599: Timing of setting the attachment is downloaded has to be after
               creating the attachment
    - #454253: Fixed a crash during alarm notify
    - #455799: Remove all .cvsignore and update svn:ignore porperty
               in all directories.
    - #458508: String issues resolved in backup-restore plugin
    - #458822: Description string fixed for mail-notification plugin
    - #460326: If the vcalendar isn't there don't crash but report invalid.
    - #460821: The second argument of g_utf8_strdown() expects
               the length of string in bytes or -1
    - #462007: Plugged memory leak.
    - #462010: Plugged memory leak.
    - #462138: Signature on top does not work if no signature is set as default
    - #462332: Avoid duplicate attachments when DnD image
    - #463061: Fixed a crash while launching preferences window
    - #463129: Fix glibc free crash when changing components
    - #463946: Calling "Add Column" more than once now works
    - #464312: Do not connect to server if the account is disabled.
    Other Contributors:
    - Fix for expanding newly created folders
    - Documentation updates
    - Exchange Calendar delegation bug fixes

 -- Sebastien Bacher <email address hidden> Mon, 13 Aug 2007 23:59:38 +0200

Changed in evolution:
status: Fix Committed → Fix Released
Changed in evolution:
importance: Unknown → Critical
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Related questions

Remote bug watches

Bug watches keep track of this bug in other bug trackers.