From fc30064a21e6a8618022eeb54c4962c2bd9c5ba2 Mon Sep 17 00:00:00 2001
From: Niko Ehrenfeuchter <nikolaus.ehrenfeuchter@unibas.ch>
Date: Thu, 15 Feb 2018 12:39:32 +0100
Subject: [PATCH] Make constructors of ServiceConfig and ServiceStatus private.
Those objects are not meant to be instantiated directly but only using
the Deserialize() methods which takes care of creating a new object in
case deserializing fails for the status class.
---
ATxCommon/Serializables/ServiceConfig.cs | 2 +-
ATxCommon/Serializables/ServiceStatus.cs | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/ATxCommon/Serializables/ServiceConfig.cs b/ATxCommon/Serializables/ServiceConfig.cs
index c451713..bf6f96a 100644
--- a/ATxCommon/Serializables/ServiceConfig.cs
+++ b/ATxCommon/Serializables/ServiceConfig.cs
@@ -224,9 +224,9 @@ namespace ATxCommon.Serializables
/// <summary>
- public ServiceConfig() {
/// ServiceConfig constructor, currently empty.
/// </summary>
+ private ServiceConfig() {
Log.Trace("ServiceConfig() constructor.");
}
diff --git a/ATxCommon/Serializables/ServiceStatus.cs b/ATxCommon/Serializables/ServiceStatus.cs
index e7321a7..a7082c0 100644
--- a/ATxCommon/Serializables/ServiceStatus.cs
+++ b/ATxCommon/Serializables/ServiceStatus.cs
@@ -40,7 +40,7 @@ namespace ATxCommon.Serializables
/// <summary>
/// The constructor, setting default values.
/// </summary>
- public ServiceStatus() {
+ private ServiceStatus() {
_currentTransferSrc = "";
_currentTargetTmp = "";
_transferInProgress = false;
--
GitLab