Refactor: scheduler: exchange the names of attr_in_string() and attr_not_in_string() to reflect the fact

The functions previously did exactly the opposites of what they claimed to do.
Invocations of pcmk__xe_remove_matching_attrs() happened to use the particular
functions that did the right jobs in practice though.