Simplify implementation of the show_plans? method
Description
While implementing !61142 (merged), we noticed the implementation for the method show_plans? is not very readable.
Also, the business logic behind it needs to be revalidated, so we're sure what we're doing there is correct. For example, we're not sure if it's correct that namespace.ultimate_plan
can be true
at the same time that namespace.trial_active?
is also true
.
Edit: the strike-trhough part will be tackled with #337520
Edited by Angelo Gulina