Skip to content

Commit 9d83b6e

Browse files
loesimmensloesimmens
authored
Merge pull request #1125 from OSGP/feature/FDP-1629-web-operator-models
Feature/fdp 1629 web operator models
2 parents 5624a94 + b054c75 commit 9d83b6e

File tree

4 files changed

+52
-36
lines changed

4 files changed

+52
-36
lines changed

‎osgp/platform/osgp-adapter-ws-core/src/main/java/org/opensmartgridplatform/adapter/ws/core/application/services/FirmwareManagementService.java

Lines changed: 26 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -210,7 +210,7 @@ public void addManufacturer(
210210
this.domainHelperService.isAllowed(organisation, PlatformFunction.CREATE_MANUFACTURER);
211211

212212
final Manufacturer databaseManufacturer =
213-
this.manufacturerRepository.findByCode(manufacturer.getCode());
213+
this.manufacturerRepository.findByCodeIgnoreCase(manufacturer.getCode());
214214

215215
if (databaseManufacturer != null) {
216216
LOGGER.info("Manufacturer already exists.");
@@ -237,7 +237,7 @@ public void changeManufacturer(
237237
this.domainHelperService.isAllowed(organisation, PlatformFunction.CHANGE_MANUFACTURER);
238238

239239
final Manufacturer databaseManufacturer =
240-
this.manufacturerRepository.findByCode(manufacturer.getCode());
240+
this.manufacturerRepository.findByCodeIgnoreCase(manufacturer.getCode());
241241

242242
if (databaseManufacturer == null) {
243243
LOGGER.info("Manufacturer not found.");
@@ -268,7 +268,7 @@ public void removeManufacturer(
268268
this.domainHelperService.isAllowed(organisation, PlatformFunction.REMOVE_MANUFACTURER);
269269

270270
final Manufacturer databaseManufacturer =
271-
this.manufacturerRepository.findByCode(manufacturerCode);
271+
this.manufacturerRepository.findByCodeIgnoreCase(manufacturerCode);
272272
final List<DeviceModel> deviceModels =
273273
this.deviceModelRepository.findByManufacturer(databaseManufacturer);
274274

@@ -377,7 +377,8 @@ public void addDeviceModel(
377377
final Manufacturer manufacturer = this.findManufacturerByCode(manufacturerCode);
378378

379379
final DeviceModel savedDeviceModel =
380-
this.deviceModelRepository.findByManufacturerAndModelCode(manufacturer, modelCode);
380+
this.deviceModelRepository.findByManufacturerAndModelCodeIgnoreCase(
381+
manufacturer, modelCode);
381382

382383
if (savedDeviceModel != null) {
383384
LOGGER.info("DeviceModel already exists.");
@@ -406,9 +407,11 @@ public void removeDeviceModel(
406407
this.domainHelperService.findOrganisation(organisationIdentification);
407408
this.domainHelperService.isAllowed(organisation, PlatformFunction.REMOVE_DEVICE_MODEL);
408409

409-
final Manufacturer databaseManufacturer = this.manufacturerRepository.findByCode(manufacturer);
410+
final Manufacturer databaseManufacturer =
411+
this.manufacturerRepository.findByCodeIgnoreCase(manufacturer);
410412
final DeviceModel removedDeviceModel =
411-
this.deviceModelRepository.findByManufacturerAndModelCode(databaseManufacturer, modelCode);
413+
this.deviceModelRepository.findByManufacturerAndModelCodeIgnoreCase(
414+
databaseManufacturer, modelCode);
412415

413416
if (removedDeviceModel == null) {
414417
LOGGER.info("DeviceModel not found.");
@@ -453,9 +456,11 @@ public void changeDeviceModel(
453456
this.domainHelperService.findOrganisation(organisationIdentification);
454457
this.domainHelperService.isAllowed(organisation, PlatformFunction.CHANGE_DEVICE_MODEL);
455458

456-
final Manufacturer databaseManufacturer = this.manufacturerRepository.findByCode(manufacturer);
459+
final Manufacturer databaseManufacturer =
460+
this.manufacturerRepository.findByCodeIgnoreCase(manufacturer);
457461
final DeviceModel changedDeviceModel =
458-
this.deviceModelRepository.findByManufacturerAndModelCode(databaseManufacturer, modelCode);
462+
this.deviceModelRepository.findByManufacturerAndModelCodeIgnoreCase(
463+
databaseManufacturer, modelCode);
459464

460465
if (changedDeviceModel == null) {
461466
LOGGER.info("DeviceModel not found.");
@@ -505,9 +510,11 @@ private List<FirmwareFile> getFirmwareFiles(
505510
return new ArrayList<>();
506511
}
507512

508-
final Manufacturer manufacturer = this.manufacturerRepository.findByCode(manufacturerCode);
513+
final Manufacturer manufacturer =
514+
this.manufacturerRepository.findByCodeIgnoreCase(manufacturerCode);
509515
final DeviceModel deviceModel =
510-
this.deviceModelRepository.findByManufacturerAndModelCode(manufacturer, modelCode);
516+
this.deviceModelRepository.findByManufacturerAndModelCodeIgnoreCase(
517+
manufacturer, modelCode);
511518

512519
if (deviceModel == null) {
513520
return new ArrayList<>();
@@ -588,7 +595,8 @@ public void addFirmware(
588595
final Manufacturer databaseManufacturer = this.findManufacturerByCode(manufacturer);
589596

590597
final DeviceModel databaseDeviceModel =
591-
this.deviceModelRepository.findByManufacturerAndModelCode(databaseManufacturer, modelCode);
598+
this.deviceModelRepository.findByManufacturerAndModelCodeIgnoreCase(
599+
databaseManufacturer, modelCode);
592600

593601
if (databaseDeviceModel == null) {
594602
LOGGER.info("DeviceModel doesn't exist.");
@@ -654,7 +662,8 @@ public void addFirmware(
654662

655663
private Manufacturer findManufacturerByCode(final String manufacturer)
656664
throws FunctionalException {
657-
final Manufacturer databaseManufacturer = this.manufacturerRepository.findByCode(manufacturer);
665+
final Manufacturer databaseManufacturer =
666+
this.manufacturerRepository.findByCodeIgnoreCase(manufacturer);
658667

659668
if (databaseManufacturer == null) {
660669
LOGGER.info("Manufacturer doesn't exist.");
@@ -694,7 +703,7 @@ public void addOrChangeFirmware(
694703
this.findManufacturerByCode(deviceModel.getManufacturer());
695704

696705
final DeviceModel databaseDeviceModel =
697-
this.deviceModelRepository.findByManufacturerAndModelCode(
706+
this.deviceModelRepository.findByManufacturerAndModelCodeIgnoreCase(
698707
databaseManufacturer, deviceModel.getModelCode());
699708

700709
if (databaseDeviceModel == null) {
@@ -812,7 +821,8 @@ public void changeFirmware(
812821
.orElseThrow(
813822
supplyFirmwareFileNotFoundException(id, firmwareFileRequest.getFileName()));
814823

815-
final Manufacturer databaseManufacturer = this.manufacturerRepository.findByCode(manufacturer);
824+
final Manufacturer databaseManufacturer =
825+
this.manufacturerRepository.findByCodeIgnoreCase(manufacturer);
816826

817827
if (databaseManufacturer == null) {
818828
LOGGER.info("Manufacturer {} doesn't exist.", manufacturer);
@@ -823,7 +833,8 @@ public void changeFirmware(
823833
}
824834

825835
final DeviceModel databaseDeviceModel =
826-
this.deviceModelRepository.findByManufacturerAndModelCode(databaseManufacturer, modelCode);
836+
this.deviceModelRepository.findByManufacturerAndModelCodeIgnoreCase(
837+
databaseManufacturer, modelCode);
827838

828839
if (databaseDeviceModel == null) {
829840
LOGGER.info(

‎osgp/platform/osgp-adapter-ws-core/src/main/java/org/opensmartgridplatform/adapter/ws/core/endpoints/FirmwareManagementEndpoint.java

Lines changed: 24 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -855,23 +855,25 @@ public AddFirmwareResponse addFirmware(
855855
@RequestPayload final AddFirmwareRequest request)
856856
throws OsgpException {
857857

858-
LOGGER.info("Adding firmware:{}.", request.getFirmware().getFilename());
858+
final Firmware firmware = request.getFirmware();
859+
860+
LOGGER.info("Adding firmware:{}.", firmware.getFilename());
859861
final AddFirmwareResponse addFirmwareResponse = new AddFirmwareResponse();
860862

861863
try {
862864
final FirmwareModuleData firmwareModuleData =
863865
this.firmwareManagementMapper.map(
864-
request.getFirmware().getFirmwareModuleData(), FirmwareModuleData.class);
866+
firmware.getFirmwareModuleData(), FirmwareModuleData.class);
865867

866868
// The AddFirmwareRequest accepts multiple DeviceModels to be related to a Firmware.
867869
// This FirmwareManagementService only accepts ONE for now
868-
final String manufacturer = this.getManufacturerFromFirmware(request.getFirmware());
869-
final String modelCode = this.getModelCodeFromFirmware(request.getFirmware());
870+
final String manufacturer = this.getManufacturerFromFirmware(firmware);
871+
final String modelCode = this.getModelCodeFromFirmware(firmware);
870872

871873
this.firmwareManagementService.addFirmware(
872874
organisationIdentification,
873-
this.firmwareFileRequestFor(request.getFirmware()),
874-
request.getFirmware().getFile(),
875+
this.firmwareFileRequestFor(firmware),
876+
firmware.getFile(),
875877
manufacturer,
876878
modelCode,
877879
firmwareModuleData);
@@ -890,7 +892,7 @@ public AddFirmwareResponse addFirmware(
890892
LOGGER.error(
891893
"Exception: {} while adding firmware: {} for organisation {}",
892894
e.getMessage(),
893-
request.getFirmware().getFilename(),
895+
firmware.getFilename(),
894896
organisationIdentification,
895897
e);
896898
this.handleException(e);
@@ -917,23 +919,24 @@ public ChangeFirmwareResponse changeFirmware(
917919
@OrganisationIdentification final String organisationIdentification,
918920
@RequestPayload final ChangeFirmwareRequest request)
919921
throws OsgpException {
922+
final Firmware firmware = request.getFirmware();
920923

921-
LOGGER.info("Changing firmware:{}.", request.getFirmware().getFilename());
924+
LOGGER.info("Changing firmware:{}.", firmware.getFilename());
922925

923926
final FirmwareModuleData firmwareModuleData =
924927
this.firmwareManagementMapper.map(
925-
request.getFirmware().getFirmwareModuleData(), FirmwareModuleData.class);
928+
firmware.getFirmwareModuleData(), FirmwareModuleData.class);
926929

927930
// The ChangeFirmwareRequest accepts multiple DeviceModels to be related to a Firmware.
928931
// This FirmwareManagementService only accepts ONE for now
929-
final String manufacturer = this.getManufacturerFromFirmware(request.getFirmware());
930-
final String modelCode = this.getModelCodeFromFirmware(request.getFirmware());
932+
final String manufacturer = this.getManufacturerFromFirmware(firmware);
933+
final String modelCode = this.getModelCodeFromFirmware(firmware);
931934

932935
try {
933936
this.firmwareManagementService.changeFirmware(
934937
organisationIdentification,
935938
request.getId(),
936-
this.firmwareFileRequestFor(request.getFirmware()),
939+
this.firmwareFileRequestFor(firmware),
937940
manufacturer,
938941
modelCode,
939942
firmwareModuleData);
@@ -944,7 +947,7 @@ public ChangeFirmwareResponse changeFirmware(
944947
LOGGER.error(
945948
"Exception: {} while Changing firmware: {} for organisation {}",
946949
e.getMessage(),
947-
request.getFirmware().getFilename(),
950+
firmware.getFilename(),
948951
organisationIdentification,
949952
e);
950953
this.handleException(e);
@@ -963,19 +966,21 @@ public AddOrChangeFirmwareResponse addOrChangeFirmware(
963966
@RequestPayload final AddOrChangeFirmwareRequest request)
964967
throws OsgpException {
965968

966-
LOGGER.info("Adding Or changing firmware:{}.", request.getFirmware().getFilename());
969+
final Firmware firmware = request.getFirmware();
970+
971+
LOGGER.info("Adding Or changing firmware:{}.", firmware.getFilename());
967972
final AddOrChangeFirmwareResponse addOrChangeFirmwareResponse =
968973
new AddOrChangeFirmwareResponse();
969974

970975
try {
971976
final FirmwareModuleData firmwareModuleData =
972977
this.firmwareManagementMapper.map(
973-
request.getFirmware().getFirmwareModuleData(), FirmwareModuleData.class);
978+
firmware.getFirmwareModuleData(), FirmwareModuleData.class);
974979

975980
final List<org.opensmartgridplatform.domain.core.valueobjects.DeviceModel> deviceModels =
976981
new ArrayList<>();
977982
for (final org.opensmartgridplatform.adapter.ws.schema.core.firmwaremanagement.DeviceModel
978-
wsDeviceModel : request.getFirmware().getDeviceModels()) {
983+
wsDeviceModel : firmware.getDeviceModels()) {
979984
final org.opensmartgridplatform.domain.core.valueobjects.DeviceModel deviceModel =
980985
this.firmwareManagementMapper.map(
981986
wsDeviceModel,
@@ -985,8 +990,8 @@ public AddOrChangeFirmwareResponse addOrChangeFirmware(
985990

986991
this.firmwareManagementService.addOrChangeFirmware(
987992
organisationIdentification,
988-
this.firmwareFileRequestFor(request.getFirmware()),
989-
request.getFirmware().getFile(),
993+
this.firmwareFileRequestFor(firmware),
994+
firmware.getFile(),
990995
deviceModels,
991996
firmwareModuleData);
992997
} catch (final ConstraintViolationException e) {
@@ -1004,7 +1009,7 @@ public AddOrChangeFirmwareResponse addOrChangeFirmware(
10041009
LOGGER.error(
10051010
"Exception: {} while adding or changing firmware: {} for organisation {}",
10061011
e.getMessage(),
1007-
request.getFirmware().getFilename(),
1012+
firmware.getFilename(),
10081013
organisationIdentification,
10091014
e);
10101015
this.handleException(e);

‎osgp/platform/osgp-adapter-ws-shared-db/src/main/java/org/opensmartgridplatform/adapter/ws/shared/db/domain/repositories/writable/WritableDeviceModelRepository.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@
1212

1313
@Repository
1414
public interface WritableDeviceModelRepository extends JpaRepository<DeviceModel, Long> {
15-
DeviceModel findByManufacturerAndModelCode(Manufacturer manufacturer, String modelCode);
15+
DeviceModel findByManufacturerAndModelCodeIgnoreCase(Manufacturer manufacturer, String modelCode);
1616

1717
DeviceModel findByManufacturerCodeAndModelCode(String string, String modelCode);
1818

‎osgp/platform/osgp-adapter-ws-shared-db/src/main/java/org/opensmartgridplatform/adapter/ws/shared/db/domain/repositories/writable/WritableManufacturerRepository.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010

1111
@Repository
1212
public interface WritableManufacturerRepository extends JpaRepository<Manufacturer, Long> {
13-
Manufacturer findByCode(String code);
13+
Manufacturer findByCodeIgnoreCase(String code);
1414

1515
Manufacturer findByName(String manufacturerName);
1616
}

0 commit comments

Comments
 (0)