fix: planController ajusta o semestre das matérias concluídas

[KakeyaK]

No description provided.

[Copilot]

Pull Request Overview

This PR ensures that the semester is correctly updated for already-saved plans when they’re marked as completed, and it tidies up brace placement in the completed method.

• Assign semester to existing plans before marking them completed in syncWithReplication().
• Normalize the } else { formatting and counter initialization in completed().

Comments suppressed due to low confidence (2)

app/Http/Controllers/PlanController.php:214

• Add or update unit tests to cover the new semester assignment logic when marking a saved plan as completed, ensuring this behavior is validated and protected against regressions.

                        $savedPlan->semester = $semesterId;

app/Http/Controllers/PlanController.php:276

• [nitpick] The brace placement here differs from the rest of the file’s style; align this with the project’s established convention (e.g., keep else on its own line or consistently use } else {).

        } else {