CfsPath>>isValidPathName: is buggy

[Michael Keppler]

Hi,

I consider CfsPath>>isValidPathName: (or more specifically some of the sub class implementations) to be broken for case insensitive file systems. E.g. on a Windows system with NTFS or FAT file system it will answer

Code: Select all

CfsPath isValidPathName: 'c:\'  --> true
CfsPath isValidPathName: 'C:\'  --> false

I fixed this by replacing the above method with the following code (as there is already a method isCaseSensitive for the different file system implementations):

Code: Select all

isValidPathName: pathName
   "Answer whether @pathName is a valid path name."
   
   | validName |
   
   validName := self defaultSubclass makeValidFormatFrom: pathName.
   ^(validName = pathName) or: [self defaultSubclass isCaseSensitive not and: [validName equalsIgnoringCase: pathName]].

Ciao, Michael

[wembley]

Michael -

You are correct -- CfsPath class>>#isValidPathName: is broken on platforms are are not case sensitive. Thank you for bringing this to our attention. I have opened case 39203 for this problem. A fix is planned for V8.0.1.