#8213 - Enter home address for existing persons #11771
Conversation
|
SonarCloud analysis: https://sonarcloud.io/dashboard?id=SORMAS-Project&pullRequest=11771 |
| List<Method> methods = Arrays.stream(location.getClass().getDeclaredMethods()) | ||
| .filter( | ||
| m -> !Modifier.isStatic(m.getModifiers()) && !Modifier.isPrivate(m.getModifiers()) && m.getName().startsWith("get") | ||
| || m.getName().startsWith("is")) |
There was a problem hiding this comment.
I think "m.getName().startsWith("get") || m.getName().startsWith("is")" should be together in separate paranthesis
| enablePersonFields(false); | ||
| person = FacadeProvider.getPersonFacade().getByUuid(pickedPerson.getUuid()); | ||
| setPerson(person); | ||
| enablePersonFields(false, true); |
There was a problem hiding this comment.
when a person is picked then all fields are disabled no matter if they are empty or not. This does not satisfy the comment added by @adinaflorea9 in the ticket.
There was a problem hiding this comment.
I think Adina's comment is out of scope for this issue because it would involve a lot more changes (e.g. this would need to be done for all create forms that include person details, not only the case create form). I've created a follow-up issue for it: #11781
|
SonarCloud analysis: https://sonarcloud.io/dashboard?id=SORMAS-Project&pullRequest=11771 |
sergiupacurariu
deleted the
change-8213-enter_home_address_existing_persons
branch
MateStrysewske
restored the
change-8213-enter_home_address_existing_persons
branch
MateStrysewske
deleted the
change-8213-enter_home_address_existing_persons
branch
Fixes #8213