AppClient::deleteApp() - Code Metrics - Inspection of "Add Rtc" - qiniu/php-sdk - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#267)

unknown

created 2018-05-09 02:22 UTC

AppClient::deleteApp() A

↳ Parent: AppClient

Complexity

Conditions	1
Paths	1

Size

Total Lines	6
Code Lines	4

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	1
eloc	4
nc	1
nop	1
dl	0
loc	6
rs	9.4285
c	1
b	0
f	0

<?php
namespace Qiniu\Rtc;

use Qiniu\Http\Client;
use Qiniu\Http\Error;
use Qiniu\Config;
use Qiniu\Auth;

class AppClient
{
    private $auth;
    private $_baseURL;

    public function __construct(Auth $auth)
    {
        $this->auth = $auth;

        $this->_baseURL = sprintf("%s/%s/apps", Config::RTCAPI_HOST, Config::RTCAPI_VERSION);
    }

    /*
     * hub: 直播空间名
     * title: app 的名称  注意，Title 不是唯一标识，重复 create 动作将生成多个 app
     * maxUsers：人数限制
     * NoAutoKickUser: bool 类型，可选，禁止自动踢人（抢流）。默认为 false ，即同一个身份的 client (app/room/user) ，新的连麦请求可以成功，旧连接被关闭。
     */
    public function createApp($hub, $title, $maxUsers = null, $noAutoKickUser = null)
    {
        $params['hub'] = $hub;
foreach ($collection as $item) {
    $myArray['foo'] = $item->getFoo();

    if ($item->hasBar()) {
        $myArray['bar'] = $item->getBar();
    }

    // do something with $myArray
}
        $params['title'] = $title;
        if (!empty($maxUsers)) {
            $params['maxUsers'] = $maxUsers;
        }
        if (!empty($noAutoKickUser)) {
            $params['noAutoKickUser'] = $noAutoKickUser;
        }
        $body = json_encode($params);
        $ret = $this->post($this->_baseURL, $body);
        return $ret;
    }

    /*
     * appId: app 的唯一标识，创建的时候由系统生成。
     * Title: app 的名称， 可选。
     * Hub: 绑定的直播 hub，可选，用于合流后 rtmp 推流。
     * MaxUsers: int 类型，可选，连麦房间支持的最大在线人数。
     * NoAutoKickUser: bool 类型，可选，禁止自动踢人。
     * MergePublishRtmp: 连麦合流转推 RTMP 的配置，可选择。其详细配置包括如下
            Enable: 布尔类型，用于开启和关闭所有房间的合流功能。
            AudioOnly: 布尔类型，可选，指定是否只合成音频。
            Height, Width: int64，可选，指定合流输出的高和宽，默认为 640 x 480。
            OutputFps: int64，可选，指定合流输出的帧率，默认为 25 fps 。
            OutputKbps: int64，可选，指定合流输出的码率，默认为 1000 。
            URL: 合流后转推旁路直播的地址，可选，支持魔法变量配置按照连麦房间号生成不同的推流地址。如果是转推到七牛直播云，不建议使用该配置。
            StreamTitle: 转推七牛直播云的流名，可选，支持魔法变量配置按照连麦房间号生成不同的流名。例如，配置 Hub 为 qn-zhibo ，配置 StreamTitle 为 $(roomName) ，则房间 meeting-001 的合流将会被转推到 rtmp://pili-publish.qn-zhibo.***.com/qn-zhibo/meeting-001地址。详细配置细则，请咨询七牛技术支持。
     */
    public function updateApp($appId, $hub, $title, $maxUsers = null, $mergePublishRtmp = null, $noAutoKickUser = null)
    {
        $url = $this->_baseURL . '/' . $appId;
        $params['hub'] = $hub;
foreach ($collection as $item) {
    $myArray['foo'] = $item->getFoo();

    if ($item->hasBar()) {
        $myArray['bar'] = $item->getBar();
    }

    // do something with $myArray
}
        $params['title'] = $title;
        if (!empty($maxUsers)) {
            $params['maxUsers'] = $maxUsers;
        }
        if (!empty($noAutoKickUser)) {
            $params['noAutoKickUser'] = $noAutoKickUser;
        }
        if (!empty($mergePublishRtmp)) {
            $params['mergePublishRtmp'] = $mergePublishRtmp;
        }
        $body = json_encode($params);
        $ret = $this->post($url, $body);
        return $ret;
    }

    /*
     * appId: app 的唯一标识，创建的时候由系统生成。
     */
    public function getApp($appId)
    {
        $url = $this->_baseURL . '/' . $appId;
        $ret  = $this->get($url);
        return $ret;
    }

    /*
     * appId: app 的唯一标识，创建的时候由系统生成
     */
    public function deleteApp($appId)
    {
        $url = $this->_baseURL . '/' . $appId;
        list(, $err)  = $this->delete($url);
        return $err;
    }

    /*
     * 获取房间的人数
     * appId: app 的唯一标识，创建的时候由系统生成。
     * roomName: 操作所查询的连麦房间。
     */
    public function getappUserNum($appId, $roomName)
    {
        $url = sprintf("%s/%s/rooms/%s/users", $this->_baseURL, $appId, $roomName);
        $ret  = $this->get($url);
        return $ret;
    }

   /*
    * 踢出玩家
    * appId: app 的唯一标识，创建的时候由系统生成。
    * roomName: 连麦房间
    * userId: 请求加入房间的用户ID
    */
    public function kickingPlayer($appId, $roomName, $userId)
    {
        $url = sprintf("%s/%s/rooms/%s/users/%s", $this->_baseURL, $appId, $roomName, $userId);
        list(, $err)  = $this->delete($url);
        return $err;
    }

    /*
     * 获取房间的人数
     * appId: app 的唯一标识，创建的时候由系统生成。
     * prefix: 所查询房间名的前缀索引，可以为空。
     * offset: int 类型，分页查询的位移标记。
     * limit: int 类型，此次查询的最大长度。
     * GET /v3/apps/<AppID>/rooms?prefix=<RoomNamePrefix>&offset=<Offset>&limit=<Limit>
     */
    public function listRooms($appId, $prefix = null, $offset = null, $limit = null)
    {
        if(isset($prefix)){
            $query['prefix'] = $prefix;
foreach ($collection as $item) {
    $myArray['foo'] = $item->getFoo();

    if ($item->hasBar()) {
        $myArray['bar'] = $item->getBar();
    }

    // do something with $myArray
}
        }
        if(isset($offset)){
            $query['offset'] = $offset;
function myFunction($a) {
    switch ($a) {
        case 'foo':
            $x = 1;
            break;

        case 'bar':
            $x = 2;
            break;
    }

    // $x is potentially undefined here.
    echo $x;
}
        }
        if(isset($limit)){
            $query['limit'] = $limit;
        }
        if ($query != null) {
            $query = '?' . http_build_query($query);
            $url = sprintf("%s/%s/rooms%s", $this->_baseURL, $appId, $query);
        } else {
            $url = sprintf("%s/%s/rooms", $this->_baseURL, $appId);
        }
        $ret  = $this->get($url);
        return $ret;
    }

    /*
     * appId: app 的唯一标识，创建的时候由系统生成。
     * roomName: 房间名称，需满足规格 ^[a-zA-Z0-9_-]{3,64}$
     * userId: 请求加入房间的用户 ID，需满足规格 ^[a-zA-Z0-9_-]{3,50}$
     * expireAt: int64 类型，鉴权的有效时间，传入以秒为单位的64位Unix绝对时间，token 将在该时间后失效。
     * permission: 该用户的房间管理权限，"admin" 或 "user"，默认为 "user" 。当权限角色为 "admin" 时，拥有将其他用户移除出房间等特权.
     */
    public function appToken($appId, $roomName, $userId, $expireAt, $permission)
    {
        $params['appId'] = $appId;
foreach ($collection as $item) {
    $myArray['foo'] = $item->getFoo();

    if ($item->hasBar()) {
        $myArray['bar'] = $item->getBar();
    }

    // do something with $myArray
}
        $params['userId'] = $userId;
        $params['roomName'] = $roomName;        
        $params['permission'] = $permission;
        $params['expireAt'] = $expireAt;
        $appAccessString = json_encode($params);
        return $this->auth->signWithData($appAccessString);
    }

    private function get($url, $cType = null)
    {
        $rtcToken = $this->auth->authorizationV2($url, "GET", $body, $cType);

        $rtcToken['Content-Type'] = $cType;
        $ret = Client::get($url, $rtcToken);
        if (!$ret->ok()) {
            return array(null, new Error($url, $ret));
        }
        return array($ret->json(), null);
    }

    private function delete($url, $cType = 'application/json')
    {
        $rtcToken = $this->auth->authorizationV2($url, "DELETE", $body, $cType);

        $rtcToken['Content-Type'] = $cType;
        $ret = Client::delete($url, $rtcToken);
        if (!$ret->ok()) {
            return array(null, new Error($url, $ret));
        }
        return array($ret->json(), null);
    }

    private function post($url, $body, $cType = 'application/json')
    {
        $rtcToken = $this->auth->authorizationV2($url, "POST", $body, $cType);
        $rtcToken['Content-Type'] = $cType;
        $ret = Client::post($url, $body, $rtcToken);
        if (!$ret->ok()) {
            return array(null, new Error($url, $ret));
        }
        $r = ($ret->body === null) ? array() : $ret->json();
        return array($r, null);
    }
}


1			<?php
2			namespace Qiniu\Rtc;
3
4			use Qiniu\Http\Client;
5			use Qiniu\Http\Error;
6			use Qiniu\Config;
7			use Qiniu\Auth;
8
9			class AppClient
10			{
11			private $auth;
12			private $_baseURL;
13
14			public function __construct(Auth $auth)
15			{
16			$this->auth = $auth;
17
18			$this->_baseURL = sprintf("%s/%s/apps", Config::RTCAPI_HOST, Config::RTCAPI_VERSION);
19			}
20
21			/*
22			* hub: 直播空间名
23			* title: app 的名称注意，Title 不是唯一标识，重复 create 动作将生成多个 app
24			* maxUsers：人数限制
25			* NoAutoKickUser: bool 类型，可选，禁止自动踢人（抢流）。默认为 false ，即同一个身份的 client (app/room/user) ，新的连麦请求可以成功，旧连接被关闭。
26			*/
27			public function createApp($hub, $title, $maxUsers = null, $noAutoKickUser = null)
28			{
29			$params['hub'] = $hub;
			0 ignored issues – show Coding Style Comprehensibility introduced 2018-05-07 03:52 UTC by Report Bug Copy Issue Report `$params` was never initialized. Although not strictly required by PHP, it is generally a good practice to add `$params = array();` before regardless. Adding an explicit array definition is generally preferable to implicit array definition as it guarantees a stable state of the code. Let’s take a look at an example: foreach ($collection as $item) { $myArray['foo'] = $item->getFoo(); if ($item->hasBar()) { $myArray['bar'] = $item->getBar(); } // do something with $myArray } As you can see in this example, the array `$myArray` is initialized the first time when the foreach loop is entered. You can also see that the value of the `bar` key is only written conditionally; thus, its value might result from a previous iteration. This might or might not be intended. To make your intention clear, your code more readible and to avoid accidental bugs, we recommend to add an explicit initialization `$myArray = array()` either outside or inside the foreach loop. Loading history...
30			$params['title'] = $title;
31			if (!empty($maxUsers)) {
32			$params['maxUsers'] = $maxUsers;
33			}
34			if (!empty($noAutoKickUser)) {
35			$params['noAutoKickUser'] = $noAutoKickUser;
36			}
37			$body = json_encode($params);
38			$ret = $this->post($this->_baseURL, $body);
39			return $ret;
40			}
41
42			/*
43			* appId: app 的唯一标识，创建的时候由系统生成。
44			* Title: app 的名称，可选。
45			* Hub: 绑定的直播 hub，可选，用于合流后 rtmp 推流。
46			* MaxUsers: int 类型，可选，连麦房间支持的最大在线人数。
47			* NoAutoKickUser: bool 类型，可选，禁止自动踢人。
48			* MergePublishRtmp: 连麦合流转推 RTMP 的配置，可选择。其详细配置包括如下
49			Enable: 布尔类型，用于开启和关闭所有房间的合流功能。
50			AudioOnly: 布尔类型，可选，指定是否只合成音频。
51			Height, Width: int64，可选，指定合流输出的高和宽，默认为 640 x 480。
52			OutputFps: int64，可选，指定合流输出的帧率，默认为 25 fps 。
53			OutputKbps: int64，可选，指定合流输出的码率，默认为 1000 。
54			URL: 合流后转推旁路直播的地址，可选，支持魔法变量配置按照连麦房间号生成不同的推流地址。如果是转推到七牛直播云，不建议使用该配置。
55			StreamTitle: 转推七牛直播云的流名，可选，支持魔法变量配置按照连麦房间号生成不同的流名。例如，配置 Hub 为 qn-zhibo ，配置 StreamTitle 为 $(roomName) ，则房间 meeting-001 的合流将会被转推到 rtmp://pili-publish.qn-zhibo.***.com/qn-zhibo/meeting-001地址。详细配置细则，请咨询七牛技术支持。
56			*/
57			public function updateApp($appId, $hub, $title, $maxUsers = null, $mergePublishRtmp = null, $noAutoKickUser = null)
58			{
59			$url = $this->_baseURL . '/' . $appId;
60			$params['hub'] = $hub;
			0 ignored issues – show Coding Style Comprehensibility introduced 2018-04-17 10:04 UTC by Report Bug Copy Issue Report `$params` was never initialized. Although not strictly required by PHP, it is generally a good practice to add `$params = array();` before regardless. Adding an explicit array definition is generally preferable to implicit array definition as it guarantees a stable state of the code. Let’s take a look at an example: foreach ($collection as $item) { $myArray['foo'] = $item->getFoo(); if ($item->hasBar()) { $myArray['bar'] = $item->getBar(); } // do something with $myArray } As you can see in this example, the array `$myArray` is initialized the first time when the foreach loop is entered. You can also see that the value of the `bar` key is only written conditionally; thus, its value might result from a previous iteration. This might or might not be intended. To make your intention clear, your code more readible and to avoid accidental bugs, we recommend to add an explicit initialization `$myArray = array()` either outside or inside the foreach loop. Loading history...
61			$params['title'] = $title;
62			if (!empty($maxUsers)) {
63			$params['maxUsers'] = $maxUsers;
64			}
65			if (!empty($noAutoKickUser)) {
66			$params['noAutoKickUser'] = $noAutoKickUser;
67			}
68			if (!empty($mergePublishRtmp)) {
69			$params['mergePublishRtmp'] = $mergePublishRtmp;
70			}
71			$body = json_encode($params);
72			$ret = $this->post($url, $body);
73			return $ret;
74			}
75
76			/*
77			* appId: app 的唯一标识，创建的时候由系统生成。
78			*/
79			public function getApp($appId)
80			{
81			$url = $this->_baseURL . '/' . $appId;
82			$ret = $this->get($url);
83			return $ret;
84			}
85
86			/*
87			* appId: app 的唯一标识，创建的时候由系统生成
88			*/
89			public function deleteApp($appId)
90			{
91			$url = $this->_baseURL . '/' . $appId;
92			list(, $err) = $this->delete($url);
93			return $err;
94			}
95
96			/*
97			* 获取房间的人数
98			* appId: app 的唯一标识，创建的时候由系统生成。
99			* roomName: 操作所查询的连麦房间。
100			*/
101			public function getappUserNum($appId, $roomName)
102			{
103			$url = sprintf("%s/%s/rooms/%s/users", $this->_baseURL, $appId, $roomName);
104			$ret = $this->get($url);
105			return $ret;
106			}
107
108			/*
109			* 踢出玩家
110			* appId: app 的唯一标识，创建的时候由系统生成。
111			* roomName: 连麦房间
112			* userId: 请求加入房间的用户ID
113			*/
114			public function kickingPlayer($appId, $roomName, $userId)
115			{
116			$url = sprintf("%s/%s/rooms/%s/users/%s", $this->_baseURL, $appId, $roomName, $userId);
117			list(, $err) = $this->delete($url);
118			return $err;
119			}
120
121			/*
122			* 获取房间的人数
123			* appId: app 的唯一标识，创建的时候由系统生成。
124			* prefix: 所查询房间名的前缀索引，可以为空。
125			* offset: int 类型，分页查询的位移标记。
126			* limit: int 类型，此次查询的最大长度。
127			* GET /v3/apps/<AppID>/rooms?prefix=<RoomNamePrefix>&offset=<Offset>&limit=<Limit>
128			*/
129			public function listRooms($appId, $prefix = null, $offset = null, $limit = null)
130			{
131			if(isset($prefix)){
132			$query['prefix'] = $prefix;
			0 ignored issues – show Coding Style Comprehensibility introduced 2018-04-17 12:19 UTC by Report Bug Copy Issue Report `$query` was never initialized. Although not strictly required by PHP, it is generally a good practice to add `$query = array();` before regardless. Adding an explicit array definition is generally preferable to implicit array definition as it guarantees a stable state of the code. Let’s take a look at an example: foreach ($collection as $item) { $myArray['foo'] = $item->getFoo(); if ($item->hasBar()) { $myArray['bar'] = $item->getBar(); } // do something with $myArray } As you can see in this example, the array `$myArray` is initialized the first time when the foreach loop is entered. You can also see that the value of the `bar` key is only written conditionally; thus, its value might result from a previous iteration. This might or might not be intended. To make your intention clear, your code more readible and to avoid accidental bugs, we recommend to add an explicit initialization `$myArray = array()` either outside or inside the foreach loop. Loading history...
133			}
134			if(isset($offset)){
135			$query['offset'] = $offset;
			0 ignored issues – show Bug introduced 2018-04-17 12:19 UTC by Report Bug Copy Issue Report The variable `$query` does not seem to be defined for all execution paths leading up to this point. If you define a variable conditionally, it can happen that it is not defined for all execution paths. Let’s take a look at an example: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } // $x is potentially undefined here. echo $x; } In the above example, the variable `$x` is defined if you pass “foo” or “bar” as argument for `$a`. However, since the `switch` statement has no default case statement, if you pass any other value, the variable `$x` would be undefined. Available Fixes Check for existence of the variable explicitly: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } if (isset($x)) { // Make sure it's always set. echo $x; } } Define a default value for the variable: function myFunction($a) { $x = ''; // Set a default which gets overridden for certain paths. switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } echo $x; } Add a value for the missing path: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; // We add support for the missing case. default: $x = ''; break; } echo $x; } Loading history...
136			}
137			if(isset($limit)){
138			$query['limit'] = $limit;
139			}
140			if ($query != null) {
141			$query = '?' . http_build_query($query);
142			$url = sprintf("%s/%s/rooms%s", $this->_baseURL, $appId, $query);
143			} else {
144			$url = sprintf("%s/%s/rooms", $this->_baseURL, $appId);
145			}
146			$ret = $this->get($url);
147			return $ret;
148			}
149
150			/*
151			* appId: app 的唯一标识，创建的时候由系统生成。
152			* roomName: 房间名称，需满足规格 ^[a-zA-Z0-9_-]{3,64}$
153			* userId: 请求加入房间的用户 ID，需满足规格 ^[a-zA-Z0-9_-]{3,50}$
154			* expireAt: int64 类型，鉴权的有效时间，传入以秒为单位的64位Unix绝对时间，token 将在该时间后失效。
155			* permission: 该用户的房间管理权限，"admin" 或 "user"，默认为 "user" 。当权限角色为 "admin" 时，拥有将其他用户移除出房间等特权.
156			*/
157			public function appToken($appId, $roomName, $userId, $expireAt, $permission)
158			{
159			$params['appId'] = $appId;
			0 ignored issues – show Coding Style Comprehensibility introduced 2018-04-17 10:04 UTC by Report Bug Copy Issue Report `$params` was never initialized. Although not strictly required by PHP, it is generally a good practice to add `$params = array();` before regardless. Adding an explicit array definition is generally preferable to implicit array definition as it guarantees a stable state of the code. Let’s take a look at an example: foreach ($collection as $item) { $myArray['foo'] = $item->getFoo(); if ($item->hasBar()) { $myArray['bar'] = $item->getBar(); } // do something with $myArray } As you can see in this example, the array `$myArray` is initialized the first time when the foreach loop is entered. You can also see that the value of the `bar` key is only written conditionally; thus, its value might result from a previous iteration. This might or might not be intended. To make your intention clear, your code more readible and to avoid accidental bugs, we recommend to add an explicit initialization `$myArray = array()` either outside or inside the foreach loop. Loading history...
160			$params['userId'] = $userId;
161			$params['roomName'] = $roomName;
162			$params['permission'] = $permission;
163			$params['expireAt'] = $expireAt;
164			$appAccessString = json_encode($params);
165			return $this->auth->signWithData($appAccessString);
166			}
167
168			private function get($url, $cType = null)
169			{
170			$rtcToken = $this->auth->authorizationV2($url, "GET", $body, $cType);
			0 ignored issues – show Bug introduced 2018-05-08 13:48 UTC by Report Bug Copy Issue Report The variable `$body` does not exist. Did you forget to declare it? This check marks access to variables or properties that have not been declared yet. While PHP has no explicit notion of declaring a variable, accessing it before a value is assigned to it is most likely a bug. Loading history...
171			$rtcToken['Content-Type'] = $cType;
172			$ret = Client::get($url, $rtcToken);
173			if (!$ret->ok()) {
174			return array(null, new Error($url, $ret));
175			}
176			return array($ret->json(), null);
177			}
178
179			private function delete($url, $cType = 'application/json')
180			{
181			$rtcToken = $this->auth->authorizationV2($url, "DELETE", $body, $cType);
			0 ignored issues – show Bug introduced 2018-05-08 13:48 UTC by Report Bug Copy Issue Report The variable `$body` does not exist. Did you forget to declare it? This check marks access to variables or properties that have not been declared yet. While PHP has no explicit notion of declaring a variable, accessing it before a value is assigned to it is most likely a bug. Loading history...
182			$rtcToken['Content-Type'] = $cType;
183			$ret = Client::delete($url, $rtcToken);
184			if (!$ret->ok()) {
185			return array(null, new Error($url, $ret));
186			}
187			return array($ret->json(), null);
188			}
189
190			private function post($url, $body, $cType = 'application/json')
191			{
192			$rtcToken = $this->auth->authorizationV2($url, "POST", $body, $cType);
193			$rtcToken['Content-Type'] = $cType;
194			$ret = Client::post($url, $body, $rtcToken);
195			if (!$ret->ok()) {
196			return array(null, new Error($url, $ret));
197			}
198			$r = ($ret->body === null) ? array() : $ret->json();
199			return array($r, null);
200			}
201			}
202

qiniu / php-sdk

Pull Request — master (#267)

AppClient::deleteApp() A

Complexity

Size

Duplication

Importance

Available Fixes

Duplication Side-by-Side

Filter issues like