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 |   47 ++++++++++++++---------------------------------
 1 files changed, 14 insertions(+), 33 deletions(-)

diff --git a/src/main/java/com/gitblit/models/PluginRegistry.java b/src/main/java/com/gitblit/models/PluginRegistry.java
index 0c5d5a1..3f692bc 100644
--- a/src/main/java/com/gitblit/models/PluginRegistry.java
+++ b/src/main/java/com/gitblit/models/PluginRegistry.java
@@ -80,8 +80,6 @@
 
 		public String projectUrl;
 
-		public String currentRelease;
-
 		public transient String installedRelease;
 
 		public transient String registry;
@@ -95,38 +93,17 @@
 
 		public PluginRelease getCurrentRelease(Version system) {
 			PluginRelease current = null;
-			if (!StringUtils.isEmpty(currentRelease)) {
-				// find specified
-				current = getRelease(currentRelease);
-			}
-
-			if (current != null) {
-				// verify the current release is acceptable for this system
+			Date date = new Date(0);
+			for (PluginRelease pv : releases) {
 				Version requires = Version.ZERO;
-				if (!StringUtils.isEmpty(current.requires)) {
-					requires = Version.createVersion(current.requires);
+				if (!StringUtils.isEmpty(pv.requires)) {
+					requires = Version.createVersion(pv.requires);
 				}
 
-				if (!system.isZero() && !system.atLeast(requires)) {
-					// requires newer system version
-					current = null;
-				}
-			}
-
-			if (current == null) {
-				// find by date
-				Date date = new Date(0);
-				for (PluginRelease pv : releases) {
-					Version requires = Version.ZERO;
-					if (!StringUtils.isEmpty(pv.requires)) {
-						requires = Version.createVersion(pv.requires);
-					}
-
-					if (system.isZero() || system.atLeast(requires)) {
-						if (pv.date.after(date)) {
-							current = pv;
-							date = pv.date;
-						}
+				if (system.isZero() || system.atLeast(requires)) {
+					if (pv.date.after(date)) {
+						current = pv;
+						date = pv.date;
 					}
 				}
 			}
@@ -142,12 +119,16 @@
 			return null;
 		}
 
-		public InstallState getInstallState() {
+		public InstallState getInstallState(Version system) {
 			if (StringUtils.isEmpty(installedRelease)) {
 				return InstallState.NOT_INSTALLED;
 			}
 			Version ir = Version.createVersion(installedRelease);
-			Version cr = Version.createVersion(currentRelease);
+			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;

--
Gitblit v1.9.1