From e68b20aaff2208e7c1e2c20535f1bb4dc1a7bc70 Mon Sep 17 00:00:00 2001
From: Heng Sin Low <hengsin@gmail.com>
Date: Thu, 27 Jun 2013 10:13:59 +0800
Subject: [PATCH] Pre-release clean up: 1) Remove firefox download link and
 redundant idempiere.org link 2) Updated browser support check. We support ff,
 chrome, webkit and ie 8+.

---
 .../src/org/adempiere/webui/apps/AEnv.java    | 27 ++++++++++++-------
 .../theme/default/zul/login/login-links.zul   | 12 ---------
 2 files changed, 18 insertions(+), 21 deletions(-)

diff --git a/org.adempiere.ui.zk/WEB-INF/src/org/adempiere/webui/apps/AEnv.java b/org.adempiere.ui.zk/WEB-INF/src/org/adempiere/webui/apps/AEnv.java
index 228bf109ac..61d499262c 100644
--- a/org.adempiere.ui.zk/WEB-INF/src/org/adempiere/webui/apps/AEnv.java
+++ b/org.adempiere.ui.zk/WEB-INF/src/org/adempiere/webui/apps/AEnv.java
@@ -548,18 +548,27 @@ public final class AEnv
     		return false;
 
     	Object n = execution.getNativeRequest();
-    	boolean supported = false;
     	if (n instanceof ServletRequest) {
-    		String userAgent = Servlets.getUserAgent((ServletRequest) n);
-    		if (userAgent.indexOf("Firefox") >= 0) {
-    			supported = true;
-    		} else if (userAgent.indexOf("AppleWebKit") >= 0) {
-    			if (userAgent.indexOf("Chrome") >= 0 || userAgent.indexOf("Safari") >= 0) {
-    				supported = true;
-    			}
+    		Double version = Servlets.getBrowser((ServletRequest) n, "ff");
+    		if (version != null) {
+    			return true;
     		}
+    		
+    		version = Servlets.getBrowser((ServletRequest) n, "chrome");
+    		if (version != null) {
+    			return true;
+    		}
+    		
+    		version = Servlets.getBrowser((ServletRequest) n, "webkit");
+    		if (version != null) {
+    			return true;
+    		}
+    		
+    		version = Servlets.getBrowser((ServletRequest) n, "ie");
+    		if (version != null && version.intValue() >= 8)
+    			return true;
     	}
-    	return supported;
+    	return false;
     }
 
     /**
diff --git a/org.adempiere.ui.zk/theme/default/zul/login/login-links.zul b/org.adempiere.ui.zk/theme/default/zul/login/login-links.zul
index 6cc67115c0..ee4f2857de 100644
--- a/org.adempiere.ui.zk/theme/default/zul/login/login-links.zul
+++ b/org.adempiere.ui.zk/theme/default/zul/login/login-links.zul
@@ -21,18 +21,6 @@ Copyright (C) 2009 Idalica Corporation
 	<toolbarbutton href="http://www.zkoss.org" image="http://www.zkoss.org/img/zkpowered_s.png"
 		tooltiptext="Powered by Zk" target="_blank"
 		/>
-	<button href="http://www.mozilla.com/firefox" id="getFirefox"
-		image="http://sfx-images.mozilla.org/affiliates/Buttons/firefox3/110x32_get_ffx.png"
-		target="_blank"
-		/>
-	<button href="http://www.idempiere.org" image="images/iD32.png"
-		tooltiptext="iDempiere Community" style="height:32px; width:32px" target="_blank"/>
 </div>
 
-<zscript if="${desktop.execution.serverName == &quot;ip-10-8-0-1.ec2.internal&quot; or desktop.execution.serverName.startsWith(&quot;demo.&quot;)}">
-<![CDATA[
-	getFirefox.setHref("http://www.mozilla.com/?from=sfx&uid=0&t=306");
-]]>
-</zscript>
-
 </zk>