Avoid double subscription by moving sub to constructor.
Check for proper properties