refactor: unwanted logs and warnings removed from console (CircuitVer…

…se#287) * console logs removed * fix: removed one transition warning * removed unused code
itsHamdySalem · Mar 18, 2024 · 594cb5b · 594cb5b
1 parent 68b7608
commit 594cb5b
Show file tree

Hide file tree

Showing 23 changed files with 1 addition and 85 deletions.
diff --git a/src/components/ContextMenu/ContextMenu.vue b/src/components/ContextMenu/ContextMenu.vue
@@ -58,7 +58,6 @@ export default {
         menuItemClicked(event) {
             this.hideContextMenu()
             const id = event.target.dataset.index
-            console.log('Hello From Context Menu' + id)
             if (id == 0) {
                 document.execCommand('copy')
             } else if (id == 1) {

diff --git a/src/components/DialogBox/BooleanTable.vue b/src/components/DialogBox/BooleanTable.vue
@@ -20,6 +20,4 @@ const props = defineProps({
     tableHeader: { type: Array, default: () => [] },
     tableBody: { type: Array, default: () => [] },
 })
-console.log(props.tableHeader)
-console.log(props.tableBody)
 </script>
diff --git a/src/components/DialogBox/CombinationalAnalysis.vue b/src/components/DialogBox/CombinationalAnalysis.vue
@@ -130,14 +130,11 @@ function clearData() {
 
 function dialogBoxConformation(selectedOption, circuitItem) {
     // SimulatorState.dialogBox.combinationalanalysis_dialog = false
-    console.log(inputArr.value)
-    console.log(selectedOption)
     // use the above value to show tables and later clear it all
     if (selectedOption == 'showLogicTable') {
         createLogicTable()
     }
     if (selectedOption == 'closeMessageBox') {
-        // console.log('Circuit was not closed')
         for (var ind = 0; ind < inputArr.value.length; ind++) {
             if (inputArr.value[ind].type == 'text') {
                 inputArr.value[ind].val = ''
@@ -150,14 +147,12 @@ function dialogBoxConformation(selectedOption, circuitItem) {
         SimulatorState.dialogBox.combinationalanalysis_dialog = false
     }
     if (selectedOption == 'generateCircuit') {
-        console.log('Generate Cirucit')
         SimulatorState.dialogBox.combinationalanalysis_dialog = false
         generateCircuit()
         clearData()
         SimulatorState.dialogBox.combinationalanalysis_dialog = false
     }
     if (selectedOption == 'printTruthTable') {
-        console.log('Print Truth Table')
         printBooleanTable()
         clearData()
         SimulatorState.dialogBox.combinationalanalysis_dialog = false
@@ -175,9 +170,6 @@ function createLogicTable() {
     outputList = outputList.filter((e) => e)
     booleanExpression = booleanExpression.replace(/ /g, '')
     booleanExpression = booleanExpression.toUpperCase()
-    console.log(inputList)
-    console.log(outputList)
-    console.log(booleanExpression)
 
     var booleanInputVariables = []
     for (var i = 0; i < booleanExpression.length; i++) {
@@ -189,10 +181,6 @@ function createLogicTable() {
     }
     booleanInputVariables.sort()
 
-    console.log(inputList)
-    console.log(outputList)
-    console.log(booleanExpression)
-
     if (
         inputList.length > 0 &&
         outputList.length > 0 &&
@@ -260,11 +248,9 @@ function createBooleanPrompt(inputList, outputList, scope = globalScope) {
     }
     if (output.value == null) {
         for (var i = 0; i < outputListNames.value.length; i++) {
-            console.log(outputListNames.value[i])
             tableHeader.value.push(outputListNames.value[i])
         }
     } else {
-        console.log(outputListNames.value)
         tableHeader.value.push(outputListNames.value)
     }
 
@@ -295,8 +281,6 @@ function createBooleanPrompt(inputList, outputList, scope = globalScope) {
                 output.value[j]
         }
     }
-    console.log(tableHeader.value)
-    console.log(tableBody.value)
     // display Message Box
     SimulatorState.dialogBox.combinationalanalysis_dialog = true
     buttonArr.value = [
@@ -333,7 +317,6 @@ function drawCombinationalAnalysis(
     outputList,
     scope = globalScope
 ) {
-    console.log('inside draw CA')
     findDimensions(scope)
     var inputCount = inputList.length
     var maxTerms = 0
@@ -641,12 +624,10 @@ function solveBooleanFunction(inputListNames, booleanExpression) {
             return ''
         }
     }
-    console.log(output.value)
 }
 
 function generateCircuit() {
     var data = generateBooleanTableData(outputListNamesInteger.value)
-    console.log(data)
     var minimizedCircuit = []
     let inputCount = inputListNames.value.length
     for (const output in data) {
@@ -668,7 +649,6 @@ function generateCircuit() {
             minimizedCircuit.push(temp.result)
         }
     }
-    console.log(output.value)
     if (output.value == null) {
         drawCombinationalAnalysis(
             minimizedCircuit,
@@ -677,7 +657,6 @@ function generateCircuit() {
             globalScope
         )
     } else {
-        console.log('HI')
         drawCombinationalAnalysis(
             minimizedCircuit,
             inputListNames.value,
@@ -688,10 +667,7 @@ function generateCircuit() {
 }
 
 function printBooleanTable() {
-    console.log($('.messageBox .v-card-text')[0])
     var sTable = $('.messageBox .v-card-text')[0].innerHTML
-    console.log('This is the table')
-    console.log(sTable)
 
     var style =
         `<style>

diff --git a/src/components/DialogBox/CustomShortcut.vue b/src/components/DialogBox/CustomShortcut.vue
@@ -222,13 +222,11 @@ function updateEdit(e: KeyboardEvent) {
 }
 
 async function resetKeybinding() {
-    console.log('Reset Keybinding')
     if (await confirmOption('Remove all custom keys & set the default keys?'))
         setDefault()
 }
 
 function saveKeybinding() {
-    console.log('Save Keybinding')
     submit()
     SimulatorState.dialogBox.customshortcut_dialog = false
 }

diff --git a/src/components/DialogBox/ImportProject.vue b/src/components/DialogBox/ImportProject.vue
@@ -110,7 +110,6 @@ function addDropFile(e: DragEvent) {
         const fileExtension = droppedFile.name.split('.').pop()
 
         if (fileExtension === 'cv') {
-            console.log('file dropped')
             file.value[0] = droppedFile
             document
                 .querySelector('.fileInput')

diff --git a/src/components/DialogBox/OpenOffline.vue b/src/components/DialogBox/OpenOffline.vue
@@ -72,24 +72,14 @@ onMounted(() => {
 })
 
 onUpdated(() => {
-    console.log('on updated')
     var data = localStorage.getItem('projectList')
-    console.log(data)
     projectList.value = JSON.parse(localStorage.getItem('projectList')) || {}
-    console.log(toRaw(projectList.value))
-    if (JSON.stringify(projectList.value) == '{}') {
-        console.log('true')
-    } else {
-        console.log('false')
-    }
 })
 
 function deleteOfflineProject(id) {
-    console.log('Hello from ok')
     localStorage.removeItem(id)
     const temp = JSON.parse(localStorage.getItem('projectList')) || {}
     delete temp[id]
-    console.log(temp)
     projectList.value = temp
     localStorage.setItem('projectList', JSON.stringify(temp))
 }

diff --git a/src/components/DialogBox/Themes/ApplyThemes.vue b/src/components/DialogBox/Themes/ApplyThemes.vue
@@ -165,15 +165,11 @@ onMounted(() => {
     selectedTheme.value = localStorage.getItem('theme')
     themes.value = Object.keys(themeOptions)
     themes.value.splice(-1, 1)
-    console.log(themes.value)
     customThemesList.value = CreateAbstraction(themeOptions['Custom Theme'])
     customThemes.value = Object.keys(customThemesList.value)
-    console.log(customThemesList.value)
-    console.log(customThemes.value)
 })
 
 function changeTheme(e) {
-    console.log(e)
     e.preventDefault()
     $('.selected').removeClass('selected')
     let themeCard = $(e.target.parentElement)
@@ -186,8 +182,6 @@ function changeTheme(e) {
 }
 
 function changeCustomTheme(e) {
-    console.log('update custom theme')
-    console.log(e)
     customThemesList.value[e.target.name].color = e.target.value
     customThemesList.value[e.target.name].ref.forEach((property) => {
         themeOptions['Custom Theme'][property] = e.target.value
@@ -197,7 +191,6 @@ function changeCustomTheme(e) {
 }
 
 function applyTheme() {
-    console.log('Apply Theme')
     if (iscustomTheme.value == false) {
         if ($('.selected label').text()) {
             localStorage.removeItem('Custom Theme')
@@ -219,7 +212,6 @@ function applyTheme() {
 }
 function applyCustomTheme() {
     iscustomTheme.value = true
-    console.log('Apply Custom Theme')
     updateThemeForStyle(localStorage.getItem('theme'))
     updateBG()
     localStorage.setItem('theme', 'Custom Theme')
@@ -250,7 +242,6 @@ function receivedText(e) {
 }
 
 function importCustomTheme() {
-    console.log('Import Custom Theme')
     $('#importThemeFile').click()
 
     $('#importThemeFile').on('change', (event) => {
@@ -267,7 +258,6 @@ function importCustomTheme() {
     })
 }
 function exportCustomTheme() {
-    console.log('Export Custom Theme')
     const dlAnchorElem = document.getElementById('downloadThemeFile')
     dlAnchorElem.setAttribute(
         'href',

diff --git a/src/components/Dropdown/DropDown.vue b/src/components/Dropdown/DropDown.vue
@@ -54,7 +54,5 @@ const props = defineProps({
     dropDownHeader: { type: String, default: '' },
     dropDownType: { type: String, default: '' },
 })
-console.log(props.dropDownHeader)
-console.log(props.listItems)
 const userId = useAuthStore().getUserId
 </script>
diff --git a/src/components/Panels/PropertiesPanel/ModuleProperty/ProjectProperty/ProjectProperty.vue b/src/components/Panels/PropertiesPanel/ModuleProperty/ProjectProperty/ProjectProperty.vue
@@ -150,7 +150,6 @@ type SimulatorStateType = {
 //         () => {
 //             circuitName.value = SimulatorState.activeCircuit.name
 //             circuitId.value = SimulatorState.activeCircuit.id
-//             console.log(circuitName.value, circuitId.value)
 //         },
 //         { deep: true }
 //     )

diff --git a/src/components/Panels/TimingDiagramPanel/TimingDiagramPanel.vue b/src/components/Panels/TimingDiagramPanel/TimingDiagramPanel.vue
@@ -68,7 +68,6 @@ const plotRef = ref<HTMLElement | null>(null)
 const cycleUnits = ref(1000)
 
 function handleButtonClick(button: string) {
-    console.log('clicked', button)
     if (button === 'smaller') {
         if (plotRef.value) {
             plotRef.value.style.width = `${Math.max(

diff --git a/src/components/TabsBar/TabsBar.vue b/src/components/TabsBar/TabsBar.vue
@@ -6,6 +6,7 @@
     >
         <draggable
             :key="updateCount"
+            :item-key="updateCount.toString()"
             v-model="SimulatorState.circuit_list"
             class="list-group"
             tag="transition-group"

diff --git a/src/pages/embed.vue b/src/pages/embed.vue
@@ -226,14 +226,12 @@ watch(clockEnabled, (val) => {
 onBeforeMount(() => {
     window.embed = true
     window.logixProjectId = route.params.projectId
-    console.log(window.logixProjectId)
 })
 
 onMounted(() => {
     // $('#zoom-in-embed').on('click', () => ZoomIn())
     // $('#zoom-out-embed').on('click', () => ZoomOut())
 
-    console.log('embed setup')
     startListeners()
     setup()
 })

diff --git a/src/pages/simulatorHandler.vue b/src/pages/simulatorHandler.vue
@@ -40,14 +40,8 @@ async function checkEditAccess() {
         // if user has edit access load circuit data
         if (res.ok) {
             res.json().then((data) => {
-                console.log('all good to go')
                 authStore.setUserInfo(data.data)
                 ;(window as any).isUserLoggedIn = true
-                console.log(
-                    authStore.getIsLoggedIn,
-                    authStore.getUsername,
-                    authStore.getUserId
-                )
                 isLoading.value = false
             })
         } else if (res.status === 403) {

diff --git a/src/simulator/src/combinationalAnalysis.js b/src/simulator/src/combinationalAnalysis.js
@@ -663,6 +663,5 @@ export function createCombinationalAnalysisPrompt(scope = globalScope) {
 //         }
 //     }
 
-//     console.log(output)
 //     return output
 // }
diff --git a/src/simulator/src/hotkey_binder/keyBinder.js b/src/simulator/src/hotkey_binder/keyBinder.js
@@ -16,7 +16,6 @@
 // export function keyBinder() {
     // const simulatorStore = SimulatorStore()
     // simulatorStore.dialogBox.customshortcut_dialog = true
-    // console.log(simulatorStore.dialogBox.customshortcut_dialog)
     // $('#customShortcutDialog').append(editPanel)
     // $('#customShortcutDialog').append(heading)
     // $('#customShortcutDialog').append(markUp)

diff --git a/src/simulator/src/modules/Splitter.js b/src/simulator/src/modules/Splitter.js
@@ -170,7 +170,6 @@ export default class Splitter extends CircuitElement {
                 simulationArea.simulationQueue.add(this.inp1)
             }
             // else if (this.inp1.value !== n) {
-            //     console.log("CONTENTION");
             // }
         }
         this.prevInpValue = this.inp1.value
@@ -190,7 +189,6 @@ export default class Splitter extends CircuitElement {
      * @return {JSON}
      */
     processVerilog() {
-        // console.log(this.inp1.verilogLabel +":"+ this.outputs[0].verilogLabel);
         if (
             this.inp1.verilogLabel !== '' &&
             this.outputs[0].verilogLabel === ''
@@ -218,7 +216,6 @@ export default class Splitter extends CircuitElement {
             const label = `{${this.outputs
                 .map((x) => x.verilogLabel)
                 .join(',')}}`
-            // console.log("HIT",label)
             if (this.inp1.verilogLabel !== label) {
                 this.inp1.verilogLabel = label
                 this.scope.stack.push(this.inp1)

diff --git a/src/simulator/src/setup.js b/src/simulator/src/setup.js
@@ -39,7 +39,6 @@ import { getToken } from '#/pages/simulatorHandler.vue'
  * @category setup
  */
 export function resetup() {
-    console.log('hello from re setup')
     DPR = window.devicePixelRatio || 1
     if (lightMode) {
         DPR = 1
@@ -105,7 +104,6 @@ function setupEnvironment() {
  * @category setup
  */
 function setupElementLists() {
-    // console.log('hello from elements list panel setup')
     // $('#menu').empty()
 
     window.circuitElementList = metadata.circuitElementList

diff --git a/src/simulator/src/subcircuit.js b/src/simulator/src/subcircuit.js
@@ -34,7 +34,6 @@ export function loadSubCircuit(savedData, scope) {
  * @category subcircuit
  */
 export function createSubCircuitPrompt(scope = globalScope) {
-    console.log('Hello')
     if (verilogModeGet() || layoutModeGet()) {
         showError('Subcircuit cannot be inserted in this mode')
         return
@@ -689,7 +688,6 @@ export default class SubCircuit extends CircuitElement {
                 )
             }
         } else {
-            console.log('Unknown Version: ', this.version)
         }
 
         for (var i = 0; i < subcircuitScope.Input.length; i++) {

diff --git a/src/simulator/src/testbench.js b/src/simulator/src/testbench.js
@@ -529,7 +529,6 @@ export function runAll(data, scope = globalScope) {
     const results = {}
     results.detailed = data
     results.summary = { passed: passedCases, total: totalCases }
-    // console.log(JSON.stringify(results.detailed));
     return results
 }
 

diff --git a/src/simulator/src/themer/customThemer.js b/src/simulator/src/themer/customThemer.js
@@ -37,7 +37,6 @@
 // //     downloadAnchor.setAttribute('id', 'downloadThemeFile')
 // //     downloadAnchor.setAttribute('style', 'display:none')
 // //     propertiesContainer.appendChild(downloadAnchor)
-// //     console.log(propertiesContainer)
 // //     return propertiesContainer
 // // }
 
@@ -46,7 +45,6 @@
 //  */
 // // export const CustomColorThemes = () => {
 // //     $('#CustomColorThemesDialog').empty()
-// //     console.log(getCustomThemeCard())
 // //     $('#CustomColorThemesDialog').append(getCustomThemeCard())
 // //     $('#CustomColorThemesDialog').dialog({
 // //         resizable: false,