Deleted users appear in search results about moodle-tool_mergeusers HOT 3 OPEN

Thanks for the reporting @leonstr .

This is unitentional. Deleted users should not be listed. Any of the other users must be listed. Imagine you merge 2 users in the inverse order. So, proceeding with the inverse merging operation should leave things as they should initially. So, for this reason, only suspended and non suspended users must be listed.

If you believe you can report a patch to solve this, it would be easier and quicker to get solved. Otherwise, we get it recorded to address it when we have availability.

Thanks a lot in advance,

Jordi

from moodle-tool_mergeusers.

@jpahullo The commit for the PR to fix #275 is built upon the commit to fix this issue as 1) they're directly related, and 2) they both touch the same code. I hope these two changes are not confusing!

from moodle-tool_mergeusers.

Ok about your above comment. Thanks a lot for working on it!

Please, close the PR that finally is not useful by now by your own, so I finally do not get confused in the future.

Let me know analyse the PR #276 then.

Thanks again.

Jordi

from moodle-tool_mergeusers.