Set min PHP to 7.1 and add checking code for minor/point release comp…

…arison, fixes #106

lib/main.js

@@ -8,7 +8,7 @@ const {AutoLanguageClient, DownloadFile} = require('atom-languageclient')
 const serverDownloadUrl = 'https://github.com/atom/ide-php/files/1708112/php-ls-5.3.6.tar.gz'
 const serverDownloadSize = 3434791
 const serverLauncher = path.join('felixfbecker', 'language-server', 'bin', 'php-language-server.php')
-const minPHPRuntime = 7
+const minPHPRuntime = '7.1'
 const bytesToMegabytes = 1024 * 1024
 
 class PHPLanguageClient extends AutoLanguageClient {
@@ -65,7 +65,7 @@ class PHPLanguageClient extends AutoLanguageClient {
         if (alreadyShownError) return
         if (exitCode === 0 && stdOut != '') {
           this.logger.debug(`Using PHP ${stdOut} from ${command}`)
-          if (Number(stdOut.split('.')[0]) >= minPHPRuntime) {
+          if (this.meetsRequiredVersion(stdOut, minPHPRuntime)) {
             resolve()
           } else {
             atom.notifications.addError(`IDE-PHP requires PHP ${minPHPRuntime} or later but found ${stdOut}`, {
@@ -89,6 +89,24 @@ class PHPLanguageClient extends AutoLanguageClient {
     })
   }
 
+  meetsRequiredVersion (versionString, requiredString) {
+    var version = versionString.split('.')
+    var required = requiredString.split('.')
+
+    for (var i = 0; i < required.length; i++) {
+      if (i >= version.length) return false
+
+      var versionPoint = parseInt(version[i])
+      var requiredPoint = parseInt(required[i])
+
+      if (versionPoint < requiredPoint) return false;
+      if (versionPoint > requiredPoint) return true;
+      // Otherwise check next part via loop
+    }
+
+    return true
+  }
+
   downloadPHP () {
     shell.openExternal('https://secure.php.net/downloads.php')
   }