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]