netpyne-24 Escape variables that are transmitted to python

Author: aranega

netpyne_ui/netpyne_geppetto.py

@@ -774,7 +774,8 @@ def doIhaveInstOrSimData(self):
         return {'haveInstance': out[0], 'haveSimData': out[1]}
 
     def rename(self, path, oldValue, newValue):
-        command = 'sim.rename(self.' + path + ',"' + oldValue + '","' + newValue + '")'
+        # command = 'sim.rename(self.' + path + ',"' + oldValue + '","' + newValue + '")'
+        command = f'sim.rename(self.{path}, {oldValue!r}, {newValue!r})'
         logging.debug('renaming ' + command)
 
         eval(command)

webapp/Utils.js

@@ -156,6 +156,8 @@ const Utils = {
   },
 
   renameKey (path, oldValue, newValue, callback) {
+    oldValue = oldValue.replace(/"/g, '\\"');
+    newValue = newValue.replace(/"/g, '\\"');
     this.execPythonMessage(`netpyne_geppetto.rename("${path}","${oldValue}","${newValue}")`)
       .then((response) => {
         callback(response, newValue);

webapp/components/general/PythonControlledCapability.js

@@ -46,11 +46,14 @@ define((require) => {
         }
 
         connectToPython (componentType, model) {
-          GeppettoUtils.execPythonMessage(`jupyter_geppetto.ComponentSync(componentType="${componentType}",model="${model}",id="${this.id}").connect()`);
+          const id = this.id.replace(/"/g, '\\"');
+          model = model.replace(/"/g, '\\"');
+          GeppettoUtils.execPythonMessage(`jupyter_geppetto.ComponentSync(componentType="${componentType}",model="${model}",id="${id}").connect()`);
         }
 
         disconnectFromPython () {
-          GeppettoUtils.execPythonMessage(`jupyter_geppetto.remove_component_sync(componentType="${this.state.componentType}",model="${this.id}")`);
+          const id = this.id.replace(/"/g, '\\"');
+          GeppettoUtils.execPythonMessage(`jupyter_geppetto.remove_component_sync(componentType="${this.state.componentType}",model="${id}")`);
           GEPPETTO.ComponentFactory.removeExistingComponent(this.state.componentType, this);
         }
 
@@ -150,7 +153,7 @@ define((require) => {
               if ((this.state.model !== nextProps.model) && (nextProps.model !== undefined)) {
                 this.setState({ model: nextProps.model });
             }
-          }          
+          }
         }
 
         componentDidUpdate (prevProps, prevState) {