RFR: 8307105: JFileChooser InvalidPathException when selecting some system folders on Windows [v3]

Alexey Ivanov aivanov at openjdk.org
Fri May 26 14:02:57 UTC 2023


On Thu, 25 May 2023 13:58:17 GMT, Tejesh R <tr at openjdk.org> wrote:

>> As for other Look-and-Feels, `isFileSystem` is used for the same purpose that it's used in the code you're modifying: if a selected object in the list is a file system object (a directory), the text field to the path of the selected object; otherwise, set the text field to `null`.
>
> Ok, then let me work the proposed fix. As per the proposed fix, `FileSystemView.isFileSystem ` has to return if its fileSystem or not......? What about .ink?

No, what I'm proposing is to modify `isLink` only so that it returns `true` if and only if the file is a Windows shortcut `.lnk` file.

---

Another option is to modify `isFileSystem` so that it returns `true` for all filesystem objects, excluding the added condition that it should be not a link to directory.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/13998#discussion_r1206832172



More information about the client-libs-dev mailing list