Oved Ourfali has submitted this change and it was merged. Change subject: api: ovirt-engine-restapi : two permissions has the same permit ID (#859233) ......................................................................
api: ovirt-engine-restapi : two permissions has the same permit ID (#859233) https://bugzilla.redhat.com/show_bug.cgi?id=859233 This patch removes the id and the role from the PermitType enum, adding suitable mappers, and changing all dependant code accordingly. Change-Id: I578e227b6e4600715e0ca7a8dfb4dbad79df15b4 Signed-off-by: Oved Ourfali <oourf...@redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/model/PermitType.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/validation/PermitValidator.java M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/PermitMapper.java M backend/manager/modules/restapi/types/src/test/java/org/ovirt/engine/api/restapi/types/PermitMapperTest.java 4 files changed, 183 insertions(+), 79 deletions(-) Approvals: Oved Ourfali: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8128 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I578e227b6e4600715e0ca7a8dfb4dbad79df15b4 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Oved Ourfali <oourf...@redhat.com> Gerrit-Reviewer: Michael Pasternak <mpast...@redhat.com> Gerrit-Reviewer: Ori Liel <ol...@redhat.com> Gerrit-Reviewer: Oved Ourfali <oourf...@redhat.com> _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
