File class-git-wrapper.php has 522 lines of code (exceeds 250 allowed). Consider refactoring.

Open

<?php
/*  Copyright 2014-2016 Presslabs SRL <ping@presslabs.com>

    This program is free software; you can redistribute it and/or modify
    it under the terms of the GNU General Public License, version 2, as

Git_Wrapper has 43 functions (exceeds 20 allowed). Consider refactoring.

Open

class Git_Wrapper {

    private $last_error = '';
    private $gitignore  = GITIGNORE;

The class Git_Wrapper has an overall complexity of 129 which is very high. The configured complexity threshold is 50.

Open

class Git_Wrapper {

    private $last_error = '';
    private $gitignore  = GITIGNORE;

• Exclude checks
  • Disable engine
  • Disable check

The class Git_Wrapper has 20 public methods. Consider refactoring Git_Wrapper to keep number of public methods under 10.

Open

class Git_Wrapper {

    private $last_error = '';
    private $gitignore  = GITIGNORE;

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

Function local_status has a Cognitive Complexity of 16 (exceeds 5 allowed). Consider refactoring.

Open

    function local_status() {
        list( $return, $response ) = $this->_call( 'status', '-s', '-b', '-u' );
        if ( 0 !== $return ) {
            return array( '', array() );
        }

• Read up
  Read up

Function get_local_changes has a Cognitive Complexity of 12 (exceeds 5 allowed). Consider refactoring.

Open

    function get_local_changes() {
        list( $return, $response ) = $this->_call( 'status', '--porcelain'  );

        if ( 0 !== $return ) {
            return array();

• Read up
  Read up

Method _call has 33 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    protected function _call(...$args) {
        $args     = join( ' ', array_map( 'escapeshellarg', $args ) );
        $return   = -1;
        $response = array();
        $env      = $this->get_env();

Method merge_with_accept_mine has 28 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    function merge_with_accept_mine(...$commits) {
        do_action( 'gitium_before_merge_with_accept_mine' );

        if ( 1 == count($commits) && is_array( $commits[0] ) ) {
            $commits = $commits[0];

Function status has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

    function status( $local_only = false ) {
        list( $branch_status, $new_response ) = $this->local_status();

        if ( $local_only ) { return array( $branch_status, $new_response ); }

• Read up
  Read up

Function cherry_pick has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

    private function cherry_pick( $commits ) {
        foreach ( $commits as $commit ) {
            if ( empty( $commit ) ) { return false; }

            list( $return, $response ) = $this->_call( 'cherry-pick', $commit );

• Read up
  Read up

Function _call has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.

Open

    protected function _call(...$args) {
        $args     = join( ' ', array_map( 'escapeshellarg', $args ) );
        $return   = -1;
        $response = array();
        $env      = $this->get_env();

• Read up
  Read up

Function _log has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.

Open

    function _log(...$args) {
        if ( ! defined( 'WP_DEBUG' ) || ! WP_DEBUG ) { return; }

        $output = '';
        if (isset($args) && $args) foreach ( $args as $arg ) {

• Read up
  Read up

Function get_last_commits has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.

Open

    function get_last_commits( $n = 20 ) {
        list( $return, $message )  = $this->_call( 'log', '-n', $n, '--pretty=format:%s' );
        if ( 0 !== $return ) { return false; }

        list( $return, $response ) = $this->_call( 'log', '-n', $n, '--pretty=format:%h|%an|%ae|%ad|%cn|%ce|%cd' );

• Read up
  Read up

Function merge_with_accept_mine has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.

Open

    function merge_with_accept_mine(...$commits) {
        do_action( 'gitium_before_merge_with_accept_mine' );

        if ( 1 == count($commits) && is_array( $commits[0] ) ) {
            $commits = $commits[0];

• Read up
  Read up

Function commit has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.

Open

    function commit( $message, $author_name = '', $author_email = '' ) {
        $author = '';
        if ( $author_email ) {
            if ( empty( $author_name ) ) {
                $author_name = $author_email;

• Read up
  Read up

Avoid using undefined variables such as '$commits' which will lead to PHP notices.

Open

                $commits[ $commit_info[0] ]['committer_name']  = $commit_info[4];

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

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$commits' which will lead to PHP notices.

Open

            $commits[ $commit_info[0] ] = array(

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

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$commits' which will lead to PHP notices.

Open

                $commits[ $commit_info[0] ]['committer_email'] = $commit_info[5];

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

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$commits' which will lead to PHP notices.

Open

                $commits[ $commit_info[0] ]['committer_date']  = $commit_info[6];

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

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$commits' which will lead to PHP notices.

Open

        return $commits;

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

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

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

Open

    function status( $local_only = false ) {

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

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

The method get_env uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

        } else {
            $env['GIT_SSH'] = dirname( __FILE__ ) . '/ssh-git';
        }

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

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

The method push uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

        } else {
            list( $return, ) = $this->_call( 'push', '--porcelain', '-u', 'origin', 'HEAD' );
        }

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

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

The method get_local_changes uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

                } else {
                    $action = 'modified';
                }

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

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

The method commit uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

        } else {
            list( $return, $response ) = $this->_call( 'commit', '-m', $message );
        }

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

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

The method merge_with_accept_mine uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

        } else {
            $this->_call( 'cherry-pick', '--abort' );
            $this->_call( 'checkout', '-b', 'merge_local' );
            $this->_call( 'branch', '-M', $local_branch );
            return false;

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

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

The method _call uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

        } else {
            $this->last_error = null;
        }

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

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

Avoid unused local variables such as '$idx'.

Open

            foreach ( $response as $idx => $line ) :

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

class Foo {
    public function doSomething()
    {
        $i = 5; // Unused
    }
}

Avoid using count() function in for loops.

Open

            for ( $idx = 0 ; $idx < count( $response ) / 2 ; $idx++ ) {
                $file   = $response[ $idx * 2 + 1 ];
                $change = $response[ $idx * 2 ];
                if ( ! isset( $new_response[ $file ] ) ) {
                    $new_response[ $file ] = "r$change";

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

class Foo {

  public function bar()
  {
    $array = array();

    for ($i = 0; count($array); $i++) {
      // ...
    }
  }
}

Each class must be in a namespace of at least one level (a top-level vendor name)

Open

class Git_Wrapper {

• Exclude checks
  • Disable engine
  • Disable check

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 19 and the first side effect is on line 18.

Open

<?php

• Exclude checks
  • Disable engine
  • Disable check

The property $private_key is not named in camelCase.

Open

class Git_Wrapper {

    private $last_error = '';
    private $gitignore  = GITIGNORE;

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

class ClassName {
    protected $property_name;
}

The parameter $repo_dir is not named in camelCase.

Open

    function __construct( $repo_dir ) {
        $this->repo_dir = $repo_dir;
    }

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

class ClassName {
    public function doSomething($user_name) {
    }
}

The class Git_Wrapper is not named in CamelCase.

Open

class Git_Wrapper {

    private $last_error = '';
    private $gitignore  = GITIGNORE;

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

class class_name {
}

The parameter $author_email is not named in camelCase.

Open

    function commit( $message, $author_name = '', $author_email = '' ) {
        $author = '';
        if ( $author_email ) {
            if ( empty( $author_name ) ) {
                $author_name = $author_email;

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

class ClassName {
    public function doSomething($user_name) {
    }
}

The parameter $author_name is not named in camelCase.

Open

    function commit( $message, $author_name = '', $author_email = '' ) {
        $author = '';
        if ( $author_email ) {
            if ( empty( $author_name ) ) {
                $author_name = $author_email;

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

class ClassName {
    public function doSomething($user_name) {
    }
}

The property $last_error is not named in camelCase.

Open

class Git_Wrapper {

    private $last_error = '';
    private $gitignore  = GITIGNORE;

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

class ClassName {
    protected $property_name;
}

The property $repo_dir is not named in camelCase.

Open

class Git_Wrapper {

    private $last_error = '';
    private $gitignore  = GITIGNORE;

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

class ClassName {
    protected $property_name;
}

Avoid variables with short names like $n. Configured minimum length is 3.

Open

    function get_last_commits( $n = 20 ) {

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

class Something {
    private $q = 15; // VIOLATION - Field
    public static function main( array $as ) { // VIOLATION - Formal
        $r = 20 + $this->q; // VIOLATION - Local
        for (int $i = 0; $i < 10; $i++) { // Not a Violation (inside FOR)
            $r += $this->q;
        }
    }
}

The parameter $private_key is not named in camelCase.

Open

    function set_key( $private_key ) {
        $this->private_key = $private_key;
    }

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

class ClassName {
    public function doSomething($user_name) {
    }
}

Avoid variables with short names like $b. Configured minimum length is 3.

Open

        $response = array_map( function( $b ) { return str_replace( "origin/", "", $b ); }, $response );

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

class Something {
    private $q = 15; // VIOLATION - Field
    public static function main( array $as ) { // VIOLATION - Formal
        $r = 20 + $this->q; // VIOLATION - Local
        for (int $i = 0; $i < 10; $i++) { // Not a Violation (inside FOR)
            $r += $this->q;
        }
    }
}

The parameter $local_only is not named in camelCase.

Open

    function status( $local_only = false ) {
        list( $branch_status, $new_response ) = $this->local_status();

        if ( $local_only ) { return array( $branch_status, $new_response ); }

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

class ClassName {
    public function doSomething($user_name) {
    }
}

Visibility must be declared on method "__construct"

Open

    function __construct( $repo_dir ) {

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on method "_rrmdir"

Open

    function _rrmdir( $dir ) {

• Exclude checks
  • Disable engine
  • Disable check

Space found after opening bracket of FOREACH loop

Open

        if (isset($args) && $args) foreach ( $args as $arg ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces between opening bracket and argument "$dir"; 1 found

Open

    function _rrmdir( $dir ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

        foreach ( $files as $file ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

        if ( defined( 'GIT_SSH' ) && GIT_SSH ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces between argument "$repo_dir" and closing bracket; 1 found

Open

    function __construct( $repo_dir ) {

• Exclude checks
  • Disable engine
  • Disable check

Space found before closing bracket of FOREACH loop

Open

        foreach ( $files as $file ) {

• Exclude checks
  • Disable engine
  • Disable check

Method name "_log" should not be prefixed with an underscore to indicate visibility

Open

    function _log(...$args) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces between opening bracket and argument "$private_key"; 1 found

Open

    function set_key( $private_key ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

        if ( 0 != $return ) {

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on method "can_exec_git"

Open

    function can_exec_git() {

• Exclude checks
  • Disable engine
  • Disable check

Space found after opening bracket of FOREACH loop

Open

        foreach ( $files as $file ) {

• Exclude checks
  • Disable engine
  • Disable check

Method name "_call" should not be prefixed with an underscore to indicate visibility

Open

    protected function _call(...$args) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

        if ( is_resource( $proc ) ) {

• Exclude checks
  • Disable engine
  • Disable check

Method name "_rrmdir" should not be prefixed with an underscore to indicate visibility

Open

    function _rrmdir( $dir ) {

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on method "set_key"

Open

    function set_key( $private_key ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces between argument "$dir" and closing bracket; 1 found

Open

    function _rrmdir( $dir ) {

• Exclude checks
  • Disable engine
  • Disable check

Method name "_git_temp_key_file" should not be prefixed with an underscore to indicate visibility

Open

    function _git_temp_key_file() {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces between argument "$private_key" and closing bracket; 1 found

Open

    function set_key( $private_key ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

        if ( defined( 'GIT_KEY_FILE' ) && GIT_KEY_FILE ) {

• Exclude checks
  • Disable engine
  • Disable check

Method name "Git_Wrapper::get_behind_commits" is not in camel caps format

Open

    function get_behind_commits() {

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on method "_log"

Open

    function _log(...$args) {

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on method "get_last_error"

Open

    function get_last_error() {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

            while ( $line = fgets( $pipes[1] ) ) {

• Exclude checks
  • Disable engine
  • Disable check

Method name "Git_Wrapper::can_exec_git" is not in camel caps format

Open

    function can_exec_git() {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

        if (isset($args) && $args) foreach ( $args as $arg ) {

• Exclude checks
  • Disable engine
  • Disable check

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

Open

class Git_Wrapper {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

        if ( empty( $dir ) || ! is_dir( $dir ) ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

        if ( ! defined( 'WP_DEBUG' ) || ! WP_DEBUG ) { return; }

• Exclude checks
  • Disable engine
  • Disable check

Method name "Git_Wrapper::set_key" is not in camel caps format

Open

    function set_key( $private_key ) {

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on method "_git_temp_key_file"

Open

    function _git_temp_key_file() {

• Exclude checks
  • Disable engine
  • Disable check

Method name "Git_Wrapper::get_last_error" is not in camel caps format

Open

    function get_last_error() {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces between opening bracket and argument "$repo_dir"; 1 found

Open

    function __construct( $repo_dir ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

        } elseif ( $this->private_key ) {

• Exclude checks
  • Disable engine
  • Disable check

Space found before closing bracket of FOREACH loop

Open

        if (isset($args) && $args) foreach ( $args as $arg ) {

• Exclude checks
  • Disable engine
  • Disable check

Method name "Git_Wrapper::_git_temp_key_file" is not in camel caps format

Open

    function _git_temp_key_file() {

• Exclude checks
  • Disable engine
  • Disable check

Method name "Git_Wrapper::get_env" is not in camel caps format

Open

    private function get_env() {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

        if ( ! defined( 'GIT_KEY_FILE' ) && isset( $env['GIT_KEY_FILE'] ) ) {

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on method "get_version"

Open

    function get_version() {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces between argument "$message" and closing bracket; 1 found

Open

    protected function _resolve_merge_conflicts( $message ) {

• Exclude checks
  • Disable engine
  • Disable check

Space found after opening bracket of FOREACH loop

Open

        foreach ( $haystack as $query ) {

• Exclude checks
  • Disable engine
  • Disable check

Space found before closing bracket of FOREACH loop

Open

        foreach ( $haystack as $query ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces between argument "$dir" and closing bracket; 1 found

Open

    function is_dot_git_dir( $dir ) {

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on method "fetch_ref"

Open

    function fetch_ref() {

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on method "get_commit_message"

Open

    function get_commit_message( $commit ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

        foreach ( $commits as $commit ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

        if ( 0 != $return ) { return ''; }

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

        if ( 1 == count($commits) && is_array( $commits[0] ) ) {

• Exclude checks
  • Disable engine
  • Disable check

Method name "Git_Wrapper::remove_remote" is not in camel caps format

Open

    function remove_remote() {

• Exclude checks
  • Disable engine
  • Disable check

Method name "Git_Wrapper::get_remote_tracking_branch" is not in camel caps format

Open

    function get_remote_tracking_branch() {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces between argument "$offset" and closing bracket; 1 found

Open

    private function strpos_haystack_array( $haystack, $needle, $offset=0 ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

            if ( empty( $commit ) ) { return false; }

• Exclude checks
  • Disable engine
  • Disable check

Method name "Git_Wrapper::merge_with_accept_mine" is not in camel caps format

Open

    function merge_with_accept_mine(...$commits) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

        if ( WP_DEBUG ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces between argument "$url" and closing bracket; 1 found

Open

    function add_remote_url( $url ) {

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on method "get_remote_tracking_branch"

Open

    function get_remote_tracking_branch() {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

        if ( 0 == $return ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

        foreach ( $changes as $path => $change ) {

• Exclude checks
  • Disable engine
  • Disable check

Method name "Git_Wrapper::get_remote_url" is not in camel caps format

Open

    function get_remote_url() {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces between opening bracket and argument "$commits"; 1 found

Open

    private function cherry_pick( $commits ) {

• Exclude checks
  • Disable engine
  • Disable check

Space found after opening bracket of FOREACH loop

Open

        foreach ( $changes as $path => $change ) {

• Exclude checks
  • Disable engine
  • Disable check

Method name "Git_Wrapper::get_commit_message" is not in camel caps format

Open

    function get_commit_message( $commit ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

        if ( 0 != $return ) {

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on method "is_status_working"

Open

    function is_status_working() {

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on method "get_behind_commits"

Open

    function get_behind_commits() {

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on method "is_dot_git_dir"

Open

    function is_dot_git_dir( $dir ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

        if ( isset( $response[0] ) ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

            } elseif ( in_array( $change, array( 'AA', 'UU', 'AU', 'UA' ) ) ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces between opening bracket and argument "$haystack"; 1 found

Open

    private function strpos_haystack_array( $haystack, $needle, $offset=0 ) {

• Exclude checks
  • Disable engine
  • Disable check

Method name "Git_Wrapper::successfully_merged" is not in camel caps format

Open

    function successfully_merged() {

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on method "init"

Open

    function init() {

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on method "get_local_branch"

Open

    function get_local_branch() {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces between argument "$commit" and closing bracket; 1 found

Open

    function get_commit_message( $commit ) {

• Exclude checks
  • Disable engine
  • Disable check

Method name "Git_Wrapper::is_dot_git_dir" is not in camel caps format

Open

    function is_dot_git_dir( $dir ) {

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on method "merge_initial_commit"

Open

    function merge_initial_commit( $commit, $branch ) {

• Exclude checks
  • Disable engine
  • Disable check

Method name "Git_Wrapper::get_local_branch" is not in camel caps format

Open

    function get_local_branch() {

• Exclude checks
  • Disable engine
  • Disable check

Method name "Git_Wrapper::fetch_ref" is not in camel caps format

Open

    function fetch_ref() {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces between opening bracket and argument "$message"; 1 found

Open

    protected function _resolve_merge_conflicts( $message ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

        if ( ! empty( $version[0] ) ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

        if ( ! empty( $realpath ) && is_dir( $realpath ) && file_exists( $git_config ) && file_exists( $git_index ) ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

            if ( in_array( $change, array( 'UD', 'DD' ) ) ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces between opening bracket and argument "$commit"; 1 found

Open

    function get_commit_message( $commit ) {

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on method "successfully_merged"

Open

    function successfully_merged() {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces between opening bracket and argument "$dir"; 1 found

Open

    function is_dot_git_dir( $dir ) {

• Exclude checks
  • Disable engine
  • Disable check

TRUE, FALSE and NULL must be lowercase; expected "true" but found "True"

Open

            return True;

• Exclude checks
  • Disable engine
  • Disable check

Method name "Git_Wrapper::merge_initial_commit" is not in camel caps format

Open

    function merge_initial_commit( $commit, $branch ) {

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on method "add_remote_url"

Open

    function add_remote_url( $url ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

        if ( 0 == $return ) {

• Exclude checks
  • Disable engine
  • Disable check

Method name "_resolve_merge_conflicts" should not be prefixed with an underscore to indicate visibility

Open

    protected function _resolve_merge_conflicts( $message ) {

• Exclude checks
  • Disable engine
  • Disable check

Method name "Git_Wrapper::strpos_haystack_array" is not in camel caps format

Open

    private function strpos_haystack_array( $haystack, $needle, $offset=0 ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces between argument "$commits" and closing bracket; 1 found

Open

    private function cherry_pick( $commits ) {

• Exclude checks
  • Disable engine
  • Disable check

Space found before closing bracket of FOREACH loop

Open

        foreach ( $commits as $commit ) {

• Exclude checks
  • Disable engine
  • Disable check

Method name "Git_Wrapper::get_version" is not in camel caps format

Open

    function get_version() {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

        foreach ( $haystack as $query ) {

• Exclude checks
  • Disable engine
  • Disable check

Space found after opening bracket of FOREACH loop

Open

        foreach ( $commits as $commit ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

            if ( false !== $this->strpos_haystack_array( $response, 'previous cherry-pick is now empty' ) ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

        if ( $return != 0 ) {

• Exclude checks
  • Disable engine
  • Disable check

Method name "Git_Wrapper::get_ahead_commits" is not in camel caps format

Open

    function get_ahead_commits() {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

        if ( count( $commits ) > 0 ) {

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on method "cleanup"

Open

    function cleanup() {

• Exclude checks
  • Disable engine
  • Disable check

TRUE, FALSE and NULL must be lowercase; expected "true" but found "True"

Open

            return True;

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces between argument "$branch" and closing bracket; 1 found

Open

    function merge_initial_commit( $commit, $branch ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

            if ( WP_DEBUG ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces between opening bracket and argument "$url"; 1 found

Open

    function add_remote_url( $url ) {

• Exclude checks
  • Disable engine
  • Disable check

Space found before closing bracket of FOREACH loop

Open

        foreach ( $changes as $path => $change ) {

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on method "remove_remote"

Open

    function remove_remote() {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

            } elseif ( 'DU' == $change ) {

• Exclude checks
  • Disable engine
  • Disable check

Incorrect spacing between default value and equals sign for argument "$offset"; expected 1 but found 0

Open

    private function strpos_haystack_array( $haystack, $needle, $offset=0 ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

        if ( ! is_array( $haystack ) ) { $haystack = array( $haystack ); }

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

            if ( strpos( $query, $needle, $offset) !== false ) { return true; }

• Exclude checks
  • Disable engine
  • Disable check

Method name "Git_Wrapper::cherry_pick" is not in camel caps format

Open

    private function cherry_pick( $commits ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

        if ( $this->successfully_merged() ) { // git status without states: AA, DD, UA, AU ...

• Exclude checks
  • Disable engine
  • Disable check

Method name "Git_Wrapper::is_status_working" is not in camel caps format

Open

    function is_status_working() {

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on method "get_ahead_commits"

Open

    function get_ahead_commits() {

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on method "get_remote_url"

Open

    function get_remote_url() {

• Exclude checks
  • Disable engine
  • Disable check

TRUE, FALSE and NULL must be lowercase; expected "false" but found "False"

Open

        return False;

• Exclude checks
  • Disable engine
  • Disable check

Method name "Git_Wrapper::_resolve_merge_conflicts" is not in camel caps format

Open

    protected function _resolve_merge_conflicts( $message ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

        if ( $this->is_dot_git_dir( $dot_git_dir ) && $this->_rrmdir( $dot_git_dir ) ) {

• Exclude checks
  • Disable engine
  • Disable check

Incorrect spacing between argument "$offset" and equals sign; expected 1 but found 0

Open

    private function strpos_haystack_array( $haystack, $needle, $offset=0 ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

            if ( $return != 0 ) {

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on method "merge_with_accept_mine"

Open

    function merge_with_accept_mine(...$commits) {

• Exclude checks
  • Disable engine
  • Disable check

TRUE, FALSE and NULL must be lowercase; expected "false" but found "False"

Open

        return False;

• Exclude checks
  • Disable engine
  • Disable check

Method name "Git_Wrapper::add_remote_url" is not in camel caps format

Open

    function add_remote_url( $url ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces between argument "$author_email" and closing bracket; 1 found

Open

    function commit( $message, $author_name = '', $author_email = '' ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

                if ( '#' == $line[0] ) { continue; } // ignore branch status

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

                if ( ( 'R' === $index_status ) && ( ! empty( $new_path[1] ) ) ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

        if ( $return != 0 ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces between argument "$b" and closing bracket; 1 found

Open

        $response = array_map( function( $b ) { return str_replace( "origin/", "", $b ); }, $response );

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on method "get_local_changes"

Open

    function get_local_changes() {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

                if ( 'D' == $work_tree_status ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

        if ( 0 !== $return ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

        if ( ! empty( $response ) ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

        foreach ( $response as $index => $value ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

            if ( ! $this->successfully_merged() ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

        if ( 1 == count($args) && is_array( $args[0] ) ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

        if ( ! empty( $response ) ) {

• Exclude checks
  • Disable engine
  • Disable check

Method name "Git_Wrapper::local_status" is not in camel caps format

Open

    function local_status() {

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on method "get_last_commits"

Open

    function get_last_commits( $n = 20 ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces between opening bracket and argument "$content"; 1 found

Open

    public function set_gitignore( $content ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces between argument "$branch" and closing bracket; 1 found

Open

    function push( $branch = '' ) {

• Exclude checks
  • Disable engine
  • Disable check

Space found before first semicolon of FOR loop

Open

            for ( $idx = 0 ; $idx < count( $response ) / 2 ; $idx++ ) {

• Exclude checks
  • Disable engine
  • Disable check

Method name "Git_Wrapper::rm_cached" is not in camel caps format

Open

    function rm_cached( $path ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 1 space after FUNCTION keyword; 0 found

Open

        $response = array_map( function( $b ) { return str_replace( "origin/", "", $b ); }, $response );

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

        if ( ! empty( $branch ) ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

                if ( ( '"' == $path[0] ) && ('"' == $path[strlen( $path ) - 1] ) ) {

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on method "status"

Open

    function status( $local_only = false ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

                if ( ! isset( $new_response[ $file ] ) ) {

• Exclude checks
  • Disable engine
  • Disable check

Space found before closing bracket of FOREACH loop

Open

        foreach ( $response as $index => $value ) {

• Exclude checks
  • Disable engine
  • Disable check

Method name "Git_Wrapper::set_gitignore" is not in camel caps format

Open

    public function set_gitignore( $content ) {

• Exclude checks
  • Disable engine
  • Disable check

Space found before closing bracket of FOREACH loop

Open

            foreach ( $response as $line ) :

• Exclude checks
  • Disable engine
  • Disable check

Method name "Git_Wrapper::get_gitignore" is not in camel caps format

Open

    public function get_gitignore() {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

        if ( 0 != $return ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

        if ( $author_email ) {

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on method "push"

Open

    function push( $branch = '' ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces between opening bracket and argument "$b"; 1 found

Open

        $response = array_map( function( $b ) { return str_replace( "origin/", "", $b ); }, $response );

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on method "add"

Open

    function add(...$args) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces between opening bracket and argument "$branch"; 1 found

Open

    function push( $branch = '' ) {

• Exclude checks
  • Disable engine
  • Disable check

Method name "Git_Wrapper::get_last_commits" is not in camel caps format

Open

    function get_last_commits( $n = 20 ) {

• Exclude checks
  • Disable engine
  • Disable check

Space found after opening bracket of FOREACH loop

Open

            foreach ( $response as $line ) :

• Exclude checks
  • Disable engine
  • Disable check

Space found before closing bracket of FOREACH loop

Open

            foreach ( $response as $idx => $line ) :

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

        if ( $behind_count ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces between argument "$content" and closing bracket; 1 found

Open

    public function set_gitignore( $content ) {

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on method "rm_cached"

Open

    function rm_cached( $path ) {

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on method "commit"

Open

    function commit( $message, $author_name = '', $author_email = '' ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

        if ( ! empty( $author ) ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

        if ( $return !== 0 ) { return false; }

• Exclude checks
  • Disable engine
  • Disable check

Space found before second semicolon of FOR loop

Open

            for ( $idx = 0 ; $idx < count( $response ) / 2 ; $idx++ ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

            for ( $idx = 0 ; $idx < count( $response ) / 2 ; $idx++ ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces between argument "$n" and closing bracket; 1 found

Open

    function get_last_commits( $n = 20 ) {

• Exclude checks
  • Disable engine
  • Disable check

Space found after opening bracket of FOREACH loop

Open

        foreach ( $response as $index => $value ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces between argument "$path" and closing bracket; 1 found

Open

    function rm_cached( $path ) {

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on method "local_status"

Open

    function local_status() {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces between opening bracket and argument "$local_only"; 1 found

Open

    function status( $local_only = false ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

        if ( 0 !== $return ) { return false; }

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces between opening bracket and argument "$commit"; 1 found

Open

    function merge_initial_commit( $commit, $branch ) {

• Exclude checks
  • Disable engine
  • Disable check

Each PHP statement must be on a line by itself

Open

        $response = array_map( function( $b ) { return str_replace( "origin/", "", $b ); }, $response );

• Exclude checks
  • Disable engine
  • Disable check

Method name "Git_Wrapper::get_local_changes" is not in camel caps format

Open

    function get_local_changes() {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

        if ( 0 !== $return ) {

• Exclude checks
  • Disable engine
  • Disable check

Space found after opening bracket of FOR loop

Open

            for ( $idx = 0 ; $idx < count( $response ) / 2 ; $idx++ ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

        if ( 0 !== $return ) { return false; }

• Exclude checks
  • Disable engine
  • Disable check

Method name "Git_Wrapper::get_remote_branches" is not in camel caps format

Open

    function get_remote_branches() {

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on method "get_remote_branches"

Open

    function get_remote_branches() {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

            foreach ( $response as $idx => $line ) :

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

                if ( empty( $line ) ) { continue; } // ignore empty lines like the last item

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces between opening bracket and argument "$message"; 1 found

Open

    function commit( $message, $author_name = '', $author_email = '' ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

            if ( empty( $author_name ) ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

        if ( $local_only ) { return array( $branch_status, $new_response ); }

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

        if ( preg_match( '/## ([^.]+)\.+([^ ]+)/', $branch_status, $matches ) ) {

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on method "get_uncommited_changes"

Open

    function get_uncommited_changes() {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces between argument "$local_only" and closing bracket; 1 found

Open

    function status( $local_only = false ) {

• Exclude checks
  • Disable engine
  • Disable check

Space found before closing bracket of FOR loop

Open

            for ( $idx = 0 ; $idx < count( $response ) / 2 ; $idx++ ) {

• Exclude checks
  • Disable engine
  • Disable check

Method name "Git_Wrapper::get_uncommited_changes" is not in camel caps format

Open

    function get_uncommited_changes() {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces between opening bracket and argument "$n"; 1 found

Open

    function get_last_commits( $n = 20 ) {

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on method "remove_wp_content_from_version_control"

Open

    function remove_wp_content_from_version_control() {

• Exclude checks
  • Disable engine
  • Disable check

Space found after opening bracket of FOREACH loop

Open

            foreach ( $response as $idx => $line ) :

• Exclude checks
  • Disable engine
  • Disable check

Visibility must be declared on method "is_dirty"

Open

    function is_dirty() {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

            if ( $commit_info[1] != $commit_info[4] && $commit_info[2] != $commit_info[5] ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces between opening bracket and argument "$path"; 1 found

Open

    function rm_cached( $path ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

            foreach ( $response as $line ) :

• Exclude checks
  • Disable engine
  • Disable check

Method name "Git_Wrapper::is_dirty" is not in camel caps format

Open

    function is_dirty() {

• Exclude checks
  • Disable engine
  • Disable check

Method name "Git_Wrapper::remove_wp_content_from_version_control" is not in camel caps format

Open

    function remove_wp_content_from_version_control() {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

        if ( is_resource( $process ) ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

if ( ! defined( 'GIT_DIR' ) ) {

• Exclude checks
  • Disable engine
  • Disable check

Only one argument is allowed per line in a multi-line function call

Open

            'cherry-pick', '--strategy', 'recursive', '--strategy-option', 'theirs', $commit

• Exclude checks
  • Disable engine
  • Disable check

Only one argument is allowed per line in a multi-line function call

Open

            'cherry-pick', '--strategy', 'recursive', '--strategy-option', 'theirs', $commit

• Exclude checks
  • Disable engine
  • Disable check

Only one argument is allowed per line in a multi-line function call

Open

            'cherry-pick', '--strategy', 'recursive', '--strategy-option', 'theirs', $commit

• Exclude checks
  • Disable engine
  • Disable check

Only one argument is allowed per line in a multi-line function call

Open

            'cherry-pick', '--strategy', 'recursive', '--strategy-option', 'theirs', $commit

• Exclude checks
  • Disable engine
  • Disable check

Only one argument is allowed per line in a multi-line function call

Open

            'cherry-pick', '--strategy', 'recursive', '--strategy-option', 'theirs', $commit

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        $this->repo_dir = $repo_dir;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    function _log(...$args) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

            $env['GIT_SSH'] = dirname( __FILE__ ) . '/ssh-git';

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

                0 => array( 'pipe', 'r' ),  // stdin

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

            ),

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

            }

• Exclude checks
  • Disable engine
  • Disable check

Inline control structures are not allowed

Open

if (!defined('GITIGNORE'))

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    private $last_error = '';

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    function _rrmdir( $dir ) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        );

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

            $env['GIT_KEY_FILE'] = GIT_KEY_FILE;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

            $this->repo_dir,

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

                $response[] = rtrim( $line, "\n\r" );

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

            $this->last_error = null;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        $output = '';

• Exclude checks
  • Disable engine
  • Disable check

Inline control structures are not allowed

Open

        if (isset($args) && $args) foreach ( $args as $arg ) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        $this->private_key = $private_key;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        return $env;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

            unlink( $env['GIT_KEY_FILE'] );

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        } else {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

            $cmd,

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        foreach ( $files as $file ) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    function set_key( $private_key ) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        $args     = join( ' ', array_map( 'escapeshellarg', $args ) );

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    function is_status_working() {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    private $gitignore  = GITIGNORE;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

            ( is_dir( $filepath ) ) ? $this->_rrmdir( $filepath ) : unlink( $filepath );

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        $git_bin_path = apply_filters( 'gitium_git_bin_path', '' );

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

            return false;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        } else {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        $key_file = tempnam( sys_get_temp_dir(), 'ssh-git' );

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        if ( is_resource( $proc ) ) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        $this->_log( "$return $cmd", join( "\n", $response ) );

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    function can_exec_git() {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    private $repo_dir = '';

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    function __construct( $repo_dir ) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

            $filepath = realpath("$dir/$file");

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        if ($output) error_log($output);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        $response = array();

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        );

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        if ( 0 != $return ) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        return ( 0 == $return );

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    function _git_temp_key_file() {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

            'HOME' => getenv( 'HOME' ),

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        $proc = proc_open(

• Exclude checks
  • Disable engine
  • Disable check