Change in ovirt-engine[master]: core: AuditLogable - replace check to isNullOrEmpty

derez at redhat.com derez at redhat.com
Wed Apr 23 15:28:37 UTC 2014


Daniel Erez has submitted this change and it was merged.

Change subject: core: AuditLogable - replace check to isNullOrEmpty
......................................................................


core: AuditLogable - replace check to isNullOrEmpty

AuditLogable - getVmIdRef:
replace nullity check with Guid.isNullOrEmpty
as mVmId member is pre-set to Guid.Empty.
The change is needed for cases when just the
VM object is set (rather than the VmId)

Change-Id: I894592ecac51674c708c1e9aba8716e4971493cb
Signed-off-by: Daniel Erez <derez at redhat.com>
---
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/auditloghandling/AuditLogableBase.java
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Daniel Erez: Verified; Looks good to me, approved



-- 
To view, visit http://gerrit.ovirt.org/26319
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I894592ecac51674c708c1e9aba8716e4971493cb
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Liron Ar <laravot at redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list