Arik Hadas has submitted this change and it was merged. Change subject: core: profiles: make AddUserProfile update the key ...................................................................... core: profiles: make AddUserProfile update the key To accomodate the need of REST API, we make AddUserProfileCommand detect when the client wants just to update the SSH Public key parameters, and we handle this case inside the command as special case of Update, trasparently. During the verification of this patch it was discovered that the update command was not refreshing the SSHPublicKeyID after each change. This is harmless for {UserPortal,Webadmin} UI, but incorrect for the REST API. So this patch also make sure that each succesfully update changes the key ID. Change-Id: Ia197bddb74186fa7c696f7e7f28e8fbe5e487e08 Bug-Url: https://bugzilla.redhat.com/1283499 Signed-off-by: Francesco Romani <fromani@redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddUserProfileCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateUserProfileCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UserProfilesOperationCommandBase.java 3 files changed, 49 insertions(+), 14 deletions(-) Approvals: Jenkins CI: Passed CI tests Francesco Romani: Verified Arik Hadas: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/48994 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia197bddb74186fa7c696f7e7f28e8fbe5e487e08 Gerrit-PatchSet: 9 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Francesco Romani <fromani@redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas@redhat.com> Gerrit-Reviewer: Francesco Romani <fromani@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Shahar Havivi <shavivi@redhat.com> Gerrit-Reviewer: Shmuel Leib Melamud <smelamud@redhat.com> Gerrit-Reviewer: gerrit-hooks <automation@ovirt.org>