Option to disable trailing comas after multiline parameters

[klkvsk]

Option to disable trailing comas after multiline parameters

[dg]

Since PHP 7 is no longer supported, I don't want to add support for it which would complicate the code.

I think it will be sufficient to remove the commas with a regular expression, something like preg_replace('~,(\n\t*\))~', '$1', $s)

[dg]

But please do it in your own Printer, I don't want to merge this into the code, because PHP 7 is outdated.

[klkvsk]

Added a flag in Printer.

But the other case is when parameters are printed within a function call:

$method = new \Nette\PhpGenerator\Method('foo');
$method->addBody('return bar(...?);', [[
    '1234567890-1234567890-1234567890-1234567890',
    '1234567890-1234567890-1234567890-1234567890',
    '1234567890-1234567890-1234567890-1234567890',
]]);

$printer = new \Nette\PhpGenerator\Printer();
echo $printer->printMethod($method);

Output:

function foo()
{
        return bar(
                '1234567890-1234567890-1234567890-1234567890',
                '1234567890-1234567890-1234567890-1234567890',
                '1234567890-1234567890-1234567890-1234567890',
        );
}

This output is done by Dumper (Dumper::dumpArguments). Sure we can pass the flag from Printer to Dumper as done with indentation and wrapLength properties. But, as I see, it's not always being passed. Like in FunctionLike::addBody there's just a new instance of Dumper created without configuration.

Not sure what's the best way to deal with that without global refactoring. Any ideas besides implementing some kind of DumperFactory stuff?

[klkvsk]

I just noticed that rendered bodies are passed through Printer::indent, where it converts the indentation where needed.
Putting said preg_replace there as the most non-invasive fix.

[nevmerzhitsky]

Issues like this (remove or add the comma) may be solved by an additional call of tools like "PHP Coding Standards Fixer" which changes code style, imho.