I think the reason "easier to locate" is good enough. The module should contain whole logic not just a part of it. I do not think that corner cases should be treated any differently. And if `delete_associated_permission_tags` is broken to some smaller (some private) methods it will also help understand-ability.