Allon Mureinik has submitted this change and it was merged. Change subject: core: UpdateStorageDomain's CDA early return ......................................................................
core: UpdateStorageDomain's CDA early return Changed UpdateStorageDomainCommand.canDoAction() to use the early return pattern in order to solve several NullPointerExceptions and enhance readability. Tests were added to all the canDoAction() flows to make sure these changes did not break any functionality. Change-Id: Iea10984090a759ef07da1fe572c4fdc323ede0e5 Signed-off-by: Allon Mureinik <[email protected]> Signed-off-by: Idan Shaby <[email protected]> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/UpdateStorageDomainCommand.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/storage/UpdateStorageDomainCommandTest.java 2 files changed, 97 insertions(+), 18 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Allon Mureinik: Verified -- To view, visit http://gerrit.ovirt.org/29809 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iea10984090a759ef07da1fe572c4fdc323ede0e5 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Daniel Erez <[email protected]> Gerrit-Reviewer: Idan Shaby <[email protected]> Gerrit-Reviewer: Maor Lipchuk <[email protected]> Gerrit-Reviewer: Tal Nisan <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
