I disagree. In my opinion, "make_person_an_outside_subscriber_if_all_accesses_revoked" is an ugly name for an ugly function. The label matches the content, and that is the primary goal of naming functions. Of course, it would be better if things were refactored into

   var allAccessesRevoked = items.All( areRevoked);
   if( allAccessesRevoked)
     person.makeOutsiedSubscriber;

but sometimes, the programmer hasn't discovered a good refactoring.