Add check isFile to file opener

[Kvaz1r]

After adding auto-completion current dir as first choice it's very easy to open some directory instead of file. This check will help user didn't get confused when he will try to save such file.

Type

• [ ] Bug Fix
• [ ] New Feature
• [x] Quality of Life Improvement

Tests

I have tested locally against:

• [x] Windows
• [ ] linux
• [ ] OS X
• [ ] BSD

I have included automated tests:

• [ ] Unit tests
• [ ] Integration tests
• [ ] End to end tests

[nelsam]

@Kvaz1r thanks for the fix! I've pushed an update for two things:

1. The old code (which I wrote) was capitalizing error messages, which is considered bad practice. I basically just changed the error message to lower case.
2. The FileOpener had access to a status.General (which can be used to show errors in the UI), but it wasn't making use of it. It now adds the error message to f.Err before returning the error, so that the user sees the message in the UI.

Let me know if it works for you and I'll merge :smiley:

[Kvaz1r]

Yes, it works for me.