wouldn't this read better as an if/else , unless its the standard pattern of the module