Refactor: merge duplicate controller directories

[lindaeskilsson]

merged controller dir to one and deleted the second

Summary by CodeRabbit

• Refactor
  • Reorganized internal code structure for improved maintainability.
  • Removed unused imports to reduce code clutter.

[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: 51edb457-0132-4127-843e-f212dd138c47

📥 Commits

Reviewing files that changed from the base of the PR and between bc8a19b and 25239f6.

📒 Files selected for processing (2)

• src/main/java/org/example/vet1177/controller/CommentController.java
• src/main/java/org/example/vet1177/controller/MedicalRecordController.java

---

📝 Walkthrough

Walkthrough

Two Java controller classes had their package names changed from plural (org.example.vet1177.controllers) to singular (org.example.vet1177.controller). Additionally, an unused wildcard import was removed from one controller file. No logic changes were made.

Changes

Cohort / File(s)	Summary
Controller Package Reorganization src/main/java/org/example/vet1177/controller/CommentController.java, src/main/java/org/example/vet1177/controller/MedicalRecordController.java	Package declaration updated from org.example.vet1177.controllers to org.example.vet1177.controller; unused wildcard import removed from MedicalRecordController.java.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

• Add MedicalRecordController for managing medical record operations #71: Directly updates package declarations for both MedicalRecordController and CommentController from plural to singular naming convention.
• Feat/comment controller #55: Modifies the same CommentController class with package renaming changes.

Suggested reviewers

• johanbriger
• TatjanaTrajkovic

Poem

🐰 Controllers hop to a new home,
Singular paths where they now roam,
Imports trimmed, the code runs clean,
Best package structure ever seen! ✨

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main change: refactoring to merge duplicate controller directories by consolidating package names from 'controllers' to 'controller'.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/controller-structure

---

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.}