RockteMQ-AI commented on issue #4583: URL: https://github.com/apache/rocketmq-dashboard/issues/4583#issuecomment-5748307310
**Issue Evaluation** Category: `bug` | Status: **Confirmed** The reported issue has been verified against the current codebase (commit d50ffecc). **Root Cause:** `openEditModal` (line 486 in `web/src/pages/ops/alerts.tsx`) restores form values and modal visibility but does not call `setTestResult(null)`, unlike its siblings: - `openCreateModal` (line 417) ✓ - `openDuplicateModal` (line 504) ✓ - Modal `onCancel` (line 1041) ✓ - Footer cancel (line 1059) ✓ **Impact:** Operators editing multiple rules after testing one will see stale dry-run samples from a previously tested rule, potentially misleading them about the current rule's match status. **Severity:** Medium — UX issue that can cause operator confusion but does not affect data integrity or system stability. **Fix:** Add `setTestResult(null);` to `openEditModal` before `setModalVisible(true);`. A fix PR (#4584) has already been linked by the reporter. --- *Automated evaluation by @RockteMQ-AI* -- 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]
