Changes

• Fixes Just an idea, is using validQsysName here for validation more robust than these individual checks? #2366
• Potentially fixes Downloading using source file with member names that have dollar signs in them #2264
• Prevent endless recursive retries to read member if it exists but fails to be read
• Quoted names:
  • You can now properly create, open, and edit members with quoted names
  • Reworked QSYS path validation to include validating quoted names
• Variant characters:
  • Should be able to now download members with variants (which failed in some cases, refer to examples here

TODO:

• Add some new tests!
• Review for other spots where triple quotes is needed
• Review use of escapePath vs escapeForShell
• Review what to do in the case runSQL is used
• Improve quoted name regex

How to test this PR

Checklist

• have tested my change
• have created one or more test cases
• updated relevant documentation
• Remove any/all console.logs I added
• have added myself to the contributors' list in CONTRIBUTING.md