Below code isn't fixed, actually is opposite.

This one:

      // get the vendor ID
      $vendor_name = variable_get(PAYPOINT_SETTING_VENDOR_NAME);
      if(isset($order->data['paypoint_overrides']['Vendor'])) {
        $vendor_name = $order->data['paypoint_overrides']['Vendor'];
      }

is converted into:

    // get the vendor ID
    $vendor_name = variable_get(PAYPOINT_SETTING_VENDOR_NAME);
    if(isset($order->data['paypoint_overrides']['Vendor'])) {
    $vendor_name = $order->data['paypoint_overrides']['Vendor'];
    }

Command I'm testing:

phpcbf --standard=Drupal commerce_paypoint/advanced_payments

Comments

kenorb created an issue. See original summary.

klausi’s picture

Status: Active » Postponed (maintainer needs more info)

for me it gets fixed to

    // Get the vendor ID.
    $vendor_name = variable_get(PAYPOINT_SETTING_VENDOR_NAME);
if (isset($order->data['paypoint_overrides']['Vendor'])) {
  $vendor_name = $order->data['paypoint_overrides']['Vendor'];
}

Which is also not great, but it is hard to fix if there is a broken 4 space indentation already. Do you have a more complete sample where the 4 space indentation is used?

klausi’s picture

Status: Postponed (maintainer needs more info) » Closed (works as designed)

Please reopen if this is still an issue.