If I set the methods to void I get an exception:

WARNING: A HTTP GET method, public void 
uk.ac.ox.oucs.vle.LoginResource.getSetHelper(java.lang.String,javax.servlet.http.HttpServletRequest,javax.servlet.http.HttpServletResponse) 
throws javax.servlet.ServletException,java.io.IOException, MUST return a 
non-void type.
06-Mar-2013 09:11:10 
com.sun.jersey.server.impl.application.WebApplicationImpl newResourceClass
SEVERE: A sub-resource locator, public void 
uk.ac.ox.oucs.vle.LoginResource.getDoLogin(javax.servlet.http.HttpServletRequest,javax.servlet.http.HttpServletResponse) 
throws javax.servlet.ServletException,java.io.IOException, MUST return a 
non-void type.
com.sun.jersey.api.container.ContainerException: Fatal issues found at 
class uk.ac.ox.oucs.vle.LoginResource. See logs for more details.


On 04/03/13 17:56, Matthew Buckett wrote:
>
> In tool/src/main/java/uk/ac/ox/oucs/vle/LoginResource.java:
>
> > +
> > +	public LoginResource(@Context ContextResolver<Object> resolver) {
> > +	}
> > +	
> > +	@Path("/sethelper")
> > +	@GET
> > +	@Produces({MediaType.APPLICATION_XHTML_XML, MediaType.TEXT_HTML})
> > +	public String getSetHelper(
> > +			@QueryParam("returnUrl") final String returnurl,
> > +			@Context final HttpServletRequest request,
> > +		    @Context final HttpServletResponse response) throws ServletException, IOException {
> > +		
> > +		Session session = SessionManager.getCurrentSession();
> > +		session.setAttribute(Tool.HELPER_DONE_URL, returnurl);
> > +		response.sendRedirect(request.getContextPath()+request.getServletPath()+"/login/dologin");
> > +		return "";
>
> Shoudln't this be a void method?
>
> —
> Reply to this email directly or view it on GitHub 
> <https://github.com/ox-it/wl-course-signup/pull/18/files#r3228476>.
>