Yup, that'd be good! "isActive" sounds like just a boolean getter, so it's surprising that it does something else.