PressLabs/gitium

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

Open

        $plugin_versions[ $name ]['msg'] = "`{$data['Name']}`";

• 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;
    }
}

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

Open

                <div style="float:left; width:auto; height:auto;"><strong><?php echo esc_html( $subject ); ?></strong><br />

• 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 '$author_name' which will lead to PHP notices.

Open

                <span title="<?php echo esc_attr( $author_email ); ?>"><?php echo esc_html( $author_name ) . ' ' . sprintf( __( 'authored %s ago', 'gitium' ), human_time_diff( strtotime( $author_date ) ) ); ?></span><?php echo $committer; ?></div>

• 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 '$theme_versions' which will lead to PHP notices.

Open

        $theme_versions[ $theme_name ]['msg'] = '`' . $theme->Name . '`';

• 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 '$theme_versions' which will lead to PHP notices.

Open

        $new_versions['themes'] = $theme_versions;

• 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 '$committer_date' which will lead to PHP notices.

Open

                $committer         = "<span title='$committer_email'> -> $committer_name " . sprintf( __( 'committed %s ago', 'gitium' ), human_time_diff( strtotime( $committer_date ) ) ) . '</span>';

• 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 '$committer_name' which will lead to PHP notices.

Open

                $committer         = "<span title='$committer_email'> -> $committer_name " . sprintf( __( 'committed %s ago', 'gitium' ), human_time_diff( strtotime( $committer_date ) ) ) . '</span>';

• 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 '$author_email' which will lead to PHP notices.

Open

                <div style="float:left; width:auto; height:auto; padding-left:2px; padding-right:5px; padding-top:2px; margin-right:5px; border-radius:2px"><?php echo get_avatar( $author_email, 32 ); ?></div>

• 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 '$committer_name' which will lead to PHP notices.

Open

            if ( isset( $committer_name ) ) {

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

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

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];

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

Open

    private function setup_step_2() {
        $git = $this->git; ?>
        <div class="wrap">
        <h2><?php _e( 'Configuration step 2', 'gitium' ); ?></h2>
        <p><?php _e( 'If you need help to set this up, please click on the "Help" button from the top right corner of this screen.' ); ?></p>

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

Open

    private function show_git_changes_table_rows( $changes = '' ) {
        ?>
        <script type="application/javascript">
        function add_path_and_submit( elem ) {
            var container = document.getElementById( 'form_status' );

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

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

Open

class Gitium_Menu_Bubble extends Gitium_Menu {

• Exclude checks
  • Disable engine
  • Disable check

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

Open

class Gitium_Admin {

• Exclude checks
  • Disable engine
  • Disable check

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

Open

class Gitium_Help {

• Exclude checks
  • Disable engine
  • Disable check

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

Open

class Gitium_Menu {

• Exclude checks
  • Disable engine
  • Disable check

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

Open

class Gitium_Submenu_Commits extends Gitium_Menu {

• Exclude checks
  • Disable engine
  • Disable check

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

Open

class Gitium_Submenu_Configure extends Gitium_Menu {

• Exclude checks
  • Disable engine
  • Disable check