copy module: avoid sprurious updates of destination directory permissions

[h3nd24]

SUMMARY

Fixes #79725:

• Refactored when we are calling chown_recursive. Now it is called when we are copying the file / directory in the copy_diff_files and copy_left_only, also called when we are creating new directory when the target is non-existent
• Changed a bit on how chown_recursive works. Now it also may change the ownership and group of the target file / directory
• Opportunistically fixed the problem where local_follow doesn't specify the default, which means when undefined, links are not copied across
• Opportunistically fixed the problem where check mode under certain circumstances will override the ownership and group

ISSUE TYPE

• Bugfix Pull Request

COMPONENT NAME

copy module

ADDITIONAL INFORMATION

[ansibot]

The test ansible-test sanity --test pep8 [explain] failed with 4 errors:

lib/ansible/modules/copy.py:360:74: W291: trailing whitespace
lib/ansible/modules/copy.py:361:58: W291: trailing whitespace
lib/ansible/modules/copy.py:362:74: W291: trailing whitespace
lib/ansible/modules/copy.py:477:1: E302: expected 2 blank lines, found 1

click here for bot help

[bcoca]

waiting_on_contributor

[s-hertel]

closing for inactivity