Change in ovirt-engine[ovirt-engine-3.3]: webadmin: Error when changing email address for user event n...

yzaslavs at redhat.com yzaslavs at redhat.com
Sun Aug 25 09:36:12 UTC 2013


Yair Zaslavsky has submitted this change and it was merged.

Change subject: webadmin: Error when changing email address for user event notification
......................................................................


webadmin: Error when changing email address for user event notification

When changing user event notification email, the
web admin portal complains that the user is already
subscribed to the events.

This happens because we are executing 2 AsyncCalls from
GWT to backend RemoveEventSubscription and AddEventSubscription.
If all RemoveEventSubscription calls do not complete
before AddEventSubscription action is called the exception
is thrown.

Change-Id: If30c1e935cad06b293589c80922f56ee377375cb
Bug-Url: https://bugzilla.redhat.com/985635
Signed-off-by: Ravi Nori <rnori at redhat.com>
---
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/users/UserEventNotifierListModel.java
1 file changed, 30 insertions(+), 4 deletions(-)

Approvals:
  Yair Zaslavsky: Verified; Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: If30c1e935cad06b293589c80922f56ee377375cb
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.3
Gerrit-Owner: Yair Zaslavsky <yzaslavs at redhat.com>
Gerrit-Reviewer: Ravi Nori <rnori at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list