Skip to content

Commit 42dc510

Browse files
committed
Refactor age query logic to use LocalDate for improved clarity and accuracy
1 parent 844e5dc commit 42dc510

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

apps/dolly-search-service/src/main/java/no/nav/testnav/dollysearchservice/utils/OpenSearchPersonQueryUtils.java

+4-4
Original file line numberDiff line numberDiff line change
@@ -39,18 +39,18 @@ public class OpenSearchPersonQueryUtils {
3939

4040
public static void addAlderQuery(BoolQueryBuilder queryBuilder, SearchRequest request) {
4141

42-
var thisYear = LocalDate.now().getYear();
42+
var now = LocalDate.now();
4343
if (nonNull(request.getPersonRequest().getAlderFom()) || nonNull(request.getPersonRequest().getAlderTom())) {
4444
queryBuilder.must(QueryBuilders.boolQuery()
4545
.must(nestedMatchQuery("hentPerson.foedselsdato", METADATA_HISTORISK, false))
4646
.must(QueryBuilders.nestedQuery("hentPerson.foedselsdato",
4747
QueryBuilders.boolQuery().must(
48-
rangeQuery("hentPerson.foedselsdato.foedselsaar",
48+
rangeQuery("hentPerson.foedselsdato.foedselsdato",
4949
Optional.ofNullable(request.getPersonRequest().getAlderTom())
50-
.map(alderTom -> thisYear - alderTom)
50+
.map(now::minusYears)
5151
.orElse(null),
5252
Optional.ofNullable(request.getPersonRequest().getAlderFom())
53-
.map(alderFom -> thisYear - alderFom)
53+
.map(now::minusYears)
5454
.orElse(null))), ScoreMode.Avg)));
5555
}
5656
}

apps/dolly-search-service/src/main/java/no/nav/testnav/dollysearchservice/utils/OpenSearchQueryUtils.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ public class OpenSearchQueryUtils {
1515
public static final String NAVSPERSONIDENTIFIKATOR = "hentPerson.navspersonidentifikator";
1616
public static final String CONCAT = "%s.%s";
1717

18-
public static QueryBuilder rangeQuery(String field, Integer value1, Integer value2) {
18+
public static QueryBuilder rangeQuery(String field, Object value1, Object value2) {
1919

2020
return QueryBuilders.rangeQuery(field).from(value1).to(value2);
2121
}

0 commit comments

Comments
 (0)