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

gchaplik at redhat.com gchaplik at redhat.com
Tue Aug 13 12:19:45 UTC 2013


Gilad Chaplik 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:
  Gilad Chaplik: Verified; Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: If30c1e935cad06b293589c80922f56ee377375cb
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Ravi Nori <rnori at redhat.com>
Gerrit-Reviewer: Barak Azulay <bazulay at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchaplik at redhat.com>
Gerrit-Reviewer: Ravi Nori <rnori at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list