> Tell me which of these express the intention which is "To refresh the session token" 

Actually, I don't want to refresh an existing token. I just want to issue a new token. So `$this->refreshToken()` would be misleading and `JFactory::getSession()->getToken(true);` would be what I want.

`JSession::checkToken() or jexit(JText::_('JINVALID_TOKEN'));` does more than "make sure the current session is valid". It also ends the script if it's not. So the method name would be wrong anyway. If you want to make it correct you would have to use. The function to validate is `JSession::checkToken()` which describes exactly what it does. The `jexit` is already the fail condition of the failed validating.

It may come down to personal preferences of course. However I'm a big fan of keeping PRs on its topic and adding new methods to wrap static calls doesn't seem to be needed for adding single task contollers.