UI5 / webcomponents-react / 25543103990 / 7 – charts

84%

main: 83%

push

github

refactor(MessageView): fix for Useless conditional (#8555)

To fix this without changing behavior, remove the redundant `groupName
&&` check in the JSX where `ListItemGroup` is rendered, since control
flow has already excluded falsy `groupName`.

Best single change:
- In `packages/main/src/components/MessageView/index.tsx`, inside the
`groupedMessages.map` callback, replace:
- `{groupName && <ListItemGroup
headerText={groupName}>{items}</ListItemGroup>}`
  with:
  - `<ListItemGroup headerText={groupName}>{items}</ListItemGroup>`

This preserves functionality exactly while eliminating the always-true
conditional that CodeQL reports.


_Suggested fixes powered by Copilot Autofix. Review carefully before
merging._

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>

486 of 3071 branches covered (15.83%)

Branch coverage included in aggregate %.

1798 of 5479 relevant lines covered (32.82%)

311.79 hits per line