use composer/semver to ensure version is newer before advertising it …

…to users
Ed-ITSolutions · Jun 21, 2018 · 5873b63 · 5873b63
1 parent 2bbb4f3
commit 5873b63
Show file tree

Hide file tree

Showing 4 changed files with 87 additions and 3 deletions.
diff --git a/.gitignore b/.gitignore
@@ -0,0 +1 @@
+vendor/
diff --git a/composer.json b/composer.json
@@ -2,7 +2,8 @@
   "name": "ed-itsolutions/wup-client",
   "description": "WordPress Update Provider client for use in plugins / themes.",
   "require": {
-    "php": "^5.3.3 || ^7.0"
+    "php": "^5.3.3 || ^7.0",
+    "composer/semver": "^1.4"
   },
   "license": "MIT",
   "authors": [
@@ -16,4 +17,4 @@
   "autoload": {
     "files": ["wup-client.php"]
   }
-}
+}
diff --git a/composer.lock b/composer.lock
diff --git a/lib/wup-client.php b/lib/wup-client.php
@@ -48,7 +48,7 @@ public function injectUpdate($updates){
       &&
       !empty($state->wupVersion)
       &&
-      $state->wupVersion != $state->localVersion
+      Composer\Semver\Comparator::greaterThan($state->wupVersion, $state->localVersion)
     ){
 			$updates->response[$this->updateResponseKey()] = $this->updateResponse($state);
     }