fix(spreadsheet): enable multiple optional loading parameters

[TheMaskedTurtle]

PR Summary

If we modify several loading options at the same time only one is taken into account for reloading related spreadsheet which is not the expected behavior.

For example, if you load lines and generators spreadsheets, if then we toggle on "inactive limit sets" on lines and "regulated terminals" on generators, we expect both types to be completely reloaded to add these data. But for now, if you do that, only the lines will be reloaded. So, this fix enables to reload both lines and generators in this situation.

[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: 3e96ba1c-7c17-4a04-847b-bdb6ae4399c4

📥 Commits

Reviewing files that changed from the base of the PR and between 87d5c94 and cce49a4.

📒 Files selected for processing (1)

• src/components/spreadsheet-view/spreadsheet/spreadsheet-content/hooks/use-optional-loading-parameters-for-equipments.ts

---

📝 Walkthrough

Walkthrough

A conditional chain in a useEffect hook is refactored from mutually exclusive if / else if blocks to independent if blocks, allowing multiple equipment type state updates to execute simultaneously in a single render cycle instead of stopping at the first matching condition.

Changes

Cohort / File(s)	Summary
Equipment Optional Loading Parameters Hook src/components/spreadsheet-view/spreadsheet/spreadsheet-content/hooks/use-optional-loading-parameters-for-equipments.ts	Changed useEffect conditional logic from if / else if chain to multiple independent if blocks for equipment types (LINE, TWO_WINDINGS_TRANSFORMER, GENERATOR, BUS). Enables simultaneous state updates for multiple equipment types in a single render cycle.

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

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.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately describes the main change: converting mutually exclusive branches to allow multiple optional loading parameters to be updated simultaneously.
Description check	✅ Passed	The description clearly explains the problem (only one spreadsheet reloads when multiple options change) and the solution, directly relating to the code change.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

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 and usage tips.}

[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