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

Already on GitHub? Sign in to your account

Add some static utilities for common handler operations #112

Closed
zznate opened this Issue Feb 25, 2013 · 2 comments

Comments

Projects
None yet
2 participants
Owner

zznate commented Feb 25, 2013

There are a lot of operations within the new Handler impls that are similar-ish enough to live in a utility class. E.g.

    Integer id = event.body.getInteger("id");
    JsonObject params = event.body.getObject("op");
    JsonObject state = event.body.getObject("state");

Create a utility class to two for encapsulation of such.

Collaborator

edwardcapriolo commented Feb 25, 2013

I think we should really focus on just using JSONObject right from the client and not use jackson anymore .

Owner

zznate commented Feb 25, 2013

Yep - my goal with this issue was simply to get to a point where we alleviate any dupped code.

@zznate zznate closed this in ae70397 Feb 26, 2013

@edwardcapriolo edwardcapriolo added a commit that referenced this issue Feb 26, 2013

@edwardcapriolo edwardcapriolo Merge pull request #113 from zznate/handler-utils
initial replace of some common operations with HandlerUtils. Closes #112
466af45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment