BaseCircle::generateUniqueId() - Code Metrics - Inspection of "display Federated Links into Circle" - nextcloud/circles - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 48e0cc...46633a )

by Maxence

created 2017-06-10 21:39 UTC

BaseCircle::generateUniqueId() A

↳ Parent: BaseCircle

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	3

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
c	0
b	0
f	0
dl	0
loc	4
rs	10
cc	1
eloc	3
nc	1
nop	0

<?php
/**
 * Circles - Bring cloud-users closer together.
 *
 * This file is licensed under the Affero General Public License version 3 or
 * later. See the COPYING file.
 *
 * @author Maxence Lange <[email protected]>
 * @copyright 2017
 * @license GNU AGPL version 3 or any later version
 *
 * This program is free software: you can redistribute it and/or modify
 * it under the terms of the GNU Affero General Public License as
 * published by the Free Software Foundation, either version 3 of the
 * License, or (at your option) any later version.
 *
 * This program is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 * GNU Affero General Public License for more details.
 *
 * You should have received a copy of the GNU Affero General Public License
 * along with this program.  If not, see <http://www.gnu.org/licenses/>.
 *
 */

namespace OCA\Circles\Model;

use OC\L10N\L10N;

class BaseCircle {

	const CIRCLES_SETTINGS_DEFAULT = [
		'allow_links'       => 'false',
		'allow_links_auto'  => 'false',
		'allow_links_files' => 'false'
	];

	const CIRCLES_PERSONAL = 1;
	const CIRCLES_HIDDEN = 2;
	const CIRCLES_PRIVATE = 4;
	const CIRCLES_PUBLIC = 8;

	const CIRCLES_ALL = 15;

	/** @var int */
	private $id;

	/** @var L10N */
	protected $l10n;

	/** @var string */
	private $uniqueId;

	/** @var string */
	private $name;

	/** @var Member */
	private $owner;

	/** @var Member */
	private $user;

	/** @var string */
	private $description = '';

	/** @var array */
	private $settings = [];

	/** @var int */
	private $type;

	/** @var string */
	private $creation;

	/** @var Member[] */
	private $members;

	/** @var FederatedLink[] */
	private $links;

	/** @var FederatedLink[] */
	private $remote;

	public function __construct($l10n, $type = -1, $name = '') {
		$this->l10n = $l10n;

		if ($type > -1) {
			$this->type = $type;
		}
		if ($name !== '') {
			$this->name = $name;
		}
	}


	public function setId($id) {
		$this->id = $id;

		return $this;
	}

	public function getId() {
		return $this->id;
	}


	/**
	 * @param string $uniqueId
	 *
	 * @return $this
	 */
	public function setUniqueId($uniqueId) {
		$this->uniqueId = (string)$uniqueId;

		return $this;
	}

	/**
	 * @return string
	 */
	public function getUniqueId() {
		return $this->uniqueId;
	}

	public function generateUniqueId() {
		$uniqueId = bin2hex(openssl_random_pseudo_bytes(24));
		$this->setUniqueId($uniqueId);
	}

	public function setName($name) {
		$this->name = $name;

		return $this;
	}

	public function getName() {
		return $this->name;
	}


	public function getOwner() {
		return $this->owner;
	}

	public function setOwner($owner) {
		$this->owner = $owner;

		return $this;
	}


	/**
	 * @return Member
	 */
	public function getUser() {
		return $this->user;
	}

	public function setUser($user) {
		$this->user = $user;

		return $this;
	}


	public function setDescription($description) {
		$this->description = $description;

		return $this;
	}

	public function getDescription() {
		return $this->description;
	}


	public function setSettings($settings) {
		if (is_array($settings)) {
			$this->settings = $settings;
		} else if (is_string($settings)) {
			$this->settings = json_decode($settings, true);

		}

		return $this;
	}

	public function getSettings($json = false) {

		if ($json) {
			return json_encode($this->settings);
		}

		$settings = $this->settings;
		if ($settings === null) {
			$settings = [];
		}

		$ak = array_keys(self::CIRCLES_SETTINGS_DEFAULT);
		foreach ($ak AS $k) {
			if (!key_exists($k, $settings)) {
				$settings[$k] = self::CIRCLES_SETTINGS_DEFAULT[$k];
			}
		}

		return $settings;
	}


	public function setSetting($k, $v) {
		$this->settings[$k] = $v;
	}


	/**
	 * @param string $k
	 *
	 * @return string|null
	 */
	public function getSetting($k) {
		if (key_exists($k, $this->settings)) {
			return $this->settings[$k];
		}
		if (key_exists($k, (array)self::CIRCLES_SETTINGS_DEFAULT)) {
			return self::CIRCLES_SETTINGS_DEFAULT[$k];
		}

		return null;
	}


	public function setType($type) {
		$this->type = self::typeInt($type);

		return $this;
	}

	public function getType() {
		return $this->type;
	}


	public function setCreation($creation) {
		$this->creation = $creation;

		return $this;
	}

	public function getCreation() {
		return $this->creation;
	}


	public function setMembers($members) {
		$this->members = $members;

		return $this;
	}

	public function getMembers() {
		return $this->members;
	}


	public function setLinks($links) {
		$this->links = $links;

		return $this;
	}

	public function getLinks() {
		return $this->links;
	}


//	public function getRemote() {

//		return $this->remote;
//	}
//
//	public function addRemote($link) {
//		array_push($this->remote, $link);
//	}
//
//	public function getRemoteFromToken($token) {
//		foreach ($this->links AS $link) {
//			if ($link->getToken() === $token) {
//				return $link;
//			}
//		}
//
//		return null;
//	}
//
//	public function getRemoteFromAddressAndId($address, $id) {
//		foreach ($this->links AS $link) {
//			if ($link->getAddress() === $address && $link->getUniqueId() === $id) {
//				return $link;
//			}
//		}
//
//		return null;
//	}


	public static function typeInt($type) {

		if (is_numeric($type)) {
			return (int)$type;
		}

		switch ($type) {

			case 'Personal':
				return self::CIRCLES_PERSONAL;
			case 'Private':
				return self::CIRCLES_PRIVATE;
			case 'Hidden':
				return self::CIRCLES_HIDDEN;
			case 'Public':
				return self::CIRCLES_PUBLIC;
		}

		return 0;
	}
}

1		<?php
2		/**
3		* Circles - Bring cloud-users closer together.
4		*
5		* This file is licensed under the Affero General Public License version 3 or
6		* later. See the COPYING file.
7		*
8		* @author Maxence Lange <[email protected]>
9		* @copyright 2017
10		* @license GNU AGPL version 3 or any later version
11		*
12		* This program is free software: you can redistribute it and/or modify
13		* it under the terms of the GNU Affero General Public License as
14		* published by the Free Software Foundation, either version 3 of the
15		* License, or (at your option) any later version.
16		*
17		* This program is distributed in the hope that it will be useful,
18		* but WITHOUT ANY WARRANTY; without even the implied warranty of
19		* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
20		* GNU Affero General Public License for more details.
21		*
22		* You should have received a copy of the GNU Affero General Public License
23		* along with this program. If not, see <http://www.gnu.org/licenses/>.
24		*
25		*/
26
27		namespace OCA\Circles\Model;
28
29		use OC\L10N\L10N;
30
31		class BaseCircle {
32
33		const CIRCLES_SETTINGS_DEFAULT = [
34		'allow_links' => 'false',
35		'allow_links_auto' => 'false',
36		'allow_links_files' => 'false'
37		];
38
39		const CIRCLES_PERSONAL = 1;
40		const CIRCLES_HIDDEN = 2;
41		const CIRCLES_PRIVATE = 4;
42		const CIRCLES_PUBLIC = 8;
43
44		const CIRCLES_ALL = 15;
45
46		/** @var int */
47		private $id;
48
49		/** @var L10N */
50		protected $l10n;
51
52		/** @var string */
53		private $uniqueId;
54
55		/** @var string */
56		private $name;
57
58		/** @var Member */
59		private $owner;
60
61		/** @var Member */
62		private $user;
63
64		/** @var string */
65		private $description = '';
66
67		/** @var array */
68		private $settings = [];
69
70		/** @var int */
71		private $type;
72
73		/** @var string */
74		private $creation;
75
76		/** @var Member[] */
77		private $members;
78
79		/** @var FederatedLink[] */
80		private $links;
81
82		/** @var FederatedLink[] */
83		private $remote;
84
85		public function __construct($l10n, $type = -1, $name = '') {
86		$this->l10n = $l10n;
87
88		if ($type > -1) {
89		$this->type = $type;
90		}
91		if ($name !== '') {
92		$this->name = $name;
93		}
94		}
95
96
97		public function setId($id) {
98		$this->id = $id;
99
100		return $this;
101		}
102
103		public function getId() {
104		return $this->id;
105		}
106
107
108		/**
109		* @param string $uniqueId
110		*
111		* @return $this
112		*/
113		public function setUniqueId($uniqueId) {
114		$this->uniqueId = (string)$uniqueId;
115
116		return $this;
117		}
118
119		/**
120		* @return string
121		*/
122		public function getUniqueId() {
123		return $this->uniqueId;
124		}
125
126		public function generateUniqueId() {
127		$uniqueId = bin2hex(openssl_random_pseudo_bytes(24));
128		$this->setUniqueId($uniqueId);
129		}
130
131		public function setName($name) {
132		$this->name = $name;
133
134		return $this;
135		}
136
137		public function getName() {
138		return $this->name;
139		}
140
141
142		public function getOwner() {
143		return $this->owner;
144		}
145
146		public function setOwner($owner) {
147		$this->owner = $owner;
148
149		return $this;
150		}
151
152
153		/**
154		* @return Member
155		*/
156		public function getUser() {
157		return $this->user;
158		}
159
160		public function setUser($user) {
161		$this->user = $user;
162
163		return $this;
164		}
165
166
167		public function setDescription($description) {
168		$this->description = $description;
169
170		return $this;
171		}
172
173		public function getDescription() {
174		return $this->description;
175		}
176
177
178		public function setSettings($settings) {
179		if (is_array($settings)) {
180		$this->settings = $settings;
181		} else if (is_string($settings)) {
182		$this->settings = json_decode($settings, true);
		0 ignored issues – show Documentation Bug introduced 2017-06-09 21:52 UTC by Report Bug Copy Issue Report It seems like `json_decode($settings, true)` of type `*` is incompatible with the declared type `array` of property `$settings`. Our type inference engine has found an assignment to a property that is incompatible with the declared type of that property. Either this assignment is in error or the assigned type should be added to the documentation/type hint for that property.. Loading history...
183		}
184
185		return $this;
186		}
187
188		public function getSettings($json = false) {
189
190		if ($json) {
191		return json_encode($this->settings);
192		}
193
194		$settings = $this->settings;
195		if ($settings === null) {
196		$settings = [];
197		}
198
199		$ak = array_keys(self::CIRCLES_SETTINGS_DEFAULT);
200		foreach ($ak AS $k) {
201		if (!key_exists($k, $settings)) {
202		$settings[$k] = self::CIRCLES_SETTINGS_DEFAULT[$k];
203		}
204		}
205
206		return $settings;
207		}
208
209
210		public function setSetting($k, $v) {
211		$this->settings[$k] = $v;
212		}
213
214
215		/**
216		* @param string $k
217		*
218		* @return string\|null
219		*/
220		public function getSetting($k) {
221		if (key_exists($k, $this->settings)) {
222		return $this->settings[$k];
223		}
224		if (key_exists($k, (array)self::CIRCLES_SETTINGS_DEFAULT)) {
225		return self::CIRCLES_SETTINGS_DEFAULT[$k];
226		}
227
228		return null;
229		}
230
231
232		public function setType($type) {
233		$this->type = self::typeInt($type);
234
235		return $this;
236		}
237
238		public function getType() {
239		return $this->type;
240		}
241
242
243		public function setCreation($creation) {
244		$this->creation = $creation;
245
246		return $this;
247		}
248
249		public function getCreation() {
250		return $this->creation;
251		}
252
253
254		public function setMembers($members) {
255		$this->members = $members;
256
257		return $this;
258		}
259
260		public function getMembers() {
261		return $this->members;
262		}
263
264
265		public function setLinks($links) {
266		$this->links = $links;
267
268		return $this;
269		}
270
271		public function getLinks() {
272		return $this->links;
273		}
274
275
276		// public function getRemote() {
		0 ignored issues – show Unused Code Comprehensibility introduced 2017-06-10 21:42 UTC by Report Bug Copy Issue Report `49%` of this comment could be valid code. Did you maybe forget this after debugging? Sometimes obsolete code just ends up commented out instead of removed. In this case it is better to remove the code once you have checked you do not need it. The code might also have been commented out for debugging purposes. In this case it is vital that someone uncomments it again or your project may behave in very unexpected ways in production. This check looks for comments that seem to be mostly valid code and reports them. Loading history...
277		// return $this->remote;
278		// }
279		//
280		// public function addRemote($link) {
281		// array_push($this->remote, $link);
282		// }
283		//
284		// public function getRemoteFromToken($token) {
285		// foreach ($this->links AS $link) {
286		// if ($link->getToken() === $token) {
287		// return $link;
288		// }
289		// }
290		//
291		// return null;
292		// }
293		//
294		// public function getRemoteFromAddressAndId($address, $id) {
295		// foreach ($this->links AS $link) {
296		// if ($link->getAddress() === $address && $link->getUniqueId() === $id) {
297		// return $link;
298		// }
299		// }
300		//
301		// return null;
302		// }
303
304
305		public static function typeInt($type) {
306
307		if (is_numeric($type)) {
308		return (int)$type;
309		}
310
311	View Code Duplication	switch ($type) {
		0 ignored issues – show Duplication introduced 2017-06-07 17:45 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
312		case 'Personal':
313		return self::CIRCLES_PERSONAL;
314		case 'Private':
315		return self::CIRCLES_PRIVATE;
316		case 'Hidden':
317		return self::CIRCLES_HIDDEN;
318		case 'Public':
319		return self::CIRCLES_PUBLIC;
320		}
321
322		return 0;
323		}
324		}

nextcloud / circles

Push — master ( 48e0cc...46633a )

BaseCircle::generateUniqueId() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like