lsp: use if statement instead of when-unless

This fixes a linting error and also fixes the wrong method being used
for the callback:

lsp-python-ms--language-server-started was being used instead of
lsp-python-ms--language-server-started-callback.

That was most likely due to a bad merge from #3.

Author: seanfarley

lsp-python-ms.el

@@ -154,8 +154,16 @@ WORKSPACE is just used for logging and _PARAMS is unused."
     (list (lsp-python-ms--find-dotnet)
           (concat lsp-python-ms-dir "Microsoft.Python.LanguageServer.dll"))))
 
-;;; Old lsp-mode
-(unless (fboundp 'lsp-register-client)
+(if (fboundp 'lsp-register-client)
+    ;; New lsp-mode
+    (lsp-register-client
+     (make-lsp-client
+      :new-connection (lsp-stdio-connection 'lsp-python-ms--command-string)
+      :major-modes '(python-mode)
+      :server-id 'mspyls
+      :initialization-options 'lsp-python-ms--extra-init-params
+      :notification-handlers (lsp-ht ("python/languageServerStarted" 'lsp-python-ms--language-server-started-callback))))
+  ;; Old lsp-mode
   (lsp-define-stdio-client
    lsp-python "python"
    #'lsp-python-ms--workspace-root
@@ -164,18 +172,6 @@ WORKSPACE is just used for logging and _PARAMS is unused."
    :extra-init-params #'lsp-python-ms--extra-init-params
    :initialize #'lsp-python-ms--client-initialized))
 
-;;; New lsp-mode
-(when (fboundp 'lsp-register-client)
-  (lsp-register-client
-   (make-lsp-client
-    :new-connection (lsp-stdio-connection 'lsp-python-ms--command-string)
-    :major-modes '(python-mode)
-    :server-id 'mspyls
-    :initialization-options 'lsp-python-ms--extra-init-params
-    :notification-handlers (lsp-ht ("python/languageServerStarted" 'lsp-python-ms--language-server-started))
-    )))
-
-
 (provide 'lsp-python-ms)
 
 ;;; lsp-python-ms.el ends here