The text looks better but is this what we really want?
I mean, even if the naming makes sense and has its coherency, is vsftpd_config_file_template , for example, a good name to refer to the template of the vsftpd main config file?
And why ancillary files like chroot_list_config_file_template are considered and not possible apache files (htpasswd for example)?
Just wondering and sharing my thoughts, no real / clear solution in my mind.

Anyway I'd try to find a sane but not necessarily definitive  resolution for this PR and merge it , then ask for comments on Puppet Users group and see if there are new ideas / proposals for a wider public validation.
It could be nice to set up a sort on online poll to let people vote for alternative solutions, as I wrote somewhere else general consensus and wider adoption is much more important than personal tastes or "feelings".