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/abstracted_fs.py (+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: mp+105651@code.launchpad.net

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.

Thanks.

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 :

Hello,

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 Launchpadhttps://code.launchpad.net/~openerp-dev/openobject-addons/trunk-opw-574727-port-mma/+merge/132311 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.

Thanks,
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
=== modified file 'document_ftp/ftpserver/abstracted_fs.py'
--- document_ftp/ftpserver/abstracted_fs.py 2011-09-09 11:39:16 +0000
+++ document_ftp/ftpserver/abstracted_fs.py 2012-05-14 13:07:20 +0000
@@ -272,7 +272,7 @@
272 if path.startswith('/'):272 if path.startswith('/'):
273 path = path[1:]273 path = path[1:]
274274
275 p_parts = path.split('/') # hard-code the unix sep here, by spec.275 p_parts = path.split(os.sep)
276276
277 assert '..' not in p_parts277 assert '..' not in p_parts
278278