Merge lp:~openerp-dev/openobject-addons/6.1-opw-574727-rgo into lp:openobject-addons/6.1

Proposed by Ravi Gohil (OpenERP)
Status: Approved
Approved by: Naresh(OpenERP)
Approved revision: 6798
Proposed branch: lp:~openerp-dev/openobject-addons/6.1-opw-574727-rgo
Merge into: lp:openobject-addons/6.1
Diff against target: 11 lines (+1/-1)
1 file modified
document_ftp/ftpserver/ (+1/-1)
To merge this branch: bzr merge lp:~openerp-dev/openobject-addons/6.1-opw-574727-rgo
Reviewer Review Type Date Requested Status
Naresh(OpenERP) (community) Approve
Ravi Gohil (OpenERP) (community) Needs Resubmitting
Review via email:

Description of the change

When using Filezilla client to connect to FTP in windows machine, it raises an error,

Steps to reproduce:

1.       Install Filezilla, Open it,
2.       Give required parameters(host, username, password, port),
3.       Click on QuickConnect,
4.       I see the databasename
5.       Click on databasename gives error “550 invalid database path”

This fix fixes this issue, kindly review it.


To post a comment you must log in.
Revision history for this message
Ravi Gohil (OpenERP) (rgo-openerp) :
review: Needs Resubmitting
Revision history for this message
Naresh(OpenERP) (nch-openerp) :
review: Approve
Revision history for this message
Naresh(OpenERP) (nch-openerp) wrote :


This bug was qualified as Confirmed on Trunk (means still existing and reproducible). A Merge Proposal for trunk was created to fix it. Here is the link to follow the MP on Launchpad and be informed once it's been merged in trunk: ... If this Merge Proposal could not be merged in v6.1 at the release of v7.0, it will be closed.

Naresh Soni

Unmerged revisions

6798. By Ravi Gohil (OpenERP)

[FIX] document_ftp: when connecting using FTP client FileZilla and clicking on database named directory after logged in, it gives 'Invalid database path' error : (Maintenance Case : 574727)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'document_ftp/ftpserver/'
2--- document_ftp/ftpserver/ 2011-09-09 11:39:16 +0000
3+++ document_ftp/ftpserver/ 2012-05-14 13:07:20 +0000
4@@ -272,7 +272,7 @@
5 if path.startswith('/'):
6 path = path[1:]
8- p_parts = path.split('/') # hard-code the unix sep here, by spec.
9+ p_parts = path.split(os.sep)
11 assert '..' not in p_parts