vindi/vindi-woocommerce

Method __construct has 27 lines of code (exceeds 25 allowed). Consider refactoring.

Open

  public function __construct(VindiSettings $vindi_settings, VindiControllers $controllers)
  {
    $this->id                   = 'vindi-bank-slip';
    $this->icon                 = apply_filters('vindi_woocommerce_bank_slip_icon', '');
    $this->method_title         = __('Vindi - Boleto Bancário', VINDI);

Avoid assigning values to variables in if clauses and the like (line '127', column '9').

Open

  public function payment_fields()
  {
    $user_country = $this->get_country_code();

    if (empty($user_country)) {

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($foo = 'bar') { // possible typo
            // ...
        }
        if ($baz = 0) { // always false
            // ...
        }
    }
}

The property $vindi_settings is not named in camelCase.

Open

class VindiBankSlipGateway extends VindiPaymentGateway
{
  /**
   * @var VindiSettings
   */

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    protected $property_name;
}

A file should declare new symbols (classes, functions, constants, etc.) and cause no other side effects, or it should execute logic with side effects, but should not do both. The first symbol is defined on line 18 and the first side effect is on line 6.

Open

<?php

• Exclude checks
  • Disable engine
  • Disable check

Function closing brace must go on the next line following the body; found 1 blank lines before brace

Open

  }

• Exclude checks
  • Disable engine
  • Disable check

Inline control structures are not allowed

Open

    if ($is_trial = $this->vindi_settings->get_is_active_sandbox())

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 8 spaces, found 6

Open

      'subscription_payment_method_change',

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 8 spaces, found 4

Open

    $this->init_form_fields();

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 4 spaces, found 2

Open

  {

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 8 spaces, found 6

Open

      'subscription_cancellation',

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 8 spaces, found 6

Open

      'subscription_reactivation',

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 8 spaces, found 6

Open

      'subscription_payment_method_change_customer',

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 8 spaces, found 4

Open

    if (empty($user_country)) {

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 4 spaces, found 2

Open

  {

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 8 spaces, found 4

Open

    $this->has_fields           = true;

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 8 spaces, found 4

Open

    add_action('woocommerce_view_order', array(&$this, 'show_bank_slip_download'), -10, 1);

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 8 spaces, found 4

Open

    add_action('woocommerce_thankyou_' . $this->id, array(&$this, 'thank_you_page'));

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 8 spaces, found 4

Open

    $this->enabled = $this->get_option('enabled');

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 8 spaces, found 4

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 8 spaces, found 4

Open

    if (!$this->routes->acceptBankSlip()) {

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 4 spaces, found 2

Open

  public function __construct(VindiSettings $vindi_settings, VindiControllers $controllers)

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 8 spaces, found 4

Open

    $this->id                   = 'vindi-bank-slip';

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 12 spaces, found 6

Open

      return;

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 8 spaces, found 6

Open

      'products',

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 8 spaces, found 6

Open

      'subscription_date_changes',

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 8 spaces, found 6

Open

      'title'           => array(

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 4 spaces, found 2

Open

  {

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 12 spaces, found 6

Open

      return;

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 8 spaces, found 4

Open

    $this->supports             = array(

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 4 spaces, found 2

Open

  public $controllers;

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 8 spaces, found 6

Open

      'multiple_subscriptions',

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 4 spaces, found 2

Open

  public function type()

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 8 spaces, found 6

Open

      'enabled'         => array(

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 8 spaces, found 4

Open

    );

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 8 spaces, found 6

Open

      if ($order->get_payment_method() == 'vindi-bank-slip') {

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 8 spaces, found 6

Open

      }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 4 spaces, found 2

Open

  exit;

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 8 spaces, found 6

Open

      'subscription_payment_method_change_admin',

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 8 spaces, found 4

Open

    return 'bank_slip';

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 4 spaces, found 2

Open

  public function init_form_fields()

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 8 spaces, found 6

Open

      $order = wc_get_order($order_id);

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 8 spaces, found 4

Open

    );

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 4 spaces, found 2

Open

  {

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 4 spaces, found 2

Open

  public function payment_fields()

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 8 spaces, found 4

Open

    $is_single_order = $this->is_single_order();

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 8 spaces, found 6

Open

      $is_trial = $this->routes->isMerchantStatusTrialOrSandbox();

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 4 spaces, found 2

Open

  }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 8 spaces, found 4

Open

    $this->method_description   = __('Aceitar pagamentos via boleto bancário utilizando a Vindi.', VINDI);

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 4 spaces, found 2

Open

  }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 4 spaces, found 2

Open

  {

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 8 spaces, found 4

Open

    $this->form_fields = array(

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 4 spaces, found 2

Open

  }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 8 spaces, found 4

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 8 spaces, found 6

Open

      'subscription_suspension',

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 8 spaces, found 6

Open

      'subscription_amount_changes',

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 8 spaces, found 4

Open

    $this->title = $this->get_option('title');

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 8 spaces, found 6

Open

      ),

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 8 spaces, found 4

Open

    $this->icon                 = apply_filters('vindi_woocommerce_bank_slip_icon', '');

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 12 spaces, found 6

Open

      _e('Selecione o País para visualizar as formas de pagamento.', VINDI);

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 8 spaces, found 6

Open

      'subscriptions',

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 8 spaces, found 6

Open

      'pre-orders'

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 4 spaces, found 2

Open

  }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 8 spaces, found 6

Open

      )

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 12 spaces, found 6

Open

      _e('Este método de pagamento não é aceito.', VINDI);

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 8 spaces, found 4

Open

    $this->init_settings();

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 8 spaces, found 4

Open

    parent::__construct($vindi_settings, $controllers);

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 8 spaces, found 4

Open

    if ($is_trial = $this->vindi_settings->get_is_active_sandbox())

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 4 spaces, found 2

Open

  public function thank_you_page($order_id)

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 14 spaces, found 12

Open

            );

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 4 spaces, found 2

Open

  public $vindi_settings;

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 8 spaces, found 4

Open

    $this->method_title         = __('Vindi - Boleto Bancário', VINDI);

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 8 spaces, found 4

Open

    $this->description = $this->get_option('description');

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 8 spaces, found 4

Open

    $user_country = $this->get_country_code();

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 8 spaces, found 4

Open

    $this->vindi_settings->get_template('bankslip-checkout.html.php', compact('is_trial', 'is_single_order'));

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 4 spaces, found 2

Open

  }

• Exclude checks
  • Disable engine
  • Disable check