[GitHub] [roller] mbien opened a new pull request #84: Use weblog.getHandle() instead of client provided String .

GitBox <[email protected]> Mon, 22 Mar 2021 02:21:13 -0000

Newsgroups	gmane.comp.java.roller.devel
Message-ID	<roller.84.MDExOlB1bGxSZXF1ZXN0NTk3NjE5NzEz.gitbox@gitbox.apache.org>

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:
[email protected]