Merge lp:~openerp-dev/openobject-addons/6.1-opw-577343-ado into lp:openobject-addons/6.1

[Amit Dodiya (OpenERP)]

Hello,

[FIX] when selecting external storage media for a document and when we access it thruough ftp it gives warning that Path is not directory

Steps:
1). Install document_ftp, configure storage media for external file storage
2). Create one directory and set Storage = External File Storage
3). Create one document and set directory = newly created directory for external file storage, save the document.
Now try to access the above created document you will face IO warning message "530:Path is not directory". It should open the newly created file.

Regards,
Amit Dodiya

[Naresh(OpenERP) (nch-openerp)]

review: Approve

[Naresh(OpenERP) (nch-openerp)]

Hello,

This bug is not reproducible on Trunk and so this bug has no meaning anymore). If this Merge Proposal could not be merged in v6.1 at the release of v7.0, it will be closed.

Thanks,
Naresh Soni

Unmerged revisions

6967. By Atul Patel <email address hidden> on 2012-08-29

[FIX] when selecting external storage media for a document and when we access it thruough ftp it gives warning that Path is not directory

=== modified file 'document/nodes.py'
--- document/nodes.py	2011-12-19 16:54:40 +0000
+++ document/nodes.py	2012-08-29 05:26:25 +0000
@@ -240,6 +240,7 @@
             s = []
         if isinstance(self.path,list):
             s+=self.path
+            s = list(set(s))
         elif self.path is None:
             s.append('')
         else:
@@ -1244,8 +1245,6 @@
         once caching is in, we might want to do that at init time and keep
         this object anyway
         """
-        if self.path or self.parent:
-            return
         assert fbro
         uid = self.context.uid