Closed Bug 59128 Opened 24 years ago Closed 22 years ago

FTP Directory doesn't show all children (case-sensitive datasource?)

Categories

(Core :: XUL, defect, P3)

x86
Other
defect

Tracking

()

RESOLVED WORKSFORME
Future

People

(Reporter: selmer, Assigned: waterson)

References

Details

(Whiteboard: [rtm-])

11/04 09 MN6 Goto internal sweetlou download page. Scroll down near the bottom (may not matter) Open any directory. Notice that N6Setup.exe tends to be the top item listed. Scroll up to the top and then back to the directory you opened. Notice that a bunch of new entries suddenly appear, usually config.ini on the top of the list, but several others too. expected all entries to be visible when children were initially drawn.
Nominating rtm with little hope of it being accepted into a build.
Keywords: rtm
This seems to be some quirk in the datasource, or how the tree is using the datasource. It is based on the letter-case of 'N6Setup.exe' vs. 'n6setup.exe'. I set up a fake mirror of the contents of sweetlou's win32 download directory at ftp://jrgm.mcom.com/pub/products/client/seamonkey/windows/32bit/x86 Go to the directory '2000-11-02-08-Mtrunk' and expand the twisty. Note that 'config.ini', 'license.txt' and 'mozilla-win32.zip' are "not there". Now scroll that directory out of view, and then go back down ... those files now show up. FTP DIR returns Moz FTP DS returns After rescroll --------------- ------------------ -------------- N6Setup.exe config.ini config.ini license.txt license.txt mozilla-win32.zip mozilla-win32.zip N6Setup.exe N6Setup.exe readme.txt readme.txt readme.txt redirect.ini redirect.ini redirect.ini size_matters.txt size_matters.txt size_matters.txt update.html update.html update.html Notice that if case-insensitive, then 'N6...' would follow 'moz...'. So, it looks like it is saying "put 'N6...' in the fourth slot, but consider it the first entry" (or hand-waving to that effect). The directory '2000-11-02-06-Mtrunk' is identical to '2000-11-02-08-Mtrunk', except the case is flattened for 'n6setup.exe'. It doesn't show this bug. The directory '2000-11-02-02-MBogus' has the files: and initial outputs: Dfile.txt afile.txt bfile.txt cfile.txt dfile.txt Dfile.txt efile.txt efile.txt ffile.txt ffile.txt gfile.txt gfile.txt So, this is really bad news that ftp has this case-sensitive bug. It may also mean that the fix is possibly small, localized to ftp datasource (?), and is testable.
Summary: FTP Directory doesn't show all children → FTP Directory doesn't show all children (case-sensitive datasource?)
By the way, for the equivalent test for a file:// source, I don't see this bug where files are missing on the first expansion of a folder. (Although, the order of the files is: afile.txt bfile.txt cfile.txt dfile.txt Dfile.txt efile.txt ffile.txt gfile.txt which is lexically wrong, since 'D' < 'a' < 'b' < 'c' < 'd'.)
wow, good analysis, john.
Aw, shucks ...
marking rtm need info to get on radar, but I don't see why this one is special, we have dozens of bugs like this that are rtm-.
Whiteboard: [rtm need info]
recommend rtm-. This is not stop ship, especially since just scrolling around a bit gets the data to appear.
then rtm- it is.
Whiteboard: [rtm need info] → [rtm-]
-> waterson. This a bug in the FTP datasource.
Assignee: hyatt → waterson
I'll look at this once we convert to outliner.
Depends on: 71530
Target Milestone: --- → mozilla1.0
Status: NEW → ASSIGNED
Bugs targeted at mozilla1.0 without the mozilla1.0 keyword moved to mozilla1.0.1 (you can query for this string to delete spam or retrieve the list of bugs I've moved)
Target Milestone: mozilla1.0 → mozilla1.0.1
Target Milestone: mozilla1.0.1 → Future
I think we can mark this WORKSFORME, since we switched to HTML view.
actually, I re-enabled the tree view and went to the testcase in comment #2, and this works fine now. reallyworksforme.
Status: ASSIGNED → RESOLVED
Closed: 22 years ago
Resolution: --- → WORKSFORME
Component: XP Toolkit/Widgets: Trees → XUL
QA Contact: jrgmorrison → xptoolkit.widgets
You need to log in before you can comment on or make changes to this bug.