[PATCH] Allow empty foldernames

Sebastian Spaeth Sebastian at SSpaeth.de
Wed Aug 17 15:05:19 BST 2011


On Wed, 17 Aug 2011 14:27:26 +0200, Vladimir Marek <Vladimir.Marek at Oracle.COM> wrote:
> The patch looks good and does what it's supposed to do ...
> 
> > --- a/offlineimap/repository/LocalStatus.py
> > +++ b/offlineimap/repository/LocalStatus.py
> > @@ -50,10 +50,17 @@ class LocalStatusRepository(BaseRepository):
> >          return '.'
> >  
> >      def getfolderfilename(self, foldername):
> > -        """Return the full path of the status file"""
> > -        # replace with 'dot' if final path name is '.'
> > -        foldername = re.sub('(^|\/)\.$','\\1dot', foldername)
> > -        return os.path.join(self.directory, foldername)
> > +        """Return the full path of the status file
> > +
> > +        This mimics the path that Folder().getfolderbasename() would return"""
> > +        if not foldername:
> > +            basename == '.'
>                         ^^
> Apart from this bit :)

Doh, I had actually fixed this locally but not included it in the
patch. As you say it makes it work, I will send an updated patch with a
fix.

Thanks for testing.

Sebastian
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
URL: <http://alioth-lists.debian.net/pipermail/offlineimap-project/attachments/20110817/e48aa5fd/attachment-0001.sig>


More information about the OfflineIMAP-project mailing list