Changeset 2133 for FCKeditor.Java/trunk/java-core/src/main/java/net/fckeditor/tool/Compatibility.java

Timestamp:

2008-06-25 09:17:42 (7 months ago)

Author:

th-schwarz

Message:

- solved ticked #2295
- cleaned up compatibility code

Files:

• FCKeditor.Java/trunk/java-core/src/main/java/net/fckeditor/tool/Compatibility.java (modified) (2 diffs)

FCKeditor.Java/trunk/java-core/src/main/java/net/fckeditor/tool/Compatibility.java

@@ -35 +35 @@
         /**
          * Checks if a browser's user agent string is compatible for the FCKeditor.
-         * <br />
-         * Adapted from:
-         * http://dev.fckeditor.net/browser/FCKeditor/releases/stable/fckeditor.php
          * 
-         * @param userAgentString
+         * @param userAgentStr
          * @return <code>true</code> if compatible otherwise <code>false</code>
          */
@@ -45 +42 @@
                 if (Utils.isEmpty(userAgentString))
                         return false;
-
-                float version;
+                
+                String userAgentStr = userAgentString.toLowerCase();
 
                 // IE 5.5+, check special keys like 'Opera' and 'mac', because there are some
                 // other browsers, containing 'MSIE' in there agent string!
-                if (userAgentString.indexOf("Opera") < 0 && userAgentString.indexOf("mac") < 0) {
-                        version = getBrowserVersion(userAgentString, ".*MSIE ([\\d]+.[\\d]+).*");
-                        if (version != -1f && version >= 5.5f)
+                if (userAgentStr.indexOf("opera") < 0 && userAgentStr.indexOf("mac") < 0) {
+                        if (getBrowserVersion(userAgentStr, ".*msie ([\\d]+.[\\d]+).*") >= 5.5f)
                                 return true;
                 }
-
-                // for mozilla only, because all firefox versions are supported
-                version = getBrowserVersion(userAgentString, ".*Gecko/([\\d]+).*");
-                if (version != -1f && version >= 20030210f)
+                
+                // for all gecko based browsers
+                if (getBrowserVersion(userAgentStr, ".*rv:([\\d]+.[\\d]+).*") > 1.7f)
                         return true;
-
+                
                 // Opera 9.5+
-                version = getBrowserVersion(userAgentString, "Opera/([\\d]+.[\\d]+).*");
-                if (version != -1f && version >= 9.5f)
-                        return true;
-                version = getBrowserVersion(userAgentString, ".*Opera ([\\d]+.[\\d]+)");
-                if (version != -1f && version >= 9.5f)
+                if (getBrowserVersion(userAgentStr, "opera/([\\d]+.[\\d]+).*") >= 9.5f
+                                || getBrowserVersion(userAgentStr, ".*opera ([\\d]+.[\\d]+)") >= 9.5f)
                         return true;
 
                 // Safari 3+
-                version = getBrowserVersion(userAgentString, ".*AppleWebKit/([\\d]+).*");
-                if (version != -1f && version >= 522f)
+                if (getBrowserVersion(userAgentStr, ".*applewebkit/([\\d]+).*") >= 522f)
                         return true;