add reportId when recreating network

[EtienneLt]

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 Plus

Run ID: 92674baf-c50d-4481-9fe0-a38da57eebba

📥 Commits

Reviewing files that changed from the base of the PR and between 32fd6ce and 689e468.

📒 Files selected for processing (1)

• src/main/java/org/gridsuite/study/server/service/StudyService.java

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

• src/main/java/org/gridsuite/study/server/service/StudyService.java

---

📝 Walkthrough

Walkthrough

Changes

StudyService now passes report UUIDs explicitly during network creation and recreation. New operations generate UUIDs. Recreation from an existing root network reuses its stored report UUID. persistNetwork forwards the UUID to networkConversionService.

Network report UUID propagation

Layer / File(s)	Summary
Report ID persistence contract src/main/java/org/gridsuite/study/server/service/StudyService.java	Study creation, root-network creation, and root-network case modification generate report UUIDs. persistNetwork forwards each supplied UUID to networkConversionService.
Recreation report propagation src/main/java/org/gridsuite/study/server/service/StudyService.java	Case-based recreation generates a report UUID. Existing-root-network recreation forwards its stored report UUID through the internal recreation flow.

Mergeability Score: ⚪ Minimal · up to 689e4

This localized change has no actionable merge-blocking risk identified at the current head and is merge-ready after normal checks and review.

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 22.22% 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 contains only a generic template and does not provide meaningful details about the changeset.	Add a concise summary that explains how reportId is passed during network recreation and how report UUIDs are generated or reused.

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly summarizes the primary change: adding reportId when recreating a network.
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