Stefano Crocco authored May 13, 2021 and David Faure committed May 13, 2021

I refactored the Refactor ClientApp::createNewWindow function to remove the goto. I split the function in four parts:

* parseBrowserApplicationString which takes the BrowserApplication string and parses it to determine if it's a command or a service, whether it calls kfmclient itself and to create the command line
* launchExternalBrowserCommand which launches the external browser if it's specified as a command
* launchExternalBrowserService which launches the external browser if it's specified as a service
* createNewWindow itself which uses the above three functions.