Bug 220241 - JJ: IRemoteFileSubSystem.list() on the Local file subsystem does not return correct results
Summary: JJ: IRemoteFileSubSystem.list() on the Local file subsystem does not return c...
Status: RESOLVED FIXED
Alias: None
Product: Target Management
Classification: Tools
Component: RSE (show other bugs)
Version: 3.0   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.0 M6   Edit
Assignee: David McKnight CLA
QA Contact: Martin Oberhuber CLA
URL:
Whiteboard:
Keywords: bugday, helpwanted
Depends on:
Blocks: 174977
  Show dependency tree
 
Reported: 2008-02-25 13:02 EST by Ankit Pasricha CLA
Modified: 2008-06-02 03:58 EDT (History)
1 user (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Ankit Pasricha CLA 2008-02-25 13:02:37 EST 
I am calling IRemoteFileSubSystem.list on the "Local" file subsystem with the following parameters:

1) a folder containing a file foo.c and 3 subfolders: foo1, foo2, foo3
2) a file name filter "foo.c"
3) IFileService.FILE_TYPE_FILES_AND_FOLDERS
4) some progress monitor

I only get the file "foo.c" as the result. I would expect to get the file foo.c along with all the subfolders since I asked for both files and folders. The file name filter should only apply to FILES.
Comment 1 Martin Oberhuber CLA 2008-02-25 21:34:35 EST 
There are several different implementations of how the filtering is handled (in SftpFileService, FTPService, DStoreFileService and its associated miner, and LocalFileService).

Apparently the LocalFileService variant is buggy and should be fixed.

Converging to the same implementation in all variants would be desired.
Comment 2 Martin Oberhuber CLA 2008-02-25 21:35:23 EST 
This should be a REALLY simple bug for beginners to start with, since they can easily review the proper implementation from SftpFileService or FTPService.
Comment 3 David McKnight CLA 2008-02-26 16:39:10 EST 
Although it's a good bugday candidate, the requestor needs a fix for this soon so I'm going to take care of it now.
Comment 4 David McKnight CLA 2008-02-26 16:40:17 EST 
I've committed the fix to cvs.