Refactor calls to delete expandedTestCaseIds

because SonarLint isn't happy: https://rules.sonarsource.com/typescript/RSPEC-2870/
kiwitcms · Sep 27, 2023 · 1e75d0a · 1e75d0a
1 parent cd98df2
commit 1e75d0a
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/tcms/testplans/static/testplans/js/get.js b/tcms/testplans/static/testplans/js/get.js
@@ -151,7 +151,7 @@ function redrawSingleRow (testCaseId, testPlanId, permissions) {
     const newRow = getTestCaseRowContent(testCaseRowDocumentFragment.cloneNode(true), allTestCases[testCaseId], permissions)
 
     // remove from expanded list b/c the comment section may have changed
-    delete expandedTestCaseIds[expandedTestCaseIds.indexOf(testCaseId)]
+    expandedTestCaseIds.splice(expandedTestCaseIds.indexOf(testCaseId), 1)
 
     // replace the element in the dom
     $(`[data-testcase-pk=${testCaseId}]`).replaceWith(newRow)