Jalview-JS/JAL-3253-applet also comments relating to JAL-3268

[jalview.git] / src / jalview / util / UrlLink.java

diff --git a/src/jalview/util/UrlLink.java b/src/jalview/util/UrlLink.java
index 0141a6a..8a60b56 100644 (file)
--- a/src/jalview/util/UrlLink.java
+++ b/src/jalview/util/UrlLink.java
@@ -35,6 +35,8 @@ import java.util.List;
 import java.util.Map;
 import java.util.Vector;
 
+import com.stevesoft.pat.Regex;
+
 /**
  * A helper class to parse URL Link strings taken from applet parameters or
  * jalview properties file using the com.stevesoft.pat.Regex implementation.
@@ -302,8 +304,7 @@ public class UrlLink
     {
       if (regexReplace != null)
       {
-        com.stevesoft.pat.Regex rg = com.stevesoft.pat.Regex
-                .perlCode("/" + regexReplace + "/");
+        Regex rg = Platform.newRegexPerl("/" + regexReplace + "/");
         if (rg.search(idstring))
         {
           int ns = rg.numSubs();
@@ -507,21 +508,8 @@ public class UrlLink
     {
       // Extract Regex and suffix
       urlSuffix = link.substring(p + endDelimiter.length());
-      regexReplace = link.substring(sqidPos + startLength, p);
-      try
-      {
-        com.stevesoft.pat.Regex rg = com.stevesoft.pat.Regex
-                .perlCode("/" + regexReplace + "/");
-        if (rg == null)
-        {
-          invalidMessage = "Invalid Regular Expression : '" + regexReplace
-                  + "'\n";
-        }
-      } catch (Exception e)
-      {
-        invalidMessage = "Invalid Regular Expression : '" + regexReplace
-                + "'\n";
-      }
+      testRegexPerl(
+              regexReplace = link.substring(sqidPos + startLength, p));
     }
     else
     {
@@ -550,6 +538,21 @@ public class UrlLink
     return p;
   }
 
+  private void testRegexPerl(String r)
+  {
+    Regex rg = null;
+    try
+    {
+      rg = Platform.newRegexPerl("/" + r + "/");
+    } catch (Exception e)
+    {
+    }
+    if (rg == null)
+    {
+      invalidMessage = "Invalid Regular Expression : '" + r + "'\n";
+    }
+  }
+
   /**
    * Create a set of URL links for a sequence
    *