initHooks::initHooksCreate() - Code Metrics - Inspection of "split service based on bigbluebutton functionality" - joisarjignesh/bigbluebutton - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 23650d...b8cfa5 )

by Jignesh

created 2020-07-29 17:31 UTC

initHooks::initHooksCreate() A

↳ Parent: initHooks

Complexity

Conditions	2
Paths	2

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	2
nc	2
nop	1
dl	0
loc	11
rs	9.9
c	0
b	0
f	0

<?php


namespace JoisarJignesh\Bigbluebutton\Services;


use BigBlueButton\Parameters\HooksCreateParameters;
use BigBlueButton\Parameters\HooksDestroyParameters;

trait initHooks
{
    /**
     * @param array $parameters
     *
     * require fields
     * callbackURL
     *
     * optional fields
     * meetingID
     * getRaw
     *
     * @return HooksCreateParameters
     */
    public function initHooksCreate(array $parameters)
    {
        $parameters = Fluent($parameters);
        $hooksCreate = new HooksCreateParameters($parameters->get('callbackURL'));
        if ($parameters->meetingID) {
            $hooksCreate->setMeetingId($parameters->meetingID);
        }
        $hooksCreate->setGetRaw($parameters->get('getRaw', false));

        return $hooksCreate;
    }

    /**
     * @param mixed $parameters
     *
     * @return HooksDestroyParameters
     */
    public function initHooksDestroy($parameters)
    {
        $hooksID = "";
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
        if (is_array($parameters)) {
            $hooksID = Fluent($parameters)->get('hooksID');
        } else {
            $hooksID = $parameters;
        }

        return new HooksDestroyParameters($hooksID);
    }


}


1			<?php
2
3
4			namespace JoisarJignesh\Bigbluebutton\Services;
5
6
7			use BigBlueButton\Parameters\HooksCreateParameters;
8			use BigBlueButton\Parameters\HooksDestroyParameters;
9
10			trait initHooks
11			{
12			/**
13			* @param array $parameters
14			*
15			* require fields
16			* callbackURL
17			*
18			* optional fields
19			* meetingID
20			* getRaw
21			*
22			* @return HooksCreateParameters
23			*/
24			public function initHooksCreate(array $parameters)
25			{
26			$parameters = Fluent($parameters);
27			$hooksCreate = new HooksCreateParameters($parameters->get('callbackURL'));
28			if ($parameters->meetingID) {
29			$hooksCreate->setMeetingId($parameters->meetingID);
30			}
31			$hooksCreate->setGetRaw($parameters->get('getRaw', false));
32
33			return $hooksCreate;
34			}
35
36			/**
37			* @param mixed $parameters
38			*
39			* @return HooksDestroyParameters
40			*/
41			public function initHooksDestroy($parameters)
42			{
43			$hooksID = "";
			0 ignored issues – show Unused Code introduced 2020-06-26 19:48 UTC by Report Bug Copy Issue Report `$hooksID` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
44			if (is_array($parameters)) {
45			$hooksID = Fluent($parameters)->get('hooksID');
46			} else {
47			$hooksID = $parameters;
48			}
49
50			return new HooksDestroyParameters($hooksID);
51			}
52
53
54			}
55

joisarjignesh / bigbluebutton

Push — master ( 23650d...b8cfa5 )

initHooks::initHooksCreate() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like