search

chkal / mvc-spec-migration-test

0 stars 0 forks source link

Discuss @View support #29

Closed chkal closed 9 years ago

chkal commented 9 years ago

Original issue MVC_SPEC-30 created by Santiago Pericas-Geertsen:

At the moment, @View is only used in controller methods that return void. In all other cases, the return value is used instead.

Are there other cases of @View that should be considered?

chkal commented 9 years ago

Comment by Christian Kaltepoth:

I just want to capture my thoughts on this for later discussions.

IMO @View should also be considered when the return type of the controller method is String (or something else) but at runtime null is returned. Wouldn't it make sense to use @View as some kind of default in these situations?

chkal commented 9 years ago

Comment by Santiago Pericas-Geertsen:

Seems reasonable to consider @View for either void or null.

chkal commented 9 years ago

Comment by Christian Kaltepoth:

Looks like we have an agreement on that:

https://java.net/projects/mvc-spec/lists/jsr371-experts/archive/2015-04/message/25

chkal commented 9 years ago

Comment by Santiago Pericas-Geertsen:

See revision:

https://java.net/projects/mvc-spec/sources/git/revision/2db5761d108849cdb715e3ac344caa05185b4e7c

chkal commented 9 years ago

Comment by Santiago Pericas-Geertsen:

Spec document updated.