-
Notifications
You must be signed in to change notification settings - Fork 38.6k
Closed
Closed
Copy link
Labels
status: declinedA suggestion or change that we don't feel we should currently applyA suggestion or change that we don't feel we should currently applytype: taskA general taskA general task
Description
Chris Beams opened SPR-5488 and commented
Petclinic currently has the following controllers:
AddOwnerForm
AddPetForm
AddVisitForm
ClinicController
EditOwnerForm
EditPetForm
FindOwnersForm
Even though these controllers have long since been migrated to @MVC
annotations and away from SimpleFormController / MultiActionController extension, their granularity reflects their inheritance-based origins.
A much better layout can be acheived now, where most or all entity-related functions are consolidated in a dedicated Controller. Simply:
OwnerController
PetController
VisitController
VetController
etc..
This issue is a sub-task of #10156
Metadata
Metadata
Assignees
Labels
status: declinedA suggestion or change that we don't feel we should currently applyA suggestion or change that we don't feel we should currently applytype: taskA general taskA general task