diff --git a/apps/dnet-orgs-database-application/src/main/java/eu/dnetlib/organizations/controller/OrganizationController.java b/apps/dnet-orgs-database-application/src/main/java/eu/dnetlib/organizations/controller/OrganizationController.java index c73de641..ce76e647 100644 --- a/apps/dnet-orgs-database-application/src/main/java/eu/dnetlib/organizations/controller/OrganizationController.java +++ b/apps/dnet-orgs-database-application/src/main/java/eu/dnetlib/organizations/controller/OrganizationController.java @@ -152,18 +152,14 @@ public class OrganizationController extends AbstractDnetController { } @GetMapping("/conflicts/byCountry/{country}") - public Collection> findConflictsByCountry(@PathVariable final String country, - @RequestParam(required = false, defaultValue = "1000") final int limit, - final Authentication authentication) { + public Collection> findConflictsByCountry(@PathVariable final String country, final Authentication authentication) { - if (UserInfo.isSuperAdmin(authentication)) { - return groupConflicts(conflictGroupViewRepository.findByCountry1OrCountry2(country, country, PageRequest.of(0, limit)).stream()); - } + if (UserInfo.isSuperAdmin(authentication)) { return groupConflicts(conflictGroupViewRepository.findByCountry1OrCountry2(country, country).stream()); } if (UserInfo.isSimpleUser(authentication) || UserInfo.isNationalAdmin(authentication)) { final Stream list = userCountryRepository.getCountriesForUser(UserInfo.getEmail(authentication)) .stream() .filter(country::equalsIgnoreCase) - .map(c -> conflictGroupViewRepository.findByCountry1OrCountry2(c, c, PageRequest.of(0, limit)).stream()) + .map(c -> conflictGroupViewRepository.findByCountry1OrCountry2(c, c).stream()) .findFirst() .orElse(Stream.empty()); return groupConflicts(list); @@ -174,7 +170,7 @@ public class OrganizationController extends AbstractDnetController { @GetMapping("/duplicates/byCountry/{country}") public Iterable findDuplicatesByCountry(@PathVariable final String country, - @RequestParam(required = false, defaultValue = "1000") final int limit, + @RequestParam(required = false, defaultValue = "${openorgs.findDuplicatesByCountry.limit.default}") final int limit, final Authentication authentication) { if (UserInfo.isSuperAdmin(authentication)) { return duplicateGroupViewRepository.findByCountry(country, PageRequest.of(0, limit)); } @@ -268,7 +264,7 @@ public class OrganizationController extends AbstractDnetController { @GetMapping("/byCountry/{status}/{code}") public List findOrgsByStatusAndCountry(@PathVariable final String status, @PathVariable final String code, - @RequestParam(required = false, defaultValue = "1000") final int limit, + @RequestParam(required = false, defaultValue = "${openorgs.findOrgsByStatusAndCountry.limit.default}") final int limit, final Authentication authentication) { if (!UserInfo.isSuperAdmin(authentication) && !userCountryRepository.verifyAuthorizationForCountry(code, UserInfo.getEmail(authentication))) { throw new RuntimeException("User not authorized"); diff --git a/apps/dnet-orgs-database-application/src/main/java/eu/dnetlib/organizations/repository/readonly/ConflictGroupViewRepository.java b/apps/dnet-orgs-database-application/src/main/java/eu/dnetlib/organizations/repository/readonly/ConflictGroupViewRepository.java index 0dbbdfe8..fc55f40a 100644 --- a/apps/dnet-orgs-database-application/src/main/java/eu/dnetlib/organizations/repository/readonly/ConflictGroupViewRepository.java +++ b/apps/dnet-orgs-database-application/src/main/java/eu/dnetlib/organizations/repository/readonly/ConflictGroupViewRepository.java @@ -2,7 +2,6 @@ package eu.dnetlib.organizations.repository.readonly; import java.util.List; -import org.springframework.data.domain.Pageable; import org.springframework.stereotype.Repository; import eu.dnetlib.organizations.model.view.ConflictGroupView; @@ -11,6 +10,6 @@ import eu.dnetlib.organizations.model.view.ConflictGroupViewPK; @Repository public interface ConflictGroupViewRepository extends ReadOnlyRepository { - List findByCountry1OrCountry2(String country1, String country2, Pageable page); + List findByCountry1OrCountry2(String country1, String country2); } diff --git a/apps/dnet-orgs-database-application/src/main/resources/application.properties b/apps/dnet-orgs-database-application/src/main/resources/application.properties index 83038eca..9ee04e33 100644 --- a/apps/dnet-orgs-database-application/src/main/resources/application.properties +++ b/apps/dnet-orgs-database-application/src/main/resources/application.properties @@ -49,6 +49,9 @@ openaire.api.https.proxy = 10.19.65.35 openorgs.support.pages = { "Ask a question": "https://www.openaire.eu/support/helpdesk?view=ticket&layout=open", "FAQ": "https://www.openaire.eu/faqs" } +openorgs.findDuplicatesByCountry.limit.default = 1000 +openorgs.findOrgsByStatusAndCountry.limit.default = 1000 + openaire.override.logout.url = openaire.explore.organization.baseurl = https://explore.openaire.eu/search/organization?organizationId=%s