Not at all. I don't really know the code behind. If the check should be factType != null, we should indeed fix it. I did not want to change the behavior so I just made it clear the method is called with null.