UI5 / webcomponents-react / 25543103990

84%

main: 84%

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>

2608 of 3403 branches covered (76.64%)

Branch coverage included in aggregate %.

5195 of 5844 relevant lines covered (88.89%)

104805.03 hits per line