Tal Nisan has uploaded a new change for review. Change subject: core: Implement toString in StorageDomain and StoragePool ......................................................................
core: Implement toString in StorageDomain and StoragePool Change-Id: I34a70ea21e6c5996d430eef013c5d93360aa04e4 Bug-Url: https://bugzilla.redhat.com/1142198 Signed-off-by: Tal Nisan <[email protected]> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/StorageDomain.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/StoragePool.java 2 files changed, 14 insertions(+), 0 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/77/32977/1 diff --git a/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/StorageDomain.java b/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/StorageDomain.java index 4d0af90..307d94c 100644 --- a/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/StorageDomain.java +++ b/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/StorageDomain.java @@ -348,4 +348,13 @@ && storageDomainOverCommitPercent == other.storageDomainOverCommitPercent && ObjectUtils.objectsEqual(totalDiskSize, other.totalDiskSize)); } + + @Override + public String toString() { + // Since the static data arrives from external source it's not guarenteed not to be null so a null check is + // mandatory in order to avoid NPE when invoking toString by the logger + String domainName = staticData.getName() == null ? "null" : staticData.getName(); + String domainId = staticData.getId() == null ? "null" : staticData.getId().toString(); + return "StorageDomain[" + domainName + ", " + domainId + "]"; + } } diff --git a/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/StoragePool.java b/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/StoragePool.java index d19d678..527fc0d 100644 --- a/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/StoragePool.java +++ b/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/StoragePool.java @@ -243,4 +243,9 @@ public void setMacPoolId(Guid macPoolId) { this.macPoolId = macPoolId; } + + @Override + public String toString() { + return "StoragePool[" + getName() + ", " + getId() + "]"; + } } -- To view, visit http://gerrit.ovirt.org/32977 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I34a70ea21e6c5996d430eef013c5d93360aa04e4 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tal Nisan <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
