From a502d96a860456ec5e8c96761db70f7cabb74751 Mon Sep 17 00:00:00 2001
From: Paul Martin <paul@paulsputer.com>
Date: Sat, 30 Apr 2016 04:19:14 -0400
Subject: [PATCH] Merge pull request #1073 from gitblit/1062-DocEditorUpdates

---
 src/main/java/com/gitblit/models/PluginRegistry.java |   39 ++++++++++++++++++++++-----------------
 1 files changed, 22 insertions(+), 17 deletions(-)

diff --git a/src/main/java/com/gitblit/models/PluginRegistry.java b/src/main/java/com/gitblit/models/PluginRegistry.java
index 7c1927f..3f692bc 100644
--- a/src/main/java/com/gitblit/models/PluginRegistry.java
+++ b/src/main/java/com/gitblit/models/PluginRegistry.java
@@ -23,7 +23,7 @@
 
 import org.parboiled.common.StringUtils;
 
-import ro.fortsoft.pf4j.PluginVersion;
+import ro.fortsoft.pf4j.Version;
 
 /**
  * Represents a list of plugin registrations.
@@ -80,8 +80,6 @@
 
 		public String projectUrl;
 
-		public String currentRelease;
-
 		public transient String installedRelease;
 
 		public transient String registry;
@@ -93,19 +91,19 @@
 			this.releases = new ArrayList<PluginRelease>();
 		}
 
-		public PluginRelease getCurrentRelease() {
+		public PluginRelease getCurrentRelease(Version system) {
 			PluginRelease current = null;
-			if (!StringUtils.isEmpty(currentRelease)) {
-				// find specified
-				current = getRelease(currentRelease);
-			}
+			Date date = new Date(0);
+			for (PluginRelease pv : releases) {
+				Version requires = Version.ZERO;
+				if (!StringUtils.isEmpty(pv.requires)) {
+					requires = Version.createVersion(pv.requires);
+				}
 
-			if (current == null) {
-				// find by date
-				Date date = new Date(0);
-				for (PluginRelease pv : releases) {
+				if (system.isZero() || system.atLeast(requires)) {
 					if (pv.date.after(date)) {
 						current = pv;
+						date = pv.date;
 					}
 				}
 			}
@@ -121,12 +119,16 @@
 			return null;
 		}
 
-		public InstallState getInstallState() {
+		public InstallState getInstallState(Version system) {
 			if (StringUtils.isEmpty(installedRelease)) {
 				return InstallState.NOT_INSTALLED;
 			}
-			PluginVersion ir = PluginVersion.createVersion(installedRelease);
-			PluginVersion cr = PluginVersion.createVersion(currentRelease);
+			Version ir = Version.createVersion(installedRelease);
+			Version cr = Version.ZERO;
+			PluginRelease curr = getCurrentRelease(system);
+			if (curr != null) {
+				cr = Version.createVersion(curr.version);
+			}
 			switch (ir.compareTo(cr)) {
 			case -1:
 				return InstallState.UNKNOWN;
@@ -143,7 +145,10 @@
 		}
 	}
 
-	public static class PluginRelease implements Comparable<PluginRelease> {
+	public static class PluginRelease implements Serializable, Comparable<PluginRelease> {
+
+		private static final long serialVersionUID = 1L;
+
 		public String version;
 		public Date date;
 		public String requires;
@@ -151,7 +156,7 @@
 
 		@Override
 		public int compareTo(PluginRelease o) {
-			return PluginVersion.createVersion(version).compareTo(PluginVersion.createVersion(o.version));
+			return Version.createVersion(version).compareTo(Version.createVersion(o.version));
 		}
 	}
 }

--
Gitblit v1.9.1