Summary: | [regression][filters] cannot expand folders below Root filter | ||||||
---|---|---|---|---|---|---|---|
Product: | [Tools] Target Management | Reporter: | Martin Oberhuber <mober.at+eclipse> | ||||
Component: | RSE | Assignee: | David McKnight <dmcknigh> | ||||
Status: | RESOLVED FIXED | QA Contact: | Martin Oberhuber <mober.at+eclipse> | ||||
Severity: | blocker | ||||||
Priority: | P1 | CC: | javier.montalvoorus, tobias.schwarz, uwe.st, wb-rel | ||||
Version: | 2.0 | ||||||
Target Milestone: | 2.0.1 | ||||||
Hardware: | All | ||||||
OS: | All | ||||||
Whiteboard: | |||||||
Bug Depends on: | 197089 | ||||||
Bug Blocks: | |||||||
Attachments: |
|
Description
Martin Oberhuber
2007-07-25 08:40:37 EDT
Dave please address this with maximum priority. I'm pretty sure that the two bugs listed are the only candidates that introduced this regression, because the regression happens on BOTH dstore and ssh and all other fixes released between I20070719-1300 and I20070724-0735 affect different components of RSE or affect only dstore or only SSH. The problem appears to be with the fix for 197089, where we set the filter to the filterString. I'll investigate. Javier noticed: It happens in my workspace that the filename filter passed to org.eclipse.rse.internal.services.files.ftp.FTPService.internalFetch(String, String, int, IProgressMonitor) in "My Home" is correct ("*") but under "Root" the filter is incorrectly passed as "/*" in all subfolders under "Root" which prevents any file being shown. Setting the bug to "blocker" since it blocks Javier from doing development on bug #197105 (gee that's our first valid blocker!) Created attachment 74556 [details]
fix to also check for separators at 0
I've put in a fix for this. The problem was that in getting the last separator we were checking if the sepIndex is > 0 instead of >= 0: int sepIndex = filterString.lastIndexOf(separator); if (sepIndex > 0) <---- PROBLEM { filter = filterString.substring(sepIndex + 1); } else { // fix for 197089 filter = filterString; } I've attached a patch of the fix and applied to cvs. There is a similar fix for dstore, which has not been committed. https://bugs.eclipse.org/bugs/show_bug.cgi?id=190824 I will check in the code once I get the eclipse id. Released for I20070726-0600 tomorrow |