From: Jim Procter <jprocter@issues.jalview.org>
Date: Wed, 6 Mar 2019 15:29:34 +0000 (+0000)
Subject: JAL-3208 re-enable the setprop CLI arg, but only in JalviewJS
X-Git-Tag: Release_2_11_4_0~45^2~18^2~236
X-Git-Url: http://source.jalview.org/gitweb/?a=commitdiff_plain;h=2d8dfad8a0aa6a6febfc0c3bc0379de0b46e0cfc;p=jalview.git

JAL-3208 re-enable the setprop CLI arg, but only in JalviewJS
---

diff --git a/src/jalview/bin/Jalview.java b/src/jalview/bin/Jalview.java
index 5527450..f2bef93 100755
--- a/src/jalview/bin/Jalview.java
+++ b/src/jalview/bin/Jalview.java
@@ -325,9 +325,10 @@ public class Jalview
       else
       {
         System.out.println("Executing setprop argument: " + defs);
-        // DISABLED FOR SECURITY REASONS
-        // TODO: add a property to allow properties to be overriden by cli args
-        // Cache.setProperty(defs.substring(0,p), defs.substring(p+1));
+        if (Platform.isJS())
+        {
+          Cache.setProperty(defs.substring(0,p), defs.substring(p+1));
+        }
       }
       defs = aparser.getValue("setprop");
     }