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. 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 will 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]
