Cache inherited settings outside of structure

LMS-11363

@nasthagiri I verified w/ red green testing
(some pylint fixes b/c my ide is being more vigilant)
Why are L143-144 within the `if` statement at L141?  Wouldn't `inherited_settings` need to be set if it were cached in self.course_entry, even if it weren't passed in?