The original line is correct. We suspect the Browser can send null parameters under certain conditions (e.g. reloading a page from an expired page). In such cases, `assertPostParamNotNull` results in a less drastic error message to the user instead of `server error` message. In fact we should use `assertPostParamNotNull` in other places too. we haven't got around to doing it yet.