From 65e1b608e334eadc69ec5ffd4ad9ac4044828c8c Mon Sep 17 00:00:00 2001 From: Paul Grey Date: Wed, 30 Nov 2022 17:41:15 -0500 Subject: [PATCH] NIFI-10916 - Controller Service allowable values dropdown list should be sorted --- .../src/main/java/org/apache/nifi/web/api/dto/DtoFactory.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/api/dto/DtoFactory.java b/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/api/dto/DtoFactory.java index d28f9d93a2a2..2039eab5a251 100644 --- a/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/api/dto/DtoFactory.java +++ b/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/api/dto/DtoFactory.java @@ -4225,7 +4225,6 @@ public PropertyDescriptorDTO createPropertyDescriptorDto(final PropertyDescripto } else { final List allowableValues = new ArrayList<>(); final List controllerServiceIdentifiers = new ArrayList<>(controllerServiceProvider.getControllerServiceIdentifiers(serviceDefinition, groupId)); - Collections.sort(controllerServiceIdentifiers, Collator.getInstance(Locale.US)); for (final String serviceIdentifier : controllerServiceIdentifiers) { final ControllerServiceNode service = controllerServiceProvider.getControllerServiceNode(serviceIdentifier); final boolean isServiceAuthorized = service.isAuthorized(authorizer, RequestAction.READ, NiFiUserUtils.getNiFiUser()); @@ -4236,6 +4235,7 @@ public PropertyDescriptorDTO createPropertyDescriptorDto(final PropertyDescripto allowableValue.setValue(serviceIdentifier); allowableValues.add(entityFactory.createAllowableValueEntity(allowableValue, isServiceAuthorized)); } + allowableValues.sort(Comparator.comparing(e -> e.getAllowableValue().getDisplayName())); dto.setAllowableValues(allowableValues); } } else { @@ -4247,7 +4247,7 @@ public PropertyDescriptorDTO createPropertyDescriptorDto(final PropertyDescripto allowableValueDto.setDescription(allowableValue.getDescription()); allowableValues.add(entityFactory.createAllowableValueEntity(allowableValueDto, true)); } - + allowableValues.sort(Comparator.comparing(e -> e.getAllowableValue().getDisplayName())); dto.setAllowableValues(allowableValues); }