Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

FtpFileObject.getChildren() should throw org.apache.commons.vfs2.FileNotFoundException instead of java.io.FileNotFoundException #638

Conversation

mareksunda
Copy link
Contributor

Fixed consistency of wrongly using java.io.FileNotFoundException instead of much preferred org.apache.commons.vfs2.FileNotFoundException

@garydgregory
Copy link
Member

garydgregory commented Nov 27, 2024

Hello @mareksunda

From our README file:

Before you pushing a PR, run mvn (by itself), this runs the default goal, which contains all build checks.

Otherwise, good catch!

@raboof
Copy link
Member

raboof commented Nov 28, 2024

Looks like the error is:

Error:  src\main\java\org\apache\commons\vfs2\provider\ftp\FtpFileObject.java:[39,1] (imports) ImportOrder: Wrong order for 'org.apache.commons.vfs2.FileNotFolderException' import.

…ead of preferred org.apache.commons.vfs2.FileNotFoundException
@mareksunda mareksunda force-pushed the vfs-file-not-found-exception-inconsistency-ms branch from 6044d9a to b85424f Compare November 28, 2024 10:06
@mareksunda
Copy link
Contributor Author

Fixed. Apologies. I ran the tests, but not all.. embarrassing.

@garydgregory
Copy link
Member

Fixed. Apologies. I ran the tests, but not all.. embarrassing.

@mareksunda
Thank you for the update. I've done worse 😜

@garydgregory garydgregory merged commit 6bb85fb into apache:master Nov 28, 2024
18 of 21 checks passed
@garydgregory garydgregory changed the title Fix consistency of wrongly used java.io.FileNotFoundException FtpFileObject.getChildren() should throw org.apache.commons.vfs2.FileNotFoundException instead of java.io.FileNotFoundException Nov 28, 2024
asfgit pushed a commit that referenced this pull request Nov 28, 2024
org.apache.commons.vfs2.FileNotFoundException instead of
java.io.FileNotFoundException #638
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants