Expected Behavior - What were you expecting to happen?
When the oomph project catalog is not containing an identifiable one to be assigned to a GEM project or environment, then the launcher should be disabled as it is a configuration problem, or a misconfigured-launcher should be present to show the error and communicate to the config managers to be fixed.
Current Behavior - What happens?
Currently (v 0.3.0), if the project ID is not present on the oomph project catalog the launcher fails with an exception. This is rather a configuration problem than a launch error, and should be handle properly.
Possible Solution
Check the project catalog on canLaunch to disable it in case that the configuration is not present
Does not provide any Eclipse launcher for an environment if it is not found on the project catalog
Provide a misconfigured launcher to show to the user that Eclipse is not configured for that project
I personally prefer 2 as it is possible that an environment is not present cause that environment does not support the Eclipse launch; also the first one is possible in case that a filter for those use cases is provided (see https://github.com/aposin/gem/issues/42).
Feature request
Expected Behavior - What were you expecting to happen?
When the oomph project catalog is not containing an identifiable one to be assigned to a GEM project or environment, then the launcher should be disabled as it is a configuration problem, or a misconfigured-launcher should be present to show the error and communicate to the config managers to be fixed.
Current Behavior - What happens?
Currently (v 0.3.0), if the project ID is not present on the oomph project catalog the launcher fails with an exception. This is rather a configuration problem than a launch error, and should be handle properly.
Possible Solution
canLaunch
to disable it in case that the configuration is not presentI personally prefer 2 as it is possible that an environment is not present cause that environment does not support the Eclipse launch; also the first one is possible in case that a filter for those use cases is provided (see https://github.com/aposin/gem/issues/42).