Are you suggesting this module just put the vhost file in `sites-enabled` directly, with no symlink to `sites-available`? puppetlabs-apache  puts the generated config file in `sites-available` then symlinks from `sites-enabled` if the vhost is enabled, and removes both the file and symlink if absent. See https://github.com/puppetlabs/puppetlabs-apache/blob/master/manifests/vhost.pp#L426.

Also if there's no symlink there'll be no way to remove a site from `sites-enabled` since puppetlabs-concat doesn't have `$ensure` support yet so it'll be impossible to remove the file concat generated.