jramos/cve-dashboard
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix ESLint react-hooks/exhaustive-deps warning in ConsolidationModal

Browse Source
This commit is contained in:
Jordan Ramos
2026-05-22 11:19:46 -06:00
parent 6b805ee633
commit 76667f65c6
1 changed files with 8 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
frontend/src/components/ConsolidationModal.js
View File

@@ -279,16 +279,17 @@ export default function ConsolidationModal({ items, onClose, onSuccess }) {
const [submitting, setSubmitting] = useState(false);
// ---------------------------------------------------------------------------
// Initialize / regenerate form fields when selectedItems changes
// Initialize form fields on mount from the original items prop.
// Uses items (prop) not selectedItems (state) to avoid re-triggering on removal.
// ---------------------------------------------------------------------------
useEffect(() => {
if (selectedItems.length >= 2) {
setSummary(generateConsolidatedSummary(selectedItems));
setDescription(generateConsolidatedDescription(selectedItems));
setCveId(extractFirstCve(selectedItems));
setVendor(extractCommonVendor(selectedItems));
if (items.length >= 2) {
setSummary(generateConsolidatedSummary(items));
setDescription(generateConsolidatedDescription(items));
setCveId(extractFirstCve(items));
setVendor(extractCommonVendor(items));
}
}, []); // Only on mount — user edits are preserved after item removal
}, [items]);
// ---------------------------------------------------------------------------
// Remove an item from the selection (minimum 2 required)