Method recipe_run has 5 arguments (exceeds 4 allowed). Consider refactoring.

Open

    public function recipe_run($rest_host, $rest_port, $username, $password, $task_id)

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

Open

    public function TargetHostRegistration($hosts, $dbh, $machine_id, $user_id) {
        $status_id = 'unknown';
        $os = 'unknown';
        $query = $dbh->prepare('INSERT INTO target_host (ipaddress, port, groups, os, status_id, machine_id, user_id)
        VALUES (:ipaddress, :port, :groups, :os, :status_id, :machine_id, :user_id);');

• Read up
  Read up

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

Open

    public function isInstalled($dba,$dbh,$search,$update,$target_host){
 
        $return = [];
 
        $return['msg'] = '';

• Read up
  Read up

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

Open

    public function variable_list($rest_host, $rest_port, $username, $password)
    {
        try {
            $host_id=1;
            $uri = 'http://' . $rest_host . ':' . $rest_port . '/eisen/api/v1.0/host/'.$host_id.'/vars';

• Read up
  Read up

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

Open

    public function isRepository($dba,$dbh,$search,$update,$target_host, $start_row, $row_in_table) {
        $return = array();
 
        $return['msg'] = '';
        $return['error'] = false;

• Read up
  Read up

Avoid too many return statements within this function.

Open

                if ( aFloat > bFloat ) { return 1; }

Avoid too many return statements within this function.

Open

                if ( aFloat < bFloat ) { return -1; }

Avoid too many return statements within this function.

Open

            return $.trim( $node[ 0 ].textContent || $node.text() );

Avoid too many return statements within this function.

Open

                    return $.trim( tmp( $node[ 0 ], c.table, cellIndex ) );

Avoid too many return statements within this function.

Open

            return 0;