Fix nested composite cut operations

[Meklo]

PR Summary

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 554280e2-eda3-4971-be74-431b003a4d0d

📥 Commits

Reviewing files that changed from the base of the PR and between 4776b96 and 05db9be.

📒 Files selected for processing (1)

• src/main/java/org/gridsuite/modification/server/service/NetworkModificationService.java

🚧 Files skipped from review as they are similar to previous changes (1)

• src/main/java/org/gridsuite/modification/server/service/NetworkModificationService.java

---

📝 Walkthrough

Walkthrough

moveModifications now always creates a mutable child set before removing selected composite UUIDs, so the later filtering step can run in both empty and non-empty selection cases.

Changes

Network modification child selection

Layer / File(s)	Summary
Mutable children set src/main/java/org/gridsuite/modification/server/service/NetworkModificationService.java	childrenOfSelectedComposites is wrapped in a mutable HashSet before removeAll(selectedCompositeUuids) runs, while keeping the same child-resolution and sub-modification UUID logic.

Suggested reviewers

• souissimai

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.
Description check	❓ Inconclusive	The description is just a template placeholder and does not meaningfully describe the changeset.	Replace the placeholder with a brief summary of what changed, why it changed, and any reviewer notes.

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly matches the main change: fixing nested composite cut operations in network modifications.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

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

See analysis details on SonarQube Cloud

[souissimai]

code and test OK