mbien opened a new pull request #84:
URL: https://github.com/apache/roller/pull/84


   Although the handle variable is used for the reply, prior to that, 
getWeblogByHandle(handle) is called. This should throw an exception and return 
500 if the weblog does not exist and/or is invalid, without posting any json 
back.
   
   Changing "handle" to weblog.getHandle() would probably make the code scanner 
happy and make it also clearer that the weblog instance is valid at this point 
in the code.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to