Inspection of "Merge pull request #10211 from nextcloud/bugfix/10..." - nextcloud/server - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( e3126f...1fd7ff )

by Morris

created 2018-07-12 10:23 UTC

Status

Indentation +47 added lines, -47 removed lines patch added patch discarded remove patch

@@ -27,57 +27,57 @@
 block discarded – undo
 
 class Manager implements \OCP\Calendar\Resource\IManager {
 
-	/** @var IBackend[] holds all registered resource backends */
-	private $backends = [];
+    /** @var IBackend[] holds all registered resource backends */
+    private $backends = [];
 
-	/**
-	 * Registers a resource backend
-	 *
-	 * @param IBackend $backend
-	 * @return void
-	 * @since 14.0.0
-	 */
-	public function registerBackend(IBackend $backend) {
-		$this->backends[$backend->getBackendIdentifier()] = $backend;
-	}
+    /**
+     * Registers a resource backend
+     *
+     * @param IBackend $backend
+     * @return void
+     * @since 14.0.0
+     */
+    public function registerBackend(IBackend $backend) {
+        $this->backends[$backend->getBackendIdentifier()] = $backend;
+    }
 
-	/**
-	 * Unregisters a resource backend
-	 *
-	 * @param IBackend $backend
-	 * @return void
-	 * @since 14.0.0
-	 */
-	public function unregisterBackend(IBackend $backend) {
-		unset($this->backends[$backend->getBackendIdentifier()]);
-	}
+    /**
+     * Unregisters a resource backend
+     *
+     * @param IBackend $backend
+     * @return void
+     * @since 14.0.0
+     */
+    public function unregisterBackend(IBackend $backend) {
+        unset($this->backends[$backend->getBackendIdentifier()]);
+    }
 
-	/**
-	 * @return IBackend[]
-	 * @since 14.0.0
-	 */
-	public function getBackends():array {
-		return array_values($this->backends);
-	}
+    /**
+     * @return IBackend[]
+     * @since 14.0.0
+     */
+    public function getBackends():array {
+        return array_values($this->backends);
+    }
 
-	/**
-	 * @param string $backendId
-	 * @return IBackend|null
-	 */
-	public function getBackend($backendId):IBackend {
-		if (!isset($this->backends[$backendId])) {
-			return null;
-		}
+    /**
+     * @param string $backendId
+     * @return IBackend|null
+     */
+    public function getBackend($backendId):IBackend {
+        if (!isset($this->backends[$backendId])) {
+            return null;
+        }
 
-		return $this->backends[$backendId];
-	}
+        return $this->backends[$backendId];
+    }
 
-	/**
-	 * removes all registered backend instances
-	 * @return void
-	 * @since 14.0.0
-	 */
-	public function clear() {
-		$this->backends = [];
-	}
+    /**
+     * removes all registered backend instances
+     * @return void
+     * @since 14.0.0
+     */
+    public function clear() {
+        $this->backends = [];
+    }
 }

Please login to merge, or discard this patch.

lib/private/Calendar/Room/Manager.php 1 patch

Indentation +47 added lines, -47 removed lines patch added patch discarded remove patch

@@ -27,57 +27,57 @@
 block discarded – undo
 
 class Manager implements \OCP\Calendar\Room\IManager {
 
-	/** @var IBackend[] holds all registered resource backends */
-	private $backends = [];
+    /** @var IBackend[] holds all registered resource backends */
+    private $backends = [];
 
-	/**
-	 * Registers a resource backend
-	 *
-	 * @param IBackend $backend
-	 * @return void
-	 * @since 14.0.0
-	 */
-	public function registerBackend(IBackend $backend) {
-		$this->backends[$backend->getBackendIdentifier()] = $backend;
-	}
+    /**
+     * Registers a resource backend
+     *
+     * @param IBackend $backend
+     * @return void
+     * @since 14.0.0
+     */
+    public function registerBackend(IBackend $backend) {
+        $this->backends[$backend->getBackendIdentifier()] = $backend;
+    }
 
-	/**
-	 * Unregisters a resource backend
-	 *
-	 * @param IBackend $backend
-	 * @return void
-	 * @since 14.0.0
-	 */
-	public function unregisterBackend(IBackend $backend) {
-		unset($this->backends[$backend->getBackendIdentifier()]);
-	}
+    /**
+     * Unregisters a resource backend
+     *
+     * @param IBackend $backend
+     * @return void
+     * @since 14.0.0
+     */
+    public function unregisterBackend(IBackend $backend) {
+        unset($this->backends[$backend->getBackendIdentifier()]);
+    }
 
-	/**
-	 * @return IBackend[]
-	 * @since 14.0.0
-	 */
-	public function getBackends():array {
-		return array_values($this->backends);
-	}
+    /**
+     * @return IBackend[]
+     * @since 14.0.0
+     */
+    public function getBackends():array {
+        return array_values($this->backends);
+    }
 
-	/**
-	 * @param string $backendId
-	 * @return IBackend|null
-	 */
-	public function getBackend($backendId):IBackend {
-		if (!isset($this->backends[$backendId])) {
-			return null;
-		}
+    /**
+     * @param string $backendId
+     * @return IBackend|null
+     */
+    public function getBackend($backendId):IBackend {
+        if (!isset($this->backends[$backendId])) {
+            return null;
+        }
 
-		return $this->backends[$backendId];
-	}
+        return $this->backends[$backendId];
+    }
 
-	/**
-	 * removes all registered backend instances
-	 * @return void
-	 * @since 14.0.0
-	 */
-	public function clear() {
-		$this->backends = [];
-	}
+    /**
+     * removes all registered backend instances
+     * @return void
+     * @since 14.0.0
+     */
+    public function clear() {
+        $this->backends = [];
+    }
 }

Please login to merge, or discard this patch.

		@@ -27,57 +27,57 @@
		block discarded – undo
27	27
28	28	class Manager implements \OCP\Calendar\Resource\IManager {
29	29
30		- /** @var IBackend[] holds all registered resource backends */
31		- private $backends = [];
	30	+ /** @var IBackend[] holds all registered resource backends */
	31	+ private $backends = [];
32	32
33		- /**
34		- * Registers a resource backend
35		- *
36		- * @param IBackend $backend
37		- * @return void
38		- * @since 14.0.0
39		- */
40		- public function registerBackend(IBackend $backend) {
41		- $this->backends[$backend->getBackendIdentifier()] = $backend;
42		- }
	33	+ /**
	34	+ * Registers a resource backend
	35	+ *
	36	+ * @param IBackend $backend
	37	+ * @return void
	38	+ * @since 14.0.0
	39	+ */
	40	+ public function registerBackend(IBackend $backend) {
	41	+ $this->backends[$backend->getBackendIdentifier()] = $backend;
	42	+ }
43	43
44		- /**
45		- * Unregisters a resource backend
46		- *
47		- * @param IBackend $backend
48		- * @return void
49		- * @since 14.0.0
50		- */
51		- public function unregisterBackend(IBackend $backend) {
52		- unset($this->backends[$backend->getBackendIdentifier()]);
53		- }
	44	+ /**
	45	+ * Unregisters a resource backend
	46	+ *
	47	+ * @param IBackend $backend
	48	+ * @return void
	49	+ * @since 14.0.0
	50	+ */
	51	+ public function unregisterBackend(IBackend $backend) {
	52	+ unset($this->backends[$backend->getBackendIdentifier()]);
	53	+ }
54	54
55		- /**
56		- * @return IBackend[]
57		- * @since 14.0.0
58		- */
59		- public function getBackends():array {
60		- return array_values($this->backends);
61		- }
	55	+ /**
	56	+ * @return IBackend[]
	57	+ * @since 14.0.0
	58	+ */
	59	+ public function getBackends():array {
	60	+ return array_values($this->backends);
	61	+ }
62	62
63		- /**
64		- * @param string $backendId
65		- * @return IBackend\|null
66		- */
67		- public function getBackend($backendId):IBackend {
68		- if (!isset($this->backends[$backendId])) {
69		- return null;
70		- }
	63	+ /**
	64	+ * @param string $backendId
	65	+ * @return IBackend\|null
	66	+ */
	67	+ public function getBackend($backendId):IBackend {
	68	+ if (!isset($this->backends[$backendId])) {
	69	+ return null;
	70	+ }
71	71
72		- return $this->backends[$backendId];
73		- }
	72	+ return $this->backends[$backendId];
	73	+ }
74	74
75		- /**
76		- * removes all registered backend instances
77		- * @return void
78		- * @since 14.0.0
79		- */
80		- public function clear() {
81		- $this->backends = [];
82		- }
	75	+ /**
	76	+ * removes all registered backend instances
	77	+ * @return void
	78	+ * @since 14.0.0
	79	+ */
	80	+ public function clear() {
	81	+ $this->backends = [];
	82	+ }
83	83	}

		@@ -27,57 +27,57 @@
		block discarded – undo
27	27
28	28	class Manager implements \OCP\Calendar\Room\IManager {
29	29
30		- /** @var IBackend[] holds all registered resource backends */
31		- private $backends = [];
	30	+ /** @var IBackend[] holds all registered resource backends */
	31	+ private $backends = [];
32	32
33		- /**
34		- * Registers a resource backend
35		- *
36		- * @param IBackend $backend
37		- * @return void
38		- * @since 14.0.0
39		- */
40		- public function registerBackend(IBackend $backend) {
41		- $this->backends[$backend->getBackendIdentifier()] = $backend;
42		- }
	33	+ /**
	34	+ * Registers a resource backend
	35	+ *
	36	+ * @param IBackend $backend
	37	+ * @return void
	38	+ * @since 14.0.0
	39	+ */
	40	+ public function registerBackend(IBackend $backend) {
	41	+ $this->backends[$backend->getBackendIdentifier()] = $backend;
	42	+ }
43	43
44		- /**
45		- * Unregisters a resource backend
46		- *
47		- * @param IBackend $backend
48		- * @return void
49		- * @since 14.0.0
50		- */
51		- public function unregisterBackend(IBackend $backend) {
52		- unset($this->backends[$backend->getBackendIdentifier()]);
53		- }
	44	+ /**
	45	+ * Unregisters a resource backend
	46	+ *
	47	+ * @param IBackend $backend
	48	+ * @return void
	49	+ * @since 14.0.0
	50	+ */
	51	+ public function unregisterBackend(IBackend $backend) {
	52	+ unset($this->backends[$backend->getBackendIdentifier()]);
	53	+ }
54	54
55		- /**
56		- * @return IBackend[]
57		- * @since 14.0.0
58		- */
59		- public function getBackends():array {
60		- return array_values($this->backends);
61		- }
	55	+ /**
	56	+ * @return IBackend[]
	57	+ * @since 14.0.0
	58	+ */
	59	+ public function getBackends():array {
	60	+ return array_values($this->backends);
	61	+ }
62	62
63		- /**
64		- * @param string $backendId
65		- * @return IBackend\|null
66		- */
67		- public function getBackend($backendId):IBackend {
68		- if (!isset($this->backends[$backendId])) {
69		- return null;
70		- }
	63	+ /**
	64	+ * @param string $backendId
	65	+ * @return IBackend\|null
	66	+ */
	67	+ public function getBackend($backendId):IBackend {
	68	+ if (!isset($this->backends[$backendId])) {
	69	+ return null;
	70	+ }
71	71
72		- return $this->backends[$backendId];
73		- }
	72	+ return $this->backends[$backendId];
	73	+ }
74	74
75		- /**
76		- * removes all registered backend instances
77		- * @return void
78		- * @since 14.0.0
79		- */
80		- public function clear() {
81		- $this->backends = [];
82		- }
	75	+ /**
	76	+ * removes all registered backend instances
	77	+ * @return void
	78	+ * @since 14.0.0
	79	+ */
	80	+ public function clear() {
	81	+ $this->backends = [];
	82	+ }
83	83	}

nextcloud / server

Push — master ( e3126f...1fd7ff )

Status

Category

Indentation +47 added lines, -47 removed lines patch added patch discarded remove patch

Indentation +47 added lines, -47 removed lines patch added patch discarded remove patch