Use a value, not pointer, for Job.TargetState field

This field should always have a value, so it should not be a pointer. It is currently a pointer because it used to be fetched independently from the Registry where the value could simply not exist. In the event that the value is not set in the registry, the default value is "inactive", so let's just provide that and centralize the logic in one place.