Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Isolate Play dependencies: Controllers #63

Open
cayhorstmann opened this issue Jun 23, 2024 · 1 comment
Open

Isolate Play dependencies: Controllers #63

cayhorstmann opened this issue Jun 23, 2024 · 1 comment
Labels
betterbuild Better build reorg 2024

Comments

@cayhorstmann
Copy link
Owner

In the controllers package, move as much code as possible to classes that do not depend on the Play API. For example, in controllers.Files, all three methods can call methods in a new class ui.Files that return the HTML to be displayed. The controllers.Files methods should only deal with the Play-specific logic (parsing HTTP requests, sending HTTP responses, cookies, etc.) ui.Files should have no Play dependencies. Repeat for Check, Upload, etc.

The views templates should be eliminated. Just generate the HTML in Java.

@cayhorstmann cayhorstmann added the betterbuild Better build reorg 2024 label Jun 23, 2024
@JustNekoChris
Copy link

I will be working on this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
betterbuild Better build reorg 2024
Projects
None yet
Development

No branches or pull requests

2 participants