Skip to content

fix(rdf): scope glossary graph by membership and surface glossary labels #28199

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
harshach merged 4 commits into from
May 17, 2026

test(rdf): fill coverage gaps surfaced during PR review

312d3a6
Select commit
Loading
Failed to load commit list.
Merged

fix(rdf): scope glossary graph by membership and surface glossary labels #28199

test(rdf): fill coverage gaps surfaced during PR review
312d3a6
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / [open-metadata-ui] SonarCloud Code Analysis succeeded May 17, 2026 in 4m 9s

Quality Gate passed

Issues
1351 New issues
0 Accepted issues

Measures
17 Security Hotspots
0.0% Coverage on New Code
4.0% Duplication on New Code

See analysis details on SonarQube Cloud

Annotations

Check failure on line 207 in src/pages/TasksPage/shared/TaskPayloadSchemaFields.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 41 to the 15 allowed. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ40I81ncT_hZbQTVMe1&open=AZ40I81ncT_hZbQTVMe1&pullRequest=28199

Check warning on line 357 in src/components/Database/Profiler/TableProfiler/ColumnProfileTable/ColumnProfileTable.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Unexpected negated condition. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ40IzJKcT_hZbQTVMSG&open=AZ40IzJKcT_hZbQTVMSG&pullRequest=28199

Check warning on line 107 in src/components/Glossary/AddGlossary/AddGlossary.component.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Extract this nested ternary operation into an independent statement. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ40I5ascT_hZbQTVMZy&open=AZ40I5ascT_hZbQTVMZy&pullRequest=28199

Check warning on line 224 in src/utils/EntityUtilClassBase.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Reduce the number of non-empty switch cases from 38 to at most 30. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ40I_CpcT_hZbQTVMiI&open=AZ40I_CpcT_hZbQTVMiI&pullRequest=28199

Check warning on line 102 in src/components/ActivityFeed/ActivityFeedCardV2/ActivityFeedCardV2.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Unexpected negated condition. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ40I3qbcT_hZbQTVMXt&open=AZ40I3qbcT_hZbQTVMXt&pullRequest=28199

Check warning on line 118 in src/constants/Schedular.constants.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

`String.raw` should be used to avoid escaping `\`. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ40JA4_cT_hZbQTVMlm&open=AZ40JA4_cT_hZbQTVMlm&pullRequest=28199

Check warning on line 148 in src/components/DataQuality/TestCases/TestCases.component.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Prefer using nullish coalescing operator (`??`) instead of a logical or (`||`), as it is a safer operator. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ40I2nIcT_hZbQTVMWq&open=AZ40I2nIcT_hZbQTVMWq&pullRequest=28199

Check warning on line 145 in src/components/MyData/Persona/PersonaSelectableList/PersonaSelectableList.component.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Handle this exception or don't catch it at all. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ40I3NvcT_hZbQTVMXQ&open=AZ40I3NvcT_hZbQTVMXQ&pullRequest=28199

Check warning on line 112 in src/components/common/QueryBuilderWidgetV1/QueryBuilderWidgetV1.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

The signature '(tree: ImmutableTree<_TreeI>, config: Config): ImmutableTree<_TreeI>' of 'QbUtils.checkTree' is deprecated. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ40I1occT_hZbQTVMVh&open=AZ40I1occT_hZbQTVMVh&pullRequest=28199

Check warning on line 188 in src/components/common/TableDataCardV2/TableDataCardV2.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Unexpected negated condition. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ40I06FcT_hZbQTVMUz&open=AZ40I06FcT_hZbQTVMUz&pullRequest=28199

Check warning on line 336 in src/hooks/useGridEditController.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

This assertion is unnecessary since the receiver accepts the original type of the expression. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ40JBS2cT_hZbQTVMmT&open=AZ40JBS2cT_hZbQTVMmT&pullRequest=28199

Check warning on line 87 in src/components/common/PopOverCard/EntityPopOverCard.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Handle this exception or don't catch it at all. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ40I1f9cT_hZbQTVMVX&open=AZ40I1f9cT_hZbQTVMVX&pullRequest=28199

Check warning on line 357 in src/components/common/CoverImageUpload/MUICoverImageUpload.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Prefer `globalThis` over `window`. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ40Iz1IcT_hZbQTVMS1&open=AZ40Iz1IcT_hZbQTVMS1&pullRequest=28199

Check warning on line 277 in src/components/common/FileUpload/MUIFileUpload.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Extract this nested ternary operation into an independent statement. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ40I0xNcT_hZbQTVMUt&open=AZ40I0xNcT_hZbQTVMUt&pullRequest=28199

Check warning on line 131 in src/utils/WorkflowConfigUtils.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Prefer using an optional chain expression instead, as it's more concise and easier to read. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ40I_JwcT_hZbQTVMic&open=AZ40I_JwcT_hZbQTVMic&pullRequest=28199

Check warning on line 141 in src/pages/ProfilerConfigurationPage/ProfilerConfigurationPage.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

This assertion is unnecessary since it does not change the type of the expression. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ40I8w2cT_hZbQTVMev&open=AZ40I8w2cT_hZbQTVMev&pullRequest=28199

Check warning on line 98 in src/components/common/FormBuilderV1/FormBuilderV1.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Prefer using an optional chain expression instead, as it's more concise and easier to read. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ40I0l9cT_hZbQTVMUI&open=AZ40I0l9cT_hZbQTVMUI&pullRequest=28199

Check warning on line 93 in src/components/common/DomainLabel/DomainLabel.component.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

This assertion is unnecessary since the receiver accepts the original type of the expression. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ40I1A0cT_hZbQTVMU6&open=AZ40I1A0cT_hZbQTVMU6&pullRequest=28199

Check failure on line 150 in src/utils/BlockEditorUtils.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Prefer `.dataset` over `getAttribute(…)`. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ40I_NMcT_hZbQTVMij&open=AZ40I_NMcT_hZbQTVMij&pullRequest=28199

Check warning on line 401 in src/components/OntologyExplorer/OntologyExplorer.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

This assertion is unnecessary since the receiver accepts the original type of the expression. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ40I7q8cT_hZbQTVMdF&open=AZ40I7q8cT_hZbQTVMdF&pullRequest=28199

Check warning on line 403 in src/hooks/useGridEditController.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Prefer `globalThis` over `window`. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ40JBS2cT_hZbQTVMmV&open=AZ40JBS2cT_hZbQTVMmV&pullRequest=28199

Check warning on line 13 in src/components/Modals/DeleteModal/DeleteModal.interface.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Complete the task associated to this "TODO" comment. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ40I2CPcT_hZbQTVMV2&open=AZ40I2CPcT_hZbQTVMV2&pullRequest=28199

Check warning on line 371 in src/components/Glossary/GlossaryV1.component.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Unexpected negated condition. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ40I5Y2cT_hZbQTVMZx&open=AZ40I5Y2cT_hZbQTVMZx&pullRequest=28199

Check warning on line 996 in src/components/DataContract/ODCSImportModal/ODCSImportModal.component.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Do not use Array index in keys 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ40I7QwcT_hZbQTVMca&open=AZ40I7QwcT_hZbQTVMca&pullRequest=28199

Check warning on line 376 in src/components/Settings/Applications/AppLogsViewer/AppLogsViewer.component.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Prefer using nullish coalescing operator (`??`) instead of a ternary expression, as it is simpler to read. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ40I4bbcT_hZbQTVMYr&open=AZ40I4bbcT_hZbQTVMYr&pullRequest=28199
View more details on SonarQubeCloud