Skip to content

fix: Show schema and register names in search results instead of IDs#991

Open
rubenvdlinde wants to merge 3 commits intodevelopmentfrom
fix/search-friendly-names
Open

fix: Show schema and register names in search results instead of IDs#991
rubenvdlinde wants to merge 3 commits intodevelopmentfrom
fix/search-friendly-names

Conversation

@rubenvdlinde
Copy link
Contributor

@rubenvdlinde rubenvdlinde commented Mar 24, 2026

Summary

  • Search result subtitles previously showed raw numeric IDs: Schema: 498 • Register: 228
  • Now resolves to human-readable titles: Client • Pipelinq • Updated: 2026-03-20
  • Uses SchemaMapper and RegisterMapper with a per-request name cache to avoid N+1 queries

Test plan

  • Search for objects via unified search bar
  • Verify subtitles show schema name (e.g. "Client", "Case") instead of numeric ID
  • Verify register name (e.g. "Pipelinq", "Procest") instead of numeric ID
  • Verify fallback to numeric ID when schema/register can't be resolved

Closes #975

Search result subtitles previously showed raw numeric IDs like
"Schema: 498 • Register: 228". Now resolves to human-readable
titles like "Client • Pipelinq" using SchemaMapper and RegisterMapper
with a per-request name cache to avoid repeated lookups.

Closes #975
@github-actions
Copy link
Contributor

Quality Report

Repository ConductionNL/openregister
Commit ff22807
Branch 991/merge
Event pull_request
Generated 2026-03-24 14:30 UTC
Workflow Run https://github.com/ConductionNL/openregister/actions/runs/23494740055

Summary

Group Result
PHP Quality FAIL
Vue Quality PASS
Security PASS
License FAIL
PHPUnit SKIP
Newman SKIP
Playwright SKIP

PHP Quality

Tool Result
lint PASS
phpcs FAIL
phpmd PASS
psalm PASS
phpstan PASS
phpmetrics PASS

Vue Quality

Tool Result
eslint PASS
stylelint PASS

Security

Ecosystem Result
composer PASS
npm PASS

License Compliance

Ecosystem Result
composer PASS
npm SKIP

composer dependencies (147 total)

Metric Count
Approved (allowlist) 146
Approved (override) 1
Denied 0

npm dependencies (586 total)

Metric Count
Approved (allowlist) 585
Approved (override) 1
Denied 0

PHPUnit Tests

PHPUnit tests were not enabled for this run.

Integration Tests (Newman)

Newman integration tests were not enabled for this run.

E2E Tests (Playwright)

Playwright E2E tests were not enabled for this run.


Generated automatically by the Quality workflow.

Download the full PDF report from the workflow artifacts.

@github-actions
Copy link
Contributor

Quality Report

Repository ConductionNL/openregister
Commit 739fdb2
Branch 991/merge
Event pull_request
Generated 2026-03-25 12:43 UTC
Workflow Run https://github.com/ConductionNL/openregister/actions/runs/23541515363

Summary

Group Result
PHP Quality FAIL
Vue Quality PASS
Security PASS
License PASS
PHPUnit SKIP
Newman SKIP
Playwright SKIP

PHP Quality

Tool Result
lint PASS
phpcs FAIL
phpmd PASS
psalm PASS
phpstan PASS
phpmetrics PASS

Vue Quality

Tool Result
eslint PASS
stylelint PASS

Security

Ecosystem Result
composer PASS
npm PASS

License Compliance

Ecosystem Result
composer PASS
npm PASS

composer dependencies (147 total)

Metric Count
Approved (allowlist) 146
Approved (override) 1
Denied 0

npm dependencies (595 total)

Metric Count
Approved (allowlist) 592
Approved (override) 3
Denied 0

PHPUnit Tests

PHPUnit tests were not enabled for this run.

Integration Tests (Newman)

Newman integration tests were not enabled for this run.

E2E Tests (Playwright)

Playwright E2E tests were not enabled for this run.


Generated automatically by the Quality workflow.

Download the full PDF report from the workflow artifacts.

@github-actions
Copy link
Contributor

Quality Report

Repository ConductionNL/openregister
Commit 3b1547b
Branch 991/merge
Event pull_request
Generated 2026-03-25 15:30 UTC
Workflow Run https://github.com/ConductionNL/openregister/actions/runs/23549189261

Summary

Group Result
PHP Quality PASS
Vue Quality PASS
Security PASS
License PASS
PHPUnit SKIP
Newman SKIP
Playwright SKIP

PHP Quality

Tool Result
lint PASS
phpcs PASS
phpmd PASS
psalm PASS
phpstan PASS
phpmetrics PASS

Vue Quality

Tool Result
eslint PASS
stylelint PASS

Security

Ecosystem Result
composer PASS
npm PASS

License Compliance

Ecosystem Result
composer PASS
npm PASS

composer dependencies (147 total)

Metric Count
Approved (allowlist) 146
Approved (override) 1
Denied 0

npm dependencies (595 total)

Metric Count
Approved (allowlist) 592
Approved (override) 3
Denied 0

PHPUnit Tests

PHPUnit tests were not enabled for this run.

Integration Tests (Newman)

Newman integration tests were not enabled for this run.

E2E Tests (Playwright)

Playwright E2E tests were not enabled for this run.


Generated automatically by the Quality workflow.

Download the full PDF report from the workflow artifacts.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant