ilgrosso commented on a change in pull request #188:
URL: https://github.com/apache/syncope/pull/188#discussion_r425745228
##########
File path:
ext/flowable/client-console/src/main/java/org/apache/syncope/client/console/panels/UserRequestFormDirectoryPanel.java
##########
@@ -226,8 +226,10 @@ public void onClick(final AjaxRequestTarget target, final
UserRequestForm ignore
UserRequestForm formTO = model.getObject();
UserTO newUserTO;
UserTO previousUserTO;
- if (formTO.getUserPatch() == null) {
+ if (formTO.getUserPatch() == null && formTO.getUserTO() !=
null) {
Review comment:
Maybe you can move this check right before the line below, e.g.
```java
if (newUserTO != null) {
// SYNCOPE-1563 do not use the password into UserTO
newUserTO.setPassword(null);
}
```
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
