Alfresco ChangeWizardFormOwnerActionExecuter User Query Issue - Githubissues

tsgrp / HPI

OpenContent Management Suite (OCMS)

http://www.tsgrp.com/products

17 stars 5 forks source link

Alfresco ChangeWizardFormOwnerActionExecuter User Query Issue #2333

Closed mwojno-tsg closed 3 years ago

mwojno-tsg commented 3 years ago

Overview

The ChangeWizardFormOwnerActionExecuter for Alfresco expects the oldOwner and newOwner variables to be the login name being passed from OCMS.
However, the display name is passed for those variables. The user query it was running is deprecated and it was only searching on login name.

Solution

A better way to do this would be to use the UserUtil.getOCAuthorityUser instead. This will first search on the login name, then if that returns null it will run the search based on the display name.
This way we can run and allow both options of searching on display name and login name just to be sure.

mwojno-tsg commented 3 years ago

Closed with OC revision: 26553. This commit addresses the issue with change wizard form owner only expected username to come from OCMS when it is valid in alfresco to come back as display name. I am taking into account both username and display name when doing the query and setting the new owner name. Removed deprecated function calls as well.

CR: gsteimer, dgrumieaux