Re: [Evolution-hackers] importers again ...

[JP Rosevear <jpr ximian com>]

On Tue, 2004-02-10 at 00:03, Not Zed wrote:
> Hmm, so it seems i've hit a problem getting mail importers going:
> 
> 82,84c84
> <               boolean loadFile (in string filename,
> <                                 in string folderpath,
> <                                 in string foldertype);
> ---
> >               boolean loadFile (in string filename);
> 
> For mail importers:
>  - you really need to be able to specify the destination folder, you
> don't always want inbox

I have some of this done in my tree - basically there is a
"createControl" method now for the mailer to pass back a control for
display in the gui (the addressbook and calendar code do this now).  You
pass the callback function to evolution_importer_new.

>  - they're used by the intelligent importers to import mails into
> specific folders

Yes, this is the annoying bit.  Either the code needs to become a bit
more modular (ie move all the hi level import mbox functions to one
spot) or the importers should collapse into a single file (like the
calendar intelligent importer) since they are all doing basically the
same thing.

> Could we add back a destination folder thing, and make "" mean 'the
> default'?

The problem is how would it ever be anything other than the default? 
The shell has no concept of folders now, hence the createControl method.

-JP
-- 
JP Rosevear <jpr ximian com>
Ximian, Inc.