Sugarcrm\UpgradeSpec\Helper\Sugarcrm

Complexity

Total Complexity

6

Size/Duplication

Total Lines	50
Duplicated Lines	40 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	getBuildVersion()	10	10	2
A	getBuildFlav()	10	10	2
A	isSugarcrmBuild()	0	4	2

<?php

namespace Sugarcrm\UpgradeSpec\Helper;

use Sugarcrm\UpgradeSpec\Version\Version;

class Sugarcrm
{
    /**
     * Extracts version number from sugarcrm build files.
     *
     * @param $path
     *
     * @return Version
     */
    public function getBuildVersion($path)

This method seems to be duplicated in your project.

    {
        if (!defined('sugarEntry')) {
            define('sugarEntry', true);

This constant is not in uppercase (expected 'SUGARENTRY').

        }

        require realpath($path) . '/sugar_version.php';

        return new Version($sugar_version);

The variable $sugar_version does not exist. Did you forget to declare it?

    }

    /**
     * Extracts flav from sugarcrm build files.
     *
     * @param $path
     *
     * @return string
     */
    public function getBuildFlav($path)

This method seems to be duplicated in your project.

    {
        if (!defined('sugarEntry')) {
            define('sugarEntry', true);

This constant is not in uppercase (expected 'SUGARENTRY').

        }

        require realpath($path) . '/sugar_version.php';

        return $sugar_flavor;

The variable $sugar_flavor does not exist. Did you forget to declare it?

    }

    /**
     * Verifies if given path represents a valid SugarCRM build.
     *
     * @param $path
     *
     * @return bool
     */
    public function isSugarcrmBuild($path)
    {
        return $path && file_exists(realpath($path) . '/sugar_version.php');
    }
}