Tal Nisan has submitted this change and it was merged.
Change subject: webadmin: Fix a bug in edit storage domain
......................................................................
webadmin: Fix a bug in edit storage domain
Patch I4e0d5a1c9158be1284ffe05d1bf1397c9113f6c0 introduced the
wipe_after_delete column in storage_domain_static table with a NOT NULL
constraint.
The edit storage domain operation in webadmin clones the editted object and
sends it to the engine for the entity updated, however, the clone storage
domain method did not clone the item correctly and omitted the wipe after
delete property from the clone thus sending null instead of the original
value, causing the update to fail with a NOT NULL constraint
Change-Id: Ia49b3460af6231356b7255d8b6322c98eef59406
Bug-url:
https://bugzilla.redhat.com/1177931
Signed-off-by: Tal Nisan <tnisan(a)redhat.com>
---
M
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/Cloner.java
1 file changed, 1 insertion(+), 0 deletions(-)
Approvals:
Tal Nisan: Verified
Allon Mureinik: Looks good to me, approved
--
To view, visit
http://gerrit.ovirt.org/36496
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ia49b3460af6231356b7255d8b6322c98eef59406
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Idan Shaby <ishaby(a)redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: Vered Volansky <vvolansk(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org