Eli Mesika has submitted this change and it was merged. Change subject: tools: notifier: more changes in NotificationProperties ......................................................................
tools: notifier: more changes in NotificationProperties split validate into sub calls (validateBasic, validateEmail ...) to sections. as more sections are expected in following commits (validateFilter, validateSnmp ). Order methods (constructor befor any other method). Apply code style & add requireAll and requireOne methods. Change-Id: I2ebbf76a23c79d354cb9026dc3ad0f4310a804be Signed-off-by: Mooli Tayer <[email protected]> --- M backend/manager/tools/src/main/java/org/ovirt/engine/core/notifier/utils/NotificationProperties.java 1 file changed, 141 insertions(+), 89 deletions(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, but someone else must approve Eli Mesika: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/24297 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2ebbf76a23c79d354cb9026dc3ad0f4310a804be Gerrit-PatchSet: 9 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: mooli tayer <[email protected]> Gerrit-Reviewer: Alon Bar-Lev <[email protected]> Gerrit-Reviewer: Eli Mesika <[email protected]> Gerrit-Reviewer: Martin Peřina <[email protected]> Gerrit-Reviewer: Yair Zaslavsky <[email protected]> Gerrit-Reviewer: mooli tayer <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
