ImplodeArgumentsOrder

info on by default quick-fix

Group: Code style · PhpStorm name: ImplodeArgumentsOrderInspection

implode() historically accepted its separator in either position; passing the pieces first and the separator second is deprecated (7.4) and removed (8.0). When the second argument is clearly the separator (a string literal), the arguments are swapped.

Example

Before

<?php

$csv   = implode($cells, ';');
$path  = \implode( $segments ,  "/" );
$ok1   = implode(' | ', $labels);
$ok2   = implode($labels, $sep);
$ok3   = implode($labels);
$ok4   = join($labels, '-');

After fix

<?php

$csv   = implode(';', $cells);
$path  = \implode("/", $segments);
$ok1   = implode(' | ', $labels);
$ok2   = implode($labels, $sep);
$ok3   = implode($labels);
$ok4   = join($labels, '-');

Reported:

• line 3: Pass the separator as the first argument of implode().
• line 4: Pass the separator as the first argument of implode().

Options

Option	Type	Default	Effect

Configure

In custos.json:

{
  "rules": {
    "ImplodeArgumentsOrder": {
      "enabled": false
    }
  }
}

Suppress

Before the statement or declaration (or the first statement of the file), either of:

// @custos-ignore ImplodeArgumentsOrder

/**
 * @noinspection ImplodeArgumentsOrderInspection
 */