Skip to content
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.

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

Opprydding/forbetring av logikk/validering/typing relatert til sorteringsfelt og -rekkefølge #1843

Merged
merged 4 commits into from
Jan 23, 2025

Conversation

slovrid
Copy link
Contributor

@slovrid slovrid commented Jan 23, 2025

Describe your changes

Bygger litt vidare på opprydningsarbeidet som blei starta for filtreringsverdiar.
Denne PR-en gjer hovedsakleg følgande:

  • Byttar frå Switch Statement til Switch Expression og fjernar default-case i OpensearchQueryBuilder.sorterQueryParametere - dette for at vi skal kunne få hjelp av Java-kompilatoren til å hugse på å eksplisitt ha ein mapping frå alle kjente Sorteringsfelt-enums til korresponderande Opensearch-filter
    • I forbindelse med dette har eg også lagt inn ein kommentar om samanhengen mellom Sorteringsfelt / OppfolgingsBruker.java
  • Legger til enum for lovlige sorteringsrekkefølge-verdiar, Sorteringsrekkefolge.java
  • Gjer litt om på valideringen av høvesvis sorteringsfelt og sorteringsrekkefølge, slik at vi no får ei direkte kopling til enumen og slepp å vedlikehalde ei separat liste inne i ValideringsRegler.java
  • Gjer tidleg konvertering/validering av sortOrder/sortField - no får vi sterk typing allereie i controllerane slik at vi kan forholde oss til sterke typar nedover i kallkjeda, i staden for String - fjernar også nokre dupliserte valideringar som vi allereie gjorde i controllerane
  • Fjernar nokre testar som ikkje lenger er nødvendige

Trello ticket number and link

TC-878

Type of change

Please delete options that are not relevant.

  • Maintenance

Checklist before requesting a review

  • I have performed a self-review of my code

@slovrid slovrid requested a review from a team as a code owner January 23, 2025 10:12
@github-actions github-actions bot added the not dependabot PRs opened by humans (and other bots) label Jan 23, 2025
@slovrid slovrid changed the title Sorteringsverdiar Opprydding/forbetring av logikk/validering/typing relatert til sorteringsfelt og -rekkefølge Jan 23, 2025
Copy link
Contributor

@Mathiamu Mathiamu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@slovrid slovrid merged commit 578de4a into dev Jan 23, 2025
12 checks passed
@slovrid slovrid deleted the sorteringsverdiar branch January 23, 2025 12:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
not dependabot PRs opened by humans (and other bots)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants