Merge pull request #31 from pagopa/PPANTT-91-fix-paging-issue

[PPANTT-91] fix: paging issues
pagopa · Aug 21, 2024 · 240e19d · 240e19d
2 parents 089087e + 1e9d386
commit 240e19d
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/main/java/it/pagopa/selfcare/pagopa/service/impl/ExternalServiceImpl.java b/src/main/java/it/pagopa/selfcare/pagopa/service/impl/ExternalServiceImpl.java
@@ -42,7 +42,7 @@ public ExternalServiceImpl(BrokerInstitutionsRepository brokerInstitutionsReposi
  @Override
  public BrokerInstitutionsResponse getBrokerInstitutions(String brokerCode, Integer limit, Integer page) {
  Optional<BrokerInstitutionAggregate> brokerInstitutionsEntity = brokerInstitutionsRepository
- .findPagedInstitutionsByBrokerCode(brokerCode, page == 0 ? 0 : ((page * limit) - 1), limit);
+ .findPagedInstitutionsByBrokerCode(brokerCode, page * limit, limit);
  if (brokerInstitutionsEntity.isEmpty() ||
  brokerInstitutionsEntity.get().getInstitutionEntities() == null) {
  throw new AppException(AppError.BROKER_INSTITUTIONS_NOT_FOUND, brokerCode);
@@ -89,7 +89,7 @@ public CIIbansResponse getCIsIbans(Integer limit, Integer page) {
  public CIIbansResponse getBrokerIbans(String brokerCode, Integer limit, Integer page) {
  Pageable pageable = PageRequest.of(page, limit);
  Optional<IbanAggregate> brokerIbanAggregate = brokerIbansRepository.getBrokerIbans(
- brokerCode, page == 0 ? 0 : ((page * limit) - 1), limit);
+ brokerCode, page * limit, limit);
  if (brokerIbanAggregate.isEmpty() ||
  brokerIbanAggregate.get().getIbansSlice() == null) {
  throw new AppException(AppError.BROKER_IBANS_NOT_FOUND, brokerCode);