From dad202eabe5daf84a1beb71977f5a7455fe57143 Mon Sep 17 00:00:00 2001
From: Ruben Daniels <ruben@c9.io>
Date: Tue, 15 Sep 2015 05:39:31 +0000
Subject: [PATCH] Added logic for addExperiment

---
 plugins/c9.ide.preferences/experimental.js | 24 ++++++++++++++++------
 1 file changed, 18 insertions(+), 6 deletions(-)

diff --git a/plugins/c9.ide.preferences/experimental.js b/plugins/c9.ide.preferences/experimental.js
index 74a8c922..5f0f3eb9 100644
--- a/plugins/c9.ide.preferences/experimental.js
+++ b/plugins/c9.ide.preferences/experimental.js
@@ -1,7 +1,7 @@
 define(function(require, exports, module) {
     main.consumes = [
         "PreferencePanel", "ui", "dialog.confirm", "settings",
-        "preferences"
+        "preferences", "c9"
     ];
     main.provides = ["preferences.experimental"];
     return main;
@@ -9,12 +9,14 @@ define(function(require, exports, module) {
     function main(options, imports, register) {
         var PreferencePanel = imports.PreferencePanel;
         var prefs = imports.preferences;
+        var settings = imports.settings;
         var ui = imports.ui;
+        var c9 = imports.c9;
         
         /***** Initialization *****/
         
         var plugin = new PreferencePanel("Ajax.org", main.consumes, {
-            caption: "Project Settings",
+            caption: "Experimental Features",
             form: true,
             index: 50
         });
@@ -53,20 +55,30 @@ define(function(require, exports, module) {
         
         /***** Methods *****/
         
+        // =0 means the value should be set to 0 to disable otherwise it is enabled
+        // =1 means the value should be set to 1 to enable otherwise it is disabled
         function addExperiment(query, name){
+            var key = query.split("=");
+            var defValue = Number(key[1]); key = key[0];
+            var uniqueId = key.replace(/\//g, "-");
+            
             var parts = name.split("/");
             var obj = {}, current = obj;
             for (var i = 0; i < parts.length; i++) {
                 current[parts[i]] = current = {};
             }
             current.type = "checkbox";
-            current.setting = "state/experiments/" + query.split("=")[0].replace(/\//g, "-");
+            current.setting = "state/experiments/" + uniqueId;
             
             plugin.add(obj);
             
-            // TODO return the value of the query (to be checked in the location or the settings):
-            // =0 means the value should be set to 0 to disable otherwise it is enabled
-            // =1 means the value should be set to 1 to enable otherwise it is disabled
+            var idx = c9.location.indexOf(query);
+            var enabled = defValue == 1 ? idx > -1 : idx === -1;
+            
+            if (!enabled)
+                enabled = settings.getBool("state/experiments/" + uniqueId);
+            
+            return enabled;
         }
         
         /***** Lifecycle *****/