Fix three null-safety bugs introduced with the map-based endpoints refactor

[pathob]

• setUsers passed map values directly to setUser, which called findUser with a null username when the UserModel had no username field set; now passes the map key as a fallback username via a new setUser(directoryId, username, model) overload
• addUserToGroups silently omitted groups from the response when the user was already a member (MembershipAlreadyExistsException); the group is now included
• ApplicationLinkModelUtil.toApplicationLinkDetails passed a null Boolean to ApplicationLinkDetails.Builder.isPrimary(boolean), causing NPE when the primary field is absent in the request; now defaults to false via Boolean.TRUE.equals()

[Copilot]

Pull request overview

This PR addresses three null-safety/edge-case regressions introduced by the map-based endpoints refactor in the Crowd users service and commons application link utilities.

Changes:

• Added setUser(directoryId, username, model) and updated setUsers(...) to pass the map key as the username fallback.
• Updated group-assignment behavior so groups are included in the response even when membership already exists.
• Made ApplicationLinkModelUtil.toApplicationLinkDetails(...) null-safe for primary by defaulting to false.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

File	Description
crowd/src/main/java/com/deftdevs/bootstrapi/crowd/service/UsersServiceImpl.java	Adds username-aware setUser overload, updates setUsers to use map keys, and includes already-existing group memberships in results.
crowd/src/test/java/com/deftdevs/bootstrapi/crowd/service/UsersServiceTest.java	Updates mocks for new overloads and adds tests for the new behavior (with a noted coverage gap for the setUsers null-username regression).
commons/src/main/java/com/deftdevs/bootstrapi/commons/model/util/ApplicationLinkModelUtil.java	Prevents NPE by converting nullable Boolean primary into a primitive boolean safely.
commons/src/test/java/com/deftdevs/bootstrapi/commons/model/util/ApplicationLinkModelUtilTest.java	Adds a regression test asserting primary == null results in isPrimary() == false.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud