mooli tayer has posted comments on this change.
Change subject: tools: Remove cumbersome NotificationMethod abstraction.
......................................................................
Patch Set 11:
(6 comments)
....................................................
File
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/EventSubscriptionCommandBase.java
Line 44: protected boolean
ValidateNotificationMethod(EventNotificationMethod eventNotificationMethod,
Line 45: event_subscriber
event_subscriber, DbUser user) {
Line 46: boolean retValue = true;
Line 47:
Line 48: switch (eventNotificationMethod) {
That's true. My next patch is going to add a new event notification method so
this will make sense in the future...
Line 49: case EMAIL:
Line 50: String mailAdress =
(StringUtils.isEmpty(event_subscriber.getmethod_address())) ? user.getEmail()
Line 51: : event_subscriber.getmethod_address();
Line 52:
....................................................
File
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/EventNotificationMethod.java
Line 5:
Line 6: public enum EventNotificationMethod {
Line 7: EMAIL(0);
Line 8:
Line 9: private int intValue;
Done
Line 10: private static Map<Integer, EventNotificationMethod> mappings;
Line 11:
Line 12: static {
Line 13: mappings = new HashMap<Integer, EventNotificationMethod>();
Line 6: public enum EventNotificationMethod {
Line 7: EMAIL(0);
Line 8:
Line 9: private int intValue;
Line 10: private static Map<Integer, EventNotificationMethod> mappings;
See my first comment on EventSubscriptionCommandBase.java
Line 11:
Line 12: static {
Line 13: mappings = new HashMap<Integer, EventNotificationMethod>();
Line 14: for (EventNotificationMethod value : values()) {
Line 23: public int getValue() {
Line 24: return intValue;
Line 25: }
Line 26:
Line 27: public static EventNotificationMethod forValue(int value) {
Done
Line 28: EventNotificationMethod eventNotificationMethod =
mappings.get(value);
Line 29: if (eventNotificationMethod == null){
Line 30: throw new IllegalArgumentException("Unsupported event
notification method value: " + value);
Line 31: }
....................................................
File
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/event_subscriber_id.java
Line 10: private static final long serialVersionUID = 9035847334394545216L;
Line 11:
Line 12: Guid subscriberId;
Line 13: String eventUpName;
Line 14: EventNotificationMethod notificationMethod;
Nop. will do in a different patch set since it's not relevant to this one.
Line 15: String tagName;
Line 16:
Line 17: @Override
Line 18: public int hashCode() {
....................................................
File
backend/manager/tools/src/main/java/org/ovirt/engine/core/notifier/NotificationMethodsMapper.java
Line 19: public EventSender getEventSender(EventNotificationMethod method) {
Line 20: return eventSenders.get(method);
Line 21: }
Line 22:
Line 23: public EventSender getEventSender(int intValue) {
Done
Line 24: return
eventSenders.get(EventNotificationMethod.forValue(intValue));
Line 25: }
--
To view, visit http://gerrit.ovirt.org/22135
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I8b71c4e78bbdca3d02d2ac4ef419b9d3d7d58761
Gerrit-PatchSet: 11
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: mooli tayer <mta...@redhat.com>
Gerrit-Reviewer: Barak Azulay <bazu...@redhat.com>
Gerrit-Reviewer: Eli Mesika <elimes...@gmail.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernan...@redhat.com>
Gerrit-Reviewer: Martin Peřina <mper...@redhat.com>
Gerrit-Reviewer: Moti Asayag <masa...@redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzasl...@redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybron...@redhat.com>
Gerrit-Reviewer: mooli tayer <mta...@redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches
