Variable-length argument lists for PropertyMappingConfiguration methods

[jdoubleu]

Yet the allowProperties(), skipProperties() and allowAllPropertiesExcept() methods of the PropertyMappingConfiguration class don't have arguments defined but use the func_get_args() function to retrieve all arguments from this function's call.

PHP5.6+ supports variable-length argument lists'.

This argument list might also be strongly typed (e.g. with string for property names).

Example:

public function allowProperties(string ...$propertyNames)
{
    foreach ($propertyNames as $propertyName) {
        $this->propertiesToBeMapped[$propertyName] = $propertyName;
    }
    return $this;
}

This would improve the API docs, the readability and the IDE support.

Is there a good reason why this isn't implemented yet? Or should it be part of #999?

May I create a PR?

[kitsunet]

As we switched our minimum required version from 5.3 to 7.0 with the latest major a few months ago I am sure there are many places where we could improve like that. Please open a PR every of those improvements is welcome :)

[albe]

What Christian said :)

[zaveryukha]

I think this issue can be closed?
PR #1035 merged.