Skip to content

Commit 5624a94

Browse files
authored
Merge pull request #1124 from OSGP/feature/SMHE-1850-remove-translateDlmsObjectType
Feature/smhe 1850 remove translate dlms object type
2 parents dffdbcd + aec0bf9 commit 5624a94

20 files changed

+54
-82
lines changed

osgp/protocol-adapter-dlms/osgp-protocol-adapter-dlms/src/main/java/org/opensmartgridplatform/adapter/protocol/dlms/domain/commands/alarm/AlarmHelperService.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,8 +9,8 @@
99
import java.util.Map;
1010
import java.util.Set;
1111
import java.util.stream.Collectors;
12-
import org.opensmartgridplatform.adapter.protocol.dlms.domain.commands.dlmsobjectconfig.DlmsObjectType;
1312
import org.opensmartgridplatform.adapter.protocol.dlms.domain.commands.utils.ByteRegisterConverter;
13+
import org.opensmartgridplatform.dlms.objectconfig.DlmsObjectType;
1414
import org.opensmartgridplatform.dto.valueobjects.smartmetering.AlarmTypeDto;
1515
import org.springframework.stereotype.Service;
1616

osgp/protocol-adapter-dlms/osgp-protocol-adapter-dlms/src/main/java/org/opensmartgridplatform/adapter/protocol/dlms/domain/commands/alarm/AlarmTypeRegisterLookup.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@
88
import java.util.Set;
99
import java.util.stream.Collectors;
1010
import lombok.Getter;
11-
import org.opensmartgridplatform.adapter.protocol.dlms.domain.commands.dlmsobjectconfig.DlmsObjectType;
11+
import org.opensmartgridplatform.dlms.objectconfig.DlmsObjectType;
1212
import org.opensmartgridplatform.dto.valueobjects.smartmetering.AlarmTypeDto;
1313

1414
/**

osgp/protocol-adapter-dlms/osgp-protocol-adapter-dlms/src/main/java/org/opensmartgridplatform/adapter/protocol/dlms/domain/commands/alarm/ClearAlarmRegisterCommandExecutor.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,14 +12,14 @@
1212
import org.openmuc.jdlms.SetParameter;
1313
import org.openmuc.jdlms.datatypes.DataObject;
1414
import org.opensmartgridplatform.adapter.protocol.dlms.domain.commands.AbstractCommandExecutor;
15-
import org.opensmartgridplatform.adapter.protocol.dlms.domain.commands.dlmsobjectconfig.DlmsObjectType;
1615
import org.opensmartgridplatform.adapter.protocol.dlms.domain.commands.utils.JdlmsObjectToStringUtil;
1716
import org.opensmartgridplatform.adapter.protocol.dlms.domain.commands.utils.ObjectConfigServiceHelper;
1817
import org.opensmartgridplatform.adapter.protocol.dlms.domain.entities.DlmsDevice;
1918
import org.opensmartgridplatform.adapter.protocol.dlms.domain.entities.Protocol;
2019
import org.opensmartgridplatform.adapter.protocol.dlms.domain.factories.DlmsConnectionManager;
2120
import org.opensmartgridplatform.adapter.protocol.dlms.exceptions.ConnectionException;
2221
import org.opensmartgridplatform.adapter.protocol.dlms.exceptions.ProtocolAdapterException;
22+
import org.opensmartgridplatform.dlms.objectconfig.DlmsObjectType;
2323
import org.opensmartgridplatform.dto.valueobjects.smartmetering.ActionRequestDto;
2424
import org.opensmartgridplatform.dto.valueobjects.smartmetering.ActionResponseDto;
2525
import org.opensmartgridplatform.dto.valueobjects.smartmetering.ClearAlarmRegisterRequestDto;

osgp/protocol-adapter-dlms/osgp-protocol-adapter-dlms/src/main/java/org/opensmartgridplatform/adapter/protocol/dlms/domain/commands/alarm/ClearMBusStatusOnAllChannelsCommandExecutor.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,6 @@
1616
import org.openmuc.jdlms.SetParameter;
1717
import org.openmuc.jdlms.datatypes.DataObject;
1818
import org.opensmartgridplatform.adapter.protocol.dlms.domain.commands.AbstractCommandExecutor;
19-
import org.opensmartgridplatform.adapter.protocol.dlms.domain.commands.dlmsobjectconfig.DlmsObjectType;
2019
import org.opensmartgridplatform.adapter.protocol.dlms.domain.commands.utils.JdlmsObjectToStringUtil;
2120
import org.opensmartgridplatform.adapter.protocol.dlms.domain.commands.utils.ObjectConfigServiceHelper;
2221
import org.opensmartgridplatform.adapter.protocol.dlms.domain.entities.DlmsDevice;
@@ -25,6 +24,7 @@
2524
import org.opensmartgridplatform.adapter.protocol.dlms.exceptions.ConnectionException;
2625
import org.opensmartgridplatform.adapter.protocol.dlms.exceptions.ProtocolAdapterException;
2726
import org.opensmartgridplatform.dlms.interfaceclass.method.MBusClientMethod;
27+
import org.opensmartgridplatform.dlms.objectconfig.DlmsObjectType;
2828
import org.opensmartgridplatform.dto.valueobjects.smartmetering.ActionRequestDto;
2929
import org.opensmartgridplatform.dto.valueobjects.smartmetering.ActionResponseDto;
3030
import org.opensmartgridplatform.dto.valueobjects.smartmetering.ClearMBusStatusOnAllChannelsRequestDto;

osgp/protocol-adapter-dlms/osgp-protocol-adapter-dlms/src/main/java/org/opensmartgridplatform/adapter/protocol/dlms/domain/commands/alarm/ReadAlarmRegisterCommandExecutor.java

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -13,14 +13,14 @@
1313
import org.openmuc.jdlms.GetResult;
1414
import org.openmuc.jdlms.datatypes.DataObject;
1515
import org.opensmartgridplatform.adapter.protocol.dlms.domain.commands.AbstractCommandExecutor;
16-
import org.opensmartgridplatform.adapter.protocol.dlms.domain.commands.dlmsobjectconfig.DlmsObjectType;
1716
import org.opensmartgridplatform.adapter.protocol.dlms.domain.commands.utils.JdlmsObjectToStringUtil;
1817
import org.opensmartgridplatform.adapter.protocol.dlms.domain.commands.utils.ObjectConfigServiceHelper;
1918
import org.opensmartgridplatform.adapter.protocol.dlms.domain.entities.DlmsDevice;
2019
import org.opensmartgridplatform.adapter.protocol.dlms.domain.entities.Protocol;
2120
import org.opensmartgridplatform.adapter.protocol.dlms.domain.factories.DlmsConnectionManager;
2221
import org.opensmartgridplatform.adapter.protocol.dlms.exceptions.ConnectionException;
2322
import org.opensmartgridplatform.adapter.protocol.dlms.exceptions.ProtocolAdapterException;
23+
import org.opensmartgridplatform.dlms.objectconfig.DlmsObjectType;
2424
import org.opensmartgridplatform.dto.valueobjects.smartmetering.ActionRequestDto;
2525
import org.opensmartgridplatform.dto.valueobjects.smartmetering.AlarmRegisterResponseDto;
2626
import org.opensmartgridplatform.dto.valueobjects.smartmetering.AlarmTypeDto;
@@ -29,7 +29,6 @@
2929
import org.opensmartgridplatform.shared.infra.jms.MessageMetadata;
3030
import org.slf4j.Logger;
3131
import org.slf4j.LoggerFactory;
32-
import org.springframework.beans.factory.annotation.Autowired;
3332
import org.springframework.stereotype.Component;
3433

3534
@Component
@@ -41,12 +40,14 @@ public class ReadAlarmRegisterCommandExecutor
4140

4241
final ObjectConfigServiceHelper objectConfigServiceHelper;
4342

44-
@Autowired private AlarmHelperService alarmHelperService;
43+
private final AlarmHelperService alarmHelperService;
4544

4645
public ReadAlarmRegisterCommandExecutor(
47-
final ObjectConfigServiceHelper objectConfigServiceHelper) {
46+
final ObjectConfigServiceHelper objectConfigServiceHelper,
47+
final AlarmHelperService alarmHelperService) {
4848
super(ReadAlarmRegisterDataDto.class);
4949
this.objectConfigServiceHelper = objectConfigServiceHelper;
50+
this.alarmHelperService = alarmHelperService;
5051
}
5152

5253
@Override

osgp/protocol-adapter-dlms/osgp-protocol-adapter-dlms/src/main/java/org/opensmartgridplatform/adapter/protocol/dlms/domain/commands/alarm/SetAlarmNotificationsCommandExecutor.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,14 +16,14 @@
1616
import org.openmuc.jdlms.SetParameter;
1717
import org.openmuc.jdlms.datatypes.DataObject;
1818
import org.opensmartgridplatform.adapter.protocol.dlms.domain.commands.AbstractCommandExecutor;
19-
import org.opensmartgridplatform.adapter.protocol.dlms.domain.commands.dlmsobjectconfig.DlmsObjectType;
2019
import org.opensmartgridplatform.adapter.protocol.dlms.domain.commands.utils.JdlmsObjectToStringUtil;
2120
import org.opensmartgridplatform.adapter.protocol.dlms.domain.commands.utils.ObjectConfigServiceHelper;
2221
import org.opensmartgridplatform.adapter.protocol.dlms.domain.entities.DlmsDevice;
2322
import org.opensmartgridplatform.adapter.protocol.dlms.domain.entities.Protocol;
2423
import org.opensmartgridplatform.adapter.protocol.dlms.domain.factories.DlmsConnectionManager;
2524
import org.opensmartgridplatform.adapter.protocol.dlms.exceptions.ConnectionException;
2625
import org.opensmartgridplatform.adapter.protocol.dlms.exceptions.ProtocolAdapterException;
26+
import org.opensmartgridplatform.dlms.objectconfig.DlmsObjectType;
2727
import org.opensmartgridplatform.dto.valueobjects.smartmetering.ActionRequestDto;
2828
import org.opensmartgridplatform.dto.valueobjects.smartmetering.ActionResponseDto;
2929
import org.opensmartgridplatform.dto.valueobjects.smartmetering.AlarmNotificationDto;

osgp/protocol-adapter-dlms/osgp-protocol-adapter-dlms/src/main/java/org/opensmartgridplatform/adapter/protocol/dlms/domain/commands/alarm/TestAlarmSchedulerCommandExecutor.java

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,6 @@
1717
import org.openmuc.jdlms.datatypes.CosemTime;
1818
import org.openmuc.jdlms.datatypes.DataObject;
1919
import org.opensmartgridplatform.adapter.protocol.dlms.domain.commands.AbstractCommandExecutor;
20-
import org.opensmartgridplatform.adapter.protocol.dlms.domain.commands.dlmsobjectconfig.DlmsObjectType;
2120
import org.opensmartgridplatform.adapter.protocol.dlms.domain.commands.utils.DlmsDateTimeConverter;
2221
import org.opensmartgridplatform.adapter.protocol.dlms.domain.commands.utils.ObjectConfigServiceHelper;
2322
import org.opensmartgridplatform.adapter.protocol.dlms.domain.entities.DlmsDevice;
@@ -26,23 +25,25 @@
2625
import org.opensmartgridplatform.adapter.protocol.dlms.exceptions.ConnectionException;
2726
import org.opensmartgridplatform.adapter.protocol.dlms.exceptions.ProtocolAdapterException;
2827
import org.opensmartgridplatform.dlms.interfaceclass.attribute.SingleActionScheduleAttribute;
28+
import org.opensmartgridplatform.dlms.objectconfig.DlmsObjectType;
2929
import org.opensmartgridplatform.dto.valueobjects.smartmetering.ActionRequestDto;
3030
import org.opensmartgridplatform.dto.valueobjects.smartmetering.ActionResponseDto;
3131
import org.opensmartgridplatform.dto.valueobjects.smartmetering.TestAlarmSchedulerRequestDto;
3232
import org.opensmartgridplatform.dto.valueobjects.smartmetering.TestAlarmTypeDto;
3333
import org.opensmartgridplatform.shared.infra.jms.MessageMetadata;
34-
import org.springframework.beans.factory.annotation.Autowired;
3534
import org.springframework.stereotype.Component;
3635

3736
@Component
3837
@Slf4j
3938
public class TestAlarmSchedulerCommandExecutor
4039
extends AbstractCommandExecutor<TestAlarmSchedulerRequestDto, AccessResultCode> {
4140

42-
@Autowired private ObjectConfigServiceHelper objectConfigServiceHelper;
41+
private final ObjectConfigServiceHelper objectConfigServiceHelper;
4342

44-
public TestAlarmSchedulerCommandExecutor() {
43+
public TestAlarmSchedulerCommandExecutor(
44+
final ObjectConfigServiceHelper objectConfigServiceHelper) {
4545
super(TestAlarmSchedulerRequestDto.class);
46+
this.objectConfigServiceHelper = objectConfigServiceHelper;
4647
}
4748

4849
@Override

osgp/protocol-adapter-dlms/osgp-protocol-adapter-dlms/src/main/java/org/opensmartgridplatform/adapter/protocol/dlms/domain/commands/utils/ObjectConfigServiceHelper.java

Lines changed: 6 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -7,13 +7,13 @@
77
import java.util.Optional;
88
import org.openmuc.jdlms.AttributeAddress;
99
import org.openmuc.jdlms.ObisCode;
10-
import org.opensmartgridplatform.adapter.protocol.dlms.domain.commands.dlmsobjectconfig.DlmsObjectType;
1110
import org.opensmartgridplatform.adapter.protocol.dlms.domain.entities.DlmsDevice;
1211
import org.opensmartgridplatform.adapter.protocol.dlms.domain.entities.Protocol;
1312
import org.opensmartgridplatform.adapter.protocol.dlms.exceptions.ProtocolAdapterException;
1413
import org.opensmartgridplatform.dlms.exceptions.ObjectConfigException;
1514
import org.opensmartgridplatform.dlms.objectconfig.Attribute;
1615
import org.opensmartgridplatform.dlms.objectconfig.CosemObject;
16+
import org.opensmartgridplatform.dlms.objectconfig.DlmsObjectType;
1717
import org.opensmartgridplatform.dlms.services.ObjectConfigService;
1818
import org.springframework.stereotype.Component;
1919

@@ -44,8 +44,7 @@ public ObjectConfigServiceHelper(final ObjectConfigService objectConfigService)
4444
* else Optional.empty()
4545
*/
4646
public Optional<AttributeAddress> findOptionalDefaultAttributeAddress(
47-
final Protocol protocol, final DlmsObjectType dlmsObjectType, final Integer channel)
48-
throws ProtocolAdapterException {
47+
final Protocol protocol, final DlmsObjectType dlmsObjectType, final Integer channel) {
4948

5049
return this.findOptionalAttributeAddress(
5150
protocol, dlmsObjectType, channel, DEFAULT_ATTRIBUTE_ID);
@@ -61,8 +60,7 @@ public Optional<AttributeAddress> findOptionalDefaultAttributeAddress(
6160
* else Optional.empty()
6261
*/
6362
public Optional<AttributeAddress> findOptionalDefaultAttributeAddress(
64-
final Protocol protocol, final DlmsObjectType dlmsObjectType)
65-
throws ProtocolAdapterException {
63+
final Protocol protocol, final DlmsObjectType dlmsObjectType) {
6664

6765
return this.findOptionalAttributeAddress(protocol, dlmsObjectType, null, DEFAULT_ATTRIBUTE_ID);
6866
}
@@ -83,8 +81,7 @@ public Optional<AttributeAddress> findOptionalAttributeAddress(
8381
final Protocol protocol,
8482
final DlmsObjectType dlmsObjectType,
8583
final Integer channel,
86-
final int attributeId)
87-
throws ProtocolAdapterException {
84+
final int attributeId) {
8885

8986
final Optional<CosemObject> optObject =
9087
this.getOptionalCosemObject(protocol.getName(), protocol.getVersion(), dlmsObjectType);
@@ -103,12 +100,10 @@ public Optional<AttributeAddress> findOptionalAttributeAddress(
103100
}
104101

105102
private Optional<CosemObject> getOptionalCosemObject(
106-
final String protocol, final String protocolVersion, final DlmsObjectType objectType)
107-
throws ProtocolAdapterException {
103+
final String protocol, final String protocolVersion, final DlmsObjectType objectType) {
108104

109105
try {
110-
return this.objectConfigService.getOptionalCosemObject(
111-
protocol, protocolVersion, translateDlmsObjectType(objectType));
106+
return this.objectConfigService.getOptionalCosemObject(protocol, protocolVersion, objectType);
112107
} catch (final ObjectConfigException e) {
113108
return Optional.empty();
114109
}
@@ -169,17 +164,6 @@ public AttributeAddress findAttributeAddress(
169164
});
170165
}
171166

172-
private static org.opensmartgridplatform.dlms.objectconfig.DlmsObjectType translateDlmsObjectType(
173-
final DlmsObjectType objectType) throws ProtocolAdapterException {
174-
try {
175-
return org.opensmartgridplatform.dlms.objectconfig.DlmsObjectType.valueOf(objectType.name());
176-
} catch (final IllegalArgumentException e) {
177-
final String message =
178-
String.format("Cannot translate the DlmsObjectType with name %s", objectType);
179-
throw new ProtocolAdapterException(message, e);
180-
}
181-
}
182-
183167
private ObisCode replaceChannel(String obisCode, final Integer channel) {
184168
if (channel != null) {
185169
obisCode = obisCode.replace("x", channel.toString());

osgp/protocol-adapter-dlms/osgp-protocol-adapter-dlms/src/main/java/org/opensmartgridplatform/adapter/protocol/dlms/infra/networking/AlarmDecoder.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,8 +9,8 @@
99
import java.util.Arrays;
1010
import java.util.Set;
1111
import org.opensmartgridplatform.adapter.protocol.dlms.domain.commands.alarm.AlarmHelperService;
12-
import org.opensmartgridplatform.adapter.protocol.dlms.domain.commands.dlmsobjectconfig.DlmsObjectType;
1312
import org.opensmartgridplatform.dlms.DlmsPushNotification;
13+
import org.opensmartgridplatform.dlms.objectconfig.DlmsObjectType;
1414
import org.opensmartgridplatform.dto.valueobjects.smartmetering.AlarmTypeDto;
1515

1616
public class AlarmDecoder {

osgp/protocol-adapter-dlms/osgp-protocol-adapter-dlms/src/main/java/org/opensmartgridplatform/adapter/protocol/dlms/infra/networking/Dsmr4AlarmDecoder.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,9 +14,9 @@
1414
import java.nio.charset.StandardCharsets;
1515
import java.util.Arrays;
1616
import org.opensmartgridplatform.adapter.protocol.dlms.domain.commands.dlmsobjectconfig.DlmsObjectConfigDsmr422;
17-
import org.opensmartgridplatform.adapter.protocol.dlms.domain.commands.dlmsobjectconfig.DlmsObjectType;
1817
import org.opensmartgridplatform.adapter.protocol.dlms.exceptions.ProtocolAdapterException;
1918
import org.opensmartgridplatform.dlms.DlmsPushNotification;
19+
import org.opensmartgridplatform.dlms.objectconfig.DlmsObjectType;
2020
import org.slf4j.Logger;
2121
import org.slf4j.LoggerFactory;
2222

0 commit comments

Comments
 (0)