shanedell commented on code in PR #704:
URL: https://github.com/apache/daffodil-vscode/pull/704#discussion_r1265450561


##########
src/launchWizard/launchWizard.js:
##########
@@ -298,15 +322,22 @@ async function updateConfigValues(config) {
   document.getElementById('useExistingServer').checked =
     config.useExistingServer
   document.getElementById('dataEditorPort').value = parseInt(
-    config.dataEditorPort
+    config.dataEditor.port
   )
-  document.getElementById('dataEditorLogFile').value = config.dataEditorLogFile
+  document.getElementById('dataEditorLogFile').value =
+    config.dataEditor.logging.file
   document.getElementById('dataEditorLogLevel').value =
-    config.dataEditorLogLevel
+    config.dataEditor.logging.level
+
+  document.getElementById('dfdlDebuggerLogFile').value =
+    config.dfdlDebugger.file
+  document.getElementById('dfdlDebuggerLogLevel').value =
+    config.dfdlDebugger.level
 
   updateInfosetOutputType()
   updateTDMLAction()
 
+  await clearClasspathList()

Review Comment:
   So, whenever a user selects an already made config if it has the 
`daffodilDebugClasspath` it will automatically add those entries to the UI. It 
does this by splitting the entries by delimeter `:`, then each item has it own 
`<li>` inside of the table. So every time the config selected is updated the 
`daffodilDebugClasspath` list is cleared out and then if 
`daffodilDebugClasspath` is set it will add the entries otherwise leave it 
blank. The clearing makes sure that items from a previous config won't still in 
the list for a different config and when new config is selected that the list 
is empty instead of container the same items from the previous config.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to