vindi/vindi-woocommerce

The closing brace for the class must go on the next line after the body

Open

}

• Exclude checks
  • Disable engine
  • Disable check

Opening brace of a class must be on the line after the definition

Open

class FrontendFilesLoader {

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on method "__construct"

Open

  function __construct() {

• Exclude checks
  • Disable engine
  • Disable check

Terminating statement must be indented to the same level as the CASE body

Open

          break;

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

if ( ! defined( 'ABSPATH' ) ) {

• Exclude checks
  • Disable engine
  • Disable check

Blank line found at start of control structure

Open

            if ($sign_up_fee != null && $sign_up_fee > 0) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

if ( ! defined( 'ABSPATH' ) ) {

• Exclude checks
  • Disable engine
  • Disable check

Blank line found at start of control structure

Open

    if ($this->is_valid_br_zip_code($shipping_address_zip)) {

• Exclude checks
  • Disable engine
  • Disable check

The DEFAULT body must start on the line following the statement

Open

            default:

• Exclude checks
  • Disable engine
  • Disable check

Opening brace of a class must be on the line after the definition

Open

class FieldValidator {

• Exclude checks
  • Disable engine
  • Disable check

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

Open

  }

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

        if (!is_null($amount) ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

    if (isset($_POST['post_data'] ) ) {

• Exclude checks
  • Disable engine
  • Disable check

The CASE body must start on the line following the statement

Open

            case static::ORDER_TYPE_SUBSCRIPTION:

• Exclude checks
  • Disable engine
  • Disable check

Closing class brace must be on a line by itself

Open

};

• Exclude checks
  • Disable engine
  • Disable check

The method create has a boolean flag argument $recreated, which is a certain sign of a Single Responsibility Principle violation.

Open

  function create($post_id, $post, $update, $recreated = false)

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

class Foo {
    public function bar($flag = true) {
    }
}

Avoid too many return statements within this method.

Open

    return $response;

Avoid too many return statements within this method.

Open

      return array(
        'product' => $variations_products,
        'plan' => $variations_plans,
      );

Avoid too many return statements within this method.

Open

        return $createdProduct;

Avoid too many return statements within this method.

Open

    return $response;