Avoid using `not`...prefer using `!`.  Especially never do `unless !` since you're getting unto double-negative territory, which is confusing.  Especially especially don't do `unless !foo.nil?` because now you're in triple-negative territory. :package: 

This line can just be `if product.nil?`