Inspection of "Merge pull request #16586 from nextcloud/feature/1..." - nextcloud/server - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 2e2d1b...edf9de )

by Roeland

created 2019-08-01 10:35 UTC

Status

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

@@ -34,288 +34,288 @@
 block discarded – undo
  * @package OCA\AdminAudit\Actions
  */
 class Sharing extends Action {
-	/**
-	 * Logs sharing of data
-	 *
-	 * @param array $params
-	 */
-	public function shared(array $params) {
-		if($params['shareType'] === Share::SHARE_TYPE_LINK) {
-			$this->log(
-				'The %s "%s" with ID "%s" has been shared via link with permissions "%s" (Share ID: %s)',
-				$params,
-				[
-					'itemType',
-					'itemTarget',
-					'itemSource',
-					'permissions',
-					'id',
-				]
-			);
-		} elseif($params['shareType'] === Share::SHARE_TYPE_USER) {
-			$this->log(
-				'The %s "%s" with ID "%s" has been shared to the user "%s" with permissions "%s"  (Share ID: %s)',
-				$params,
-				[
-					'itemType',
-					'itemTarget',
-					'itemSource',
-					'shareWith',
-					'permissions',
-					'id',
-				]
-			);
-		} elseif($params['shareType'] === Share::SHARE_TYPE_GROUP) {
-			$this->log(
-				'The %s "%s" with ID "%s" has been shared to the group "%s" with permissions "%s"  (Share ID: %s)',
-				$params,
-				[
-					'itemType',
-					'itemTarget',
-					'itemSource',
-					'shareWith',
-					'permissions',
-					'id',
-				]
-			);
-		} elseif($params['shareType'] === Share::SHARE_TYPE_ROOM) {
-			$this->log(
-				'The %s "%s" with ID "%s" has been shared to the room "%s" with permissions "%s" (Share ID: %s)',
-				$params,
-				[
-					'itemType',
-					'itemTarget',
-					'itemSource',
-					'shareWith',
-					'permissions',
-					'id',
-				]
-			);
-		} elseif($params['shareType'] === Share::SHARE_TYPE_EMAIL) {
-			$this->log(
-				'The %s "%s" with ID "%s" has been shared to the email recipient "%s" with permissions "%s" (Share ID: %s)',
-				$params,
-				[
-					'itemType',
-					'itemTarget',
-					'itemSource',
-					'shareWith',
-					'permissions',
-					'id',
-				]
-			);
-		} elseif($params['shareType'] === Share::SHARE_TYPE_CIRCLE) {
-			$this->log(
-				'The %s "%s" with ID "%s" has been shared to the circle "%s" with permissions "%s" (Share ID: %s)',
-				$params,
-				[
-					'itemType',
-					'itemTarget',
-					'itemSource',
-					'shareWith',
-					'permissions',
-					'id',
-				]
-			);
-		} elseif($params['shareType'] === Share::SHARE_TYPE_REMOTE) {
-			$this->log(
-				'The %s "%s" with ID "%s" has been shared to the remote user "%s" with permissions "%s" (Share ID: %s)',
-				$params,
-				[
-					'itemType',
-					'itemTarget',
-					'itemSource',
-					'shareWith',
-					'permissions',
-					'id',
-				]
-			);
-		} elseif($params['shareType'] === Share::SHARE_TYPE_REMOTE_GROUP) {
-			$this->log(
-				'The %s "%s" with ID "%s" has been shared to the remote group "%s" with permissions "%s" (Share ID: %s)',
-				$params,
-				[
-					'itemType',
-					'itemTarget',
-					'itemSource',
-					'shareWith',
-					'permissions',
-					'id',
-				]
-			);
-		}
-	}
+    /**
+     * Logs sharing of data
+     *
+     * @param array $params
+     */
+    public function shared(array $params) {
+        if($params['shareType'] === Share::SHARE_TYPE_LINK) {
+            $this->log(
+                'The %s "%s" with ID "%s" has been shared via link with permissions "%s" (Share ID: %s)',
+                $params,
+                [
+                    'itemType',
+                    'itemTarget',
+                    'itemSource',
+                    'permissions',
+                    'id',
+                ]
+            );
+        } elseif($params['shareType'] === Share::SHARE_TYPE_USER) {
+            $this->log(
+                'The %s "%s" with ID "%s" has been shared to the user "%s" with permissions "%s"  (Share ID: %s)',
+                $params,
+                [
+                    'itemType',
+                    'itemTarget',
+                    'itemSource',
+                    'shareWith',
+                    'permissions',
+                    'id',
+                ]
+            );
+        } elseif($params['shareType'] === Share::SHARE_TYPE_GROUP) {
+            $this->log(
+                'The %s "%s" with ID "%s" has been shared to the group "%s" with permissions "%s"  (Share ID: %s)',
+                $params,
+                [
+                    'itemType',
+                    'itemTarget',
+                    'itemSource',
+                    'shareWith',
+                    'permissions',
+                    'id',
+                ]
+            );
+        } elseif($params['shareType'] === Share::SHARE_TYPE_ROOM) {
+            $this->log(
+                'The %s "%s" with ID "%s" has been shared to the room "%s" with permissions "%s" (Share ID: %s)',
+                $params,
+                [
+                    'itemType',
+                    'itemTarget',
+                    'itemSource',
+                    'shareWith',
+                    'permissions',
+                    'id',
+                ]
+            );
+        } elseif($params['shareType'] === Share::SHARE_TYPE_EMAIL) {
+            $this->log(
+                'The %s "%s" with ID "%s" has been shared to the email recipient "%s" with permissions "%s" (Share ID: %s)',
+                $params,
+                [
+                    'itemType',
+                    'itemTarget',
+                    'itemSource',
+                    'shareWith',
+                    'permissions',
+                    'id',
+                ]
+            );
+        } elseif($params['shareType'] === Share::SHARE_TYPE_CIRCLE) {
+            $this->log(
+                'The %s "%s" with ID "%s" has been shared to the circle "%s" with permissions "%s" (Share ID: %s)',
+                $params,
+                [
+                    'itemType',
+                    'itemTarget',
+                    'itemSource',
+                    'shareWith',
+                    'permissions',
+                    'id',
+                ]
+            );
+        } elseif($params['shareType'] === Share::SHARE_TYPE_REMOTE) {
+            $this->log(
+                'The %s "%s" with ID "%s" has been shared to the remote user "%s" with permissions "%s" (Share ID: %s)',
+                $params,
+                [
+                    'itemType',
+                    'itemTarget',
+                    'itemSource',
+                    'shareWith',
+                    'permissions',
+                    'id',
+                ]
+            );
+        } elseif($params['shareType'] === Share::SHARE_TYPE_REMOTE_GROUP) {
+            $this->log(
+                'The %s "%s" with ID "%s" has been shared to the remote group "%s" with permissions "%s" (Share ID: %s)',
+                $params,
+                [
+                    'itemType',
+                    'itemTarget',
+                    'itemSource',
+                    'shareWith',
+                    'permissions',
+                    'id',
+                ]
+            );
+        }
+    }
 
-	/**
-	 * Logs unsharing of data
-	 *
-	 * @param array $params
-	 */
-	public function unshare(array $params) {
-		if($params['shareType'] === Share::SHARE_TYPE_LINK) {
-			$this->log(
-				'The %s "%s" with ID "%s" has been unshared (Share ID: %s)',
-				$params,
-				[
-					'itemType',
-					'fileTarget',
-					'itemSource',
-					'id',
-				]
-			);
-		} elseif($params['shareType'] === Share::SHARE_TYPE_USER) {
-			$this->log(
-				'The %s "%s" with ID "%s" has been unshared from the user "%s" (Share ID: %s)',
-				$params,
-				[
-					'itemType',
-					'fileTarget',
-					'itemSource',
-					'shareWith',
-					'id',
-				]
-			);
-		} elseif($params['shareType'] === Share::SHARE_TYPE_GROUP) {
-			$this->log(
-				'The %s "%s" with ID "%s" has been unshared from the group "%s" (Share ID: %s)',
-				$params,
-				[
-					'itemType',
-					'fileTarget',
-					'itemSource',
-					'shareWith',
-					'id',
-				]
-			);
-		} elseif($params['shareType'] === Share::SHARE_TYPE_ROOM) {
-			$this->log(
-				'The %s "%s" with ID "%s" has been unshared from the room "%s" (Share ID: %s)',
-				$params,
-				[
-					'itemType',
-					'fileTarget',
-					'itemSource',
-					'shareWith',
-					'id',
-				]
-			);
-		} elseif($params['shareType'] === Share::SHARE_TYPE_EMAIL) {
-			$this->log(
-				'The %s "%s" with ID "%s" has been unshared from the email recipient "%s" (Share ID: %s)',
-				$params,
-				[
-					'itemType',
-					'fileTarget',
-					'itemSource',
-					'shareWith',
-					'id',
-				]
-			);
-		} elseif($params['shareType'] === Share::SHARE_TYPE_CIRCLE) {
-			$this->log(
-				'The %s "%s" with ID "%s" has been unshared from the circle "%s" (Share ID: %s)',
-				$params,
-				[
-					'itemType',
-					'fileTarget',
-					'itemSource',
-					'shareWith',
-					'id',
-				]
-			);
-		} elseif($params['shareType'] === Share::SHARE_TYPE_REMOTE) {
-			$this->log(
-				'The %s "%s" with ID "%s" has been unshared from the remote user "%s" (Share ID: %s)',
-				$params,
-				[
-					'itemType',
-					'fileTarget',
-					'itemSource',
-					'shareWith',
-					'id',
-				]
-			);
-		} elseif($params['shareType'] === Share::SHARE_TYPE_REMOTE_GROUP) {
-			$this->log(
-				'The %s "%s" with ID "%s" has been unshared from the remote group "%s" (Share ID: %s)',
-				$params,
-				[
-					'itemType',
-					'fileTarget',
-					'itemSource',
-					'shareWith',
-					'id',
-				]
-			);
-		}
-	}
+    /**
+     * Logs unsharing of data
+     *
+     * @param array $params
+     */
+    public function unshare(array $params) {
+        if($params['shareType'] === Share::SHARE_TYPE_LINK) {
+            $this->log(
+                'The %s "%s" with ID "%s" has been unshared (Share ID: %s)',
+                $params,
+                [
+                    'itemType',
+                    'fileTarget',
+                    'itemSource',
+                    'id',
+                ]
+            );
+        } elseif($params['shareType'] === Share::SHARE_TYPE_USER) {
+            $this->log(
+                'The %s "%s" with ID "%s" has been unshared from the user "%s" (Share ID: %s)',
+                $params,
+                [
+                    'itemType',
+                    'fileTarget',
+                    'itemSource',
+                    'shareWith',
+                    'id',
+                ]
+            );
+        } elseif($params['shareType'] === Share::SHARE_TYPE_GROUP) {
+            $this->log(
+                'The %s "%s" with ID "%s" has been unshared from the group "%s" (Share ID: %s)',
+                $params,
+                [
+                    'itemType',
+                    'fileTarget',
+                    'itemSource',
+                    'shareWith',
+                    'id',
+                ]
+            );
+        } elseif($params['shareType'] === Share::SHARE_TYPE_ROOM) {
+            $this->log(
+                'The %s "%s" with ID "%s" has been unshared from the room "%s" (Share ID: %s)',
+                $params,
+                [
+                    'itemType',
+                    'fileTarget',
+                    'itemSource',
+                    'shareWith',
+                    'id',
+                ]
+            );
+        } elseif($params['shareType'] === Share::SHARE_TYPE_EMAIL) {
+            $this->log(
+                'The %s "%s" with ID "%s" has been unshared from the email recipient "%s" (Share ID: %s)',
+                $params,
+                [
+                    'itemType',
+                    'fileTarget',
+                    'itemSource',
+                    'shareWith',
+                    'id',
+                ]
+            );
+        } elseif($params['shareType'] === Share::SHARE_TYPE_CIRCLE) {
+            $this->log(
+                'The %s "%s" with ID "%s" has been unshared from the circle "%s" (Share ID: %s)',
+                $params,
+                [
+                    'itemType',
+                    'fileTarget',
+                    'itemSource',
+                    'shareWith',
+                    'id',
+                ]
+            );
+        } elseif($params['shareType'] === Share::SHARE_TYPE_REMOTE) {
+            $this->log(
+                'The %s "%s" with ID "%s" has been unshared from the remote user "%s" (Share ID: %s)',
+                $params,
+                [
+                    'itemType',
+                    'fileTarget',
+                    'itemSource',
+                    'shareWith',
+                    'id',
+                ]
+            );
+        } elseif($params['shareType'] === Share::SHARE_TYPE_REMOTE_GROUP) {
+            $this->log(
+                'The %s "%s" with ID "%s" has been unshared from the remote group "%s" (Share ID: %s)',
+                $params,
+                [
+                    'itemType',
+                    'fileTarget',
+                    'itemSource',
+                    'shareWith',
+                    'id',
+                ]
+            );
+        }
+    }
 
-	/**
-	 * Logs the updating of permission changes for shares
-	 *
-	 * @param array $params
-	 */
-	public function updatePermissions(array $params) {
-		$this->log(
-			'The permissions of the shared %s "%s" with ID "%s" have been changed to "%s"',
-			$params,
-			[
-				'itemType',
-				'path',
-				'itemSource',
-				'permissions',
-			]
-		);
-	}
+    /**
+     * Logs the updating of permission changes for shares
+     *
+     * @param array $params
+     */
+    public function updatePermissions(array $params) {
+        $this->log(
+            'The permissions of the shared %s "%s" with ID "%s" have been changed to "%s"',
+            $params,
+            [
+                'itemType',
+                'path',
+                'itemSource',
+                'permissions',
+            ]
+        );
+    }
 
-	/**
-	 * Logs the password changes for a share
-	 *
-	 * @param array $params
-	 */
-	public function updatePassword(array $params) {
-		$this->log(
-			'The password of the publicly shared %s "%s" with ID "%s" has been changed',
-			$params,
-			[
-				'itemType',
-				'token',
-				'itemSource',
-			]
-		);
-	}
+    /**
+     * Logs the password changes for a share
+     *
+     * @param array $params
+     */
+    public function updatePassword(array $params) {
+        $this->log(
+            'The password of the publicly shared %s "%s" with ID "%s" has been changed',
+            $params,
+            [
+                'itemType',
+                'token',
+                'itemSource',
+            ]
+        );
+    }
 
-	/**
-	 * Logs the expiration date changes for a share
-	 *
-	 * @param array $params
-	 */
-	public function updateExpirationDate(array $params) {
-		$this->log(
-			'The expiration date of the publicly shared %s with ID "%s" has been changed to "%s"',
-			$params,
-			[
-				'itemType',
-				'itemSource',
-				'date',
-			]
-		);
-	}
+    /**
+     * Logs the expiration date changes for a share
+     *
+     * @param array $params
+     */
+    public function updateExpirationDate(array $params) {
+        $this->log(
+            'The expiration date of the publicly shared %s with ID "%s" has been changed to "%s"',
+            $params,
+            [
+                'itemType',
+                'itemSource',
+                'date',
+            ]
+        );
+    }
 
-	/**
-	 * Logs access of shared files
-	 *
-	 * @param array $params
-	 */
-	public function shareAccessed(array $params) {
-		$this->log(
-			'The shared %s with the token "%s" by "%s" has been accessed.',
-			$params,
-			[
-				'itemType',
-				'token',
-				'uidOwner',
-			]
-		);
-	}
+    /**
+     * Logs access of shared files
+     *
+     * @param array $params
+     */
+    public function shareAccessed(array $params) {
+        $this->log(
+            'The shared %s with the token "%s" by "%s" has been accessed.',
+            $params,
+            [
+                'itemType',
+                'token',
+                'uidOwner',
+            ]
+        );
+    }
 }

Please login to merge, or discard this patch.

		@@ -34,288 +34,288 @@
		block discarded – undo
34	34	* @package OCA\AdminAudit\Actions
35	35	*/
36	36	class Sharing extends Action {
37		- /**
38		- * Logs sharing of data
39		- *
40		- * @param array $params
41		- */
42		- public function shared(array $params) {
43		- if($params['shareType'] === Share::SHARE_TYPE_LINK) {
44		- $this->log(
45		- 'The %s "%s" with ID "%s" has been shared via link with permissions "%s" (Share ID: %s)',
46		- $params,
47		- [
48		- 'itemType',
49		- 'itemTarget',
50		- 'itemSource',
51		- 'permissions',
52		- 'id',
53		- ]
54		- );
55		- } elseif($params['shareType'] === Share::SHARE_TYPE_USER) {
56		- $this->log(
57		- 'The %s "%s" with ID "%s" has been shared to the user "%s" with permissions "%s" (Share ID: %s)',
58		- $params,
59		- [
60		- 'itemType',
61		- 'itemTarget',
62		- 'itemSource',
63		- 'shareWith',
64		- 'permissions',
65		- 'id',
66		- ]
67		- );
68		- } elseif($params['shareType'] === Share::SHARE_TYPE_GROUP) {
69		- $this->log(
70		- 'The %s "%s" with ID "%s" has been shared to the group "%s" with permissions "%s" (Share ID: %s)',
71		- $params,
72		- [
73		- 'itemType',
74		- 'itemTarget',
75		- 'itemSource',
76		- 'shareWith',
77		- 'permissions',
78		- 'id',
79		- ]
80		- );
81		- } elseif($params['shareType'] === Share::SHARE_TYPE_ROOM) {
82		- $this->log(
83		- 'The %s "%s" with ID "%s" has been shared to the room "%s" with permissions "%s" (Share ID: %s)',
84		- $params,
85		- [
86		- 'itemType',
87		- 'itemTarget',
88		- 'itemSource',
89		- 'shareWith',
90		- 'permissions',
91		- 'id',
92		- ]
93		- );
94		- } elseif($params['shareType'] === Share::SHARE_TYPE_EMAIL) {
95		- $this->log(
96		- 'The %s "%s" with ID "%s" has been shared to the email recipient "%s" with permissions "%s" (Share ID: %s)',
97		- $params,
98		- [
99		- 'itemType',
100		- 'itemTarget',
101		- 'itemSource',
102		- 'shareWith',
103		- 'permissions',
104		- 'id',
105		- ]
106		- );
107		- } elseif($params['shareType'] === Share::SHARE_TYPE_CIRCLE) {
108		- $this->log(
109		- 'The %s "%s" with ID "%s" has been shared to the circle "%s" with permissions "%s" (Share ID: %s)',
110		- $params,
111		- [
112		- 'itemType',
113		- 'itemTarget',
114		- 'itemSource',
115		- 'shareWith',
116		- 'permissions',
117		- 'id',
118		- ]
119		- );
120		- } elseif($params['shareType'] === Share::SHARE_TYPE_REMOTE) {
121		- $this->log(
122		- 'The %s "%s" with ID "%s" has been shared to the remote user "%s" with permissions "%s" (Share ID: %s)',
123		- $params,
124		- [
125		- 'itemType',
126		- 'itemTarget',
127		- 'itemSource',
128		- 'shareWith',
129		- 'permissions',
130		- 'id',
131		- ]
132		- );
133		- } elseif($params['shareType'] === Share::SHARE_TYPE_REMOTE_GROUP) {
134		- $this->log(
135		- 'The %s "%s" with ID "%s" has been shared to the remote group "%s" with permissions "%s" (Share ID: %s)',
136		- $params,
137		- [
138		- 'itemType',
139		- 'itemTarget',
140		- 'itemSource',
141		- 'shareWith',
142		- 'permissions',
143		- 'id',
144		- ]
145		- );
146		- }
147		- }
	37	+ /**
	38	+ * Logs sharing of data
	39	+ *
	40	+ * @param array $params
	41	+ */
	42	+ public function shared(array $params) {
	43	+ if($params['shareType'] === Share::SHARE_TYPE_LINK) {
	44	+ $this->log(
	45	+ 'The %s "%s" with ID "%s" has been shared via link with permissions "%s" (Share ID: %s)',
	46	+ $params,
	47	+ [
	48	+ 'itemType',
	49	+ 'itemTarget',
	50	+ 'itemSource',
	51	+ 'permissions',
	52	+ 'id',
	53	+ ]
	54	+ );
	55	+ } elseif($params['shareType'] === Share::SHARE_TYPE_USER) {
	56	+ $this->log(
	57	+ 'The %s "%s" with ID "%s" has been shared to the user "%s" with permissions "%s" (Share ID: %s)',
	58	+ $params,
	59	+ [
	60	+ 'itemType',
	61	+ 'itemTarget',
	62	+ 'itemSource',
	63	+ 'shareWith',
	64	+ 'permissions',
	65	+ 'id',
	66	+ ]
	67	+ );
	68	+ } elseif($params['shareType'] === Share::SHARE_TYPE_GROUP) {
	69	+ $this->log(
	70	+ 'The %s "%s" with ID "%s" has been shared to the group "%s" with permissions "%s" (Share ID: %s)',
	71	+ $params,
	72	+ [
	73	+ 'itemType',
	74	+ 'itemTarget',
	75	+ 'itemSource',
	76	+ 'shareWith',
	77	+ 'permissions',
	78	+ 'id',
	79	+ ]
	80	+ );
	81	+ } elseif($params['shareType'] === Share::SHARE_TYPE_ROOM) {
	82	+ $this->log(
	83	+ 'The %s "%s" with ID "%s" has been shared to the room "%s" with permissions "%s" (Share ID: %s)',
	84	+ $params,
	85	+ [
	86	+ 'itemType',
	87	+ 'itemTarget',
	88	+ 'itemSource',
	89	+ 'shareWith',
	90	+ 'permissions',
	91	+ 'id',
	92	+ ]
	93	+ );
	94	+ } elseif($params['shareType'] === Share::SHARE_TYPE_EMAIL) {
	95	+ $this->log(
	96	+ 'The %s "%s" with ID "%s" has been shared to the email recipient "%s" with permissions "%s" (Share ID: %s)',
	97	+ $params,
	98	+ [
	99	+ 'itemType',
	100	+ 'itemTarget',
	101	+ 'itemSource',
	102	+ 'shareWith',
	103	+ 'permissions',
	104	+ 'id',
	105	+ ]
	106	+ );
	107	+ } elseif($params['shareType'] === Share::SHARE_TYPE_CIRCLE) {
	108	+ $this->log(
	109	+ 'The %s "%s" with ID "%s" has been shared to the circle "%s" with permissions "%s" (Share ID: %s)',
	110	+ $params,
	111	+ [
	112	+ 'itemType',
	113	+ 'itemTarget',
	114	+ 'itemSource',
	115	+ 'shareWith',
	116	+ 'permissions',
	117	+ 'id',
	118	+ ]
	119	+ );
	120	+ } elseif($params['shareType'] === Share::SHARE_TYPE_REMOTE) {
	121	+ $this->log(
	122	+ 'The %s "%s" with ID "%s" has been shared to the remote user "%s" with permissions "%s" (Share ID: %s)',
	123	+ $params,
	124	+ [
	125	+ 'itemType',
	126	+ 'itemTarget',
	127	+ 'itemSource',
	128	+ 'shareWith',
	129	+ 'permissions',
	130	+ 'id',
	131	+ ]
	132	+ );
	133	+ } elseif($params['shareType'] === Share::SHARE_TYPE_REMOTE_GROUP) {
	134	+ $this->log(
	135	+ 'The %s "%s" with ID "%s" has been shared to the remote group "%s" with permissions "%s" (Share ID: %s)',
	136	+ $params,
	137	+ [
	138	+ 'itemType',
	139	+ 'itemTarget',
	140	+ 'itemSource',
	141	+ 'shareWith',
	142	+ 'permissions',
	143	+ 'id',
	144	+ ]
	145	+ );
	146	+ }
	147	+ }
148	148
149		- /**
150		- * Logs unsharing of data
151		- *
152		- * @param array $params
153		- */
154		- public function unshare(array $params) {
155		- if($params['shareType'] === Share::SHARE_TYPE_LINK) {
156		- $this->log(
157		- 'The %s "%s" with ID "%s" has been unshared (Share ID: %s)',
158		- $params,
159		- [
160		- 'itemType',
161		- 'fileTarget',
162		- 'itemSource',
163		- 'id',
164		- ]
165		- );
166		- } elseif($params['shareType'] === Share::SHARE_TYPE_USER) {
167		- $this->log(
168		- 'The %s "%s" with ID "%s" has been unshared from the user "%s" (Share ID: %s)',
169		- $params,
170		- [
171		- 'itemType',
172		- 'fileTarget',
173		- 'itemSource',
174		- 'shareWith',
175		- 'id',
176		- ]
177		- );
178		- } elseif($params['shareType'] === Share::SHARE_TYPE_GROUP) {
179		- $this->log(
180		- 'The %s "%s" with ID "%s" has been unshared from the group "%s" (Share ID: %s)',
181		- $params,
182		- [
183		- 'itemType',
184		- 'fileTarget',
185		- 'itemSource',
186		- 'shareWith',
187		- 'id',
188		- ]
189		- );
190		- } elseif($params['shareType'] === Share::SHARE_TYPE_ROOM) {
191		- $this->log(
192		- 'The %s "%s" with ID "%s" has been unshared from the room "%s" (Share ID: %s)',
193		- $params,
194		- [
195		- 'itemType',
196		- 'fileTarget',
197		- 'itemSource',
198		- 'shareWith',
199		- 'id',
200		- ]
201		- );
202		- } elseif($params['shareType'] === Share::SHARE_TYPE_EMAIL) {
203		- $this->log(
204		- 'The %s "%s" with ID "%s" has been unshared from the email recipient "%s" (Share ID: %s)',
205		- $params,
206		- [
207		- 'itemType',
208		- 'fileTarget',
209		- 'itemSource',
210		- 'shareWith',
211		- 'id',
212		- ]
213		- );
214		- } elseif($params['shareType'] === Share::SHARE_TYPE_CIRCLE) {
215		- $this->log(
216		- 'The %s "%s" with ID "%s" has been unshared from the circle "%s" (Share ID: %s)',
217		- $params,
218		- [
219		- 'itemType',
220		- 'fileTarget',
221		- 'itemSource',
222		- 'shareWith',
223		- 'id',
224		- ]
225		- );
226		- } elseif($params['shareType'] === Share::SHARE_TYPE_REMOTE) {
227		- $this->log(
228		- 'The %s "%s" with ID "%s" has been unshared from the remote user "%s" (Share ID: %s)',
229		- $params,
230		- [
231		- 'itemType',
232		- 'fileTarget',
233		- 'itemSource',
234		- 'shareWith',
235		- 'id',
236		- ]
237		- );
238		- } elseif($params['shareType'] === Share::SHARE_TYPE_REMOTE_GROUP) {
239		- $this->log(
240		- 'The %s "%s" with ID "%s" has been unshared from the remote group "%s" (Share ID: %s)',
241		- $params,
242		- [
243		- 'itemType',
244		- 'fileTarget',
245		- 'itemSource',
246		- 'shareWith',
247		- 'id',
248		- ]
249		- );
250		- }
251		- }
	149	+ /**
	150	+ * Logs unsharing of data
	151	+ *
	152	+ * @param array $params
	153	+ */
	154	+ public function unshare(array $params) {
	155	+ if($params['shareType'] === Share::SHARE_TYPE_LINK) {
	156	+ $this->log(
	157	+ 'The %s "%s" with ID "%s" has been unshared (Share ID: %s)',
	158	+ $params,
	159	+ [
	160	+ 'itemType',
	161	+ 'fileTarget',
	162	+ 'itemSource',
	163	+ 'id',
	164	+ ]
	165	+ );
	166	+ } elseif($params['shareType'] === Share::SHARE_TYPE_USER) {
	167	+ $this->log(
	168	+ 'The %s "%s" with ID "%s" has been unshared from the user "%s" (Share ID: %s)',
	169	+ $params,
	170	+ [
	171	+ 'itemType',
	172	+ 'fileTarget',
	173	+ 'itemSource',
	174	+ 'shareWith',
	175	+ 'id',
	176	+ ]
	177	+ );
	178	+ } elseif($params['shareType'] === Share::SHARE_TYPE_GROUP) {
	179	+ $this->log(
	180	+ 'The %s "%s" with ID "%s" has been unshared from the group "%s" (Share ID: %s)',
	181	+ $params,
	182	+ [
	183	+ 'itemType',
	184	+ 'fileTarget',
	185	+ 'itemSource',
	186	+ 'shareWith',
	187	+ 'id',
	188	+ ]
	189	+ );
	190	+ } elseif($params['shareType'] === Share::SHARE_TYPE_ROOM) {
	191	+ $this->log(
	192	+ 'The %s "%s" with ID "%s" has been unshared from the room "%s" (Share ID: %s)',
	193	+ $params,
	194	+ [
	195	+ 'itemType',
	196	+ 'fileTarget',
	197	+ 'itemSource',
	198	+ 'shareWith',
	199	+ 'id',
	200	+ ]
	201	+ );
	202	+ } elseif($params['shareType'] === Share::SHARE_TYPE_EMAIL) {
	203	+ $this->log(
	204	+ 'The %s "%s" with ID "%s" has been unshared from the email recipient "%s" (Share ID: %s)',
	205	+ $params,
	206	+ [
	207	+ 'itemType',
	208	+ 'fileTarget',
	209	+ 'itemSource',
	210	+ 'shareWith',
	211	+ 'id',
	212	+ ]
	213	+ );
	214	+ } elseif($params['shareType'] === Share::SHARE_TYPE_CIRCLE) {
	215	+ $this->log(
	216	+ 'The %s "%s" with ID "%s" has been unshared from the circle "%s" (Share ID: %s)',
	217	+ $params,
	218	+ [
	219	+ 'itemType',
	220	+ 'fileTarget',
	221	+ 'itemSource',
	222	+ 'shareWith',
	223	+ 'id',
	224	+ ]
	225	+ );
	226	+ } elseif($params['shareType'] === Share::SHARE_TYPE_REMOTE) {
	227	+ $this->log(
	228	+ 'The %s "%s" with ID "%s" has been unshared from the remote user "%s" (Share ID: %s)',
	229	+ $params,
	230	+ [
	231	+ 'itemType',
	232	+ 'fileTarget',
	233	+ 'itemSource',
	234	+ 'shareWith',
	235	+ 'id',
	236	+ ]
	237	+ );
	238	+ } elseif($params['shareType'] === Share::SHARE_TYPE_REMOTE_GROUP) {
	239	+ $this->log(
	240	+ 'The %s "%s" with ID "%s" has been unshared from the remote group "%s" (Share ID: %s)',
	241	+ $params,
	242	+ [
	243	+ 'itemType',
	244	+ 'fileTarget',
	245	+ 'itemSource',
	246	+ 'shareWith',
	247	+ 'id',
	248	+ ]
	249	+ );
	250	+ }
	251	+ }
252	252
253		- /**
254		- * Logs the updating of permission changes for shares
255		- *
256		- * @param array $params
257		- */
258		- public function updatePermissions(array $params) {
259		- $this->log(
260		- 'The permissions of the shared %s "%s" with ID "%s" have been changed to "%s"',
261		- $params,
262		- [
263		- 'itemType',
264		- 'path',
265		- 'itemSource',
266		- 'permissions',
267		- ]
268		- );
269		- }
	253	+ /**
	254	+ * Logs the updating of permission changes for shares
	255	+ *
	256	+ * @param array $params
	257	+ */
	258	+ public function updatePermissions(array $params) {
	259	+ $this->log(
	260	+ 'The permissions of the shared %s "%s" with ID "%s" have been changed to "%s"',
	261	+ $params,
	262	+ [
	263	+ 'itemType',
	264	+ 'path',
	265	+ 'itemSource',
	266	+ 'permissions',
	267	+ ]
	268	+ );
	269	+ }
270	270
271		- /**
272		- * Logs the password changes for a share
273		- *
274		- * @param array $params
275		- */
276		- public function updatePassword(array $params) {
277		- $this->log(
278		- 'The password of the publicly shared %s "%s" with ID "%s" has been changed',
279		- $params,
280		- [
281		- 'itemType',
282		- 'token',
283		- 'itemSource',
284		- ]
285		- );
286		- }
	271	+ /**
	272	+ * Logs the password changes for a share
	273	+ *
	274	+ * @param array $params
	275	+ */
	276	+ public function updatePassword(array $params) {
	277	+ $this->log(
	278	+ 'The password of the publicly shared %s "%s" with ID "%s" has been changed',
	279	+ $params,
	280	+ [
	281	+ 'itemType',
	282	+ 'token',
	283	+ 'itemSource',
	284	+ ]
	285	+ );
	286	+ }
287	287
288		- /**
289		- * Logs the expiration date changes for a share
290		- *
291		- * @param array $params
292		- */
293		- public function updateExpirationDate(array $params) {
294		- $this->log(
295		- 'The expiration date of the publicly shared %s with ID "%s" has been changed to "%s"',
296		- $params,
297		- [
298		- 'itemType',
299		- 'itemSource',
300		- 'date',
301		- ]
302		- );
303		- }
	288	+ /**
	289	+ * Logs the expiration date changes for a share
	290	+ *
	291	+ * @param array $params
	292	+ */
	293	+ public function updateExpirationDate(array $params) {
	294	+ $this->log(
	295	+ 'The expiration date of the publicly shared %s with ID "%s" has been changed to "%s"',
	296	+ $params,
	297	+ [
	298	+ 'itemType',
	299	+ 'itemSource',
	300	+ 'date',
	301	+ ]
	302	+ );
	303	+ }
304	304
305		- /**
306		- * Logs access of shared files
307		- *
308		- * @param array $params
309		- */
310		- public function shareAccessed(array $params) {
311		- $this->log(
312		- 'The shared %s with the token "%s" by "%s" has been accessed.',
313		- $params,
314		- [
315		- 'itemType',
316		- 'token',
317		- 'uidOwner',
318		- ]
319		- );
320		- }
	305	+ /**
	306	+ * Logs access of shared files
	307	+ *
	308	+ * @param array $params
	309	+ */
	310	+ public function shareAccessed(array $params) {
	311	+ $this->log(
	312	+ 'The shared %s with the token "%s" by "%s" has been accessed.',
	313	+ $params,
	314	+ [
	315	+ 'itemType',
	316	+ 'token',
	317	+ 'uidOwner',
	318	+ ]
	319	+ );
	320	+ }
321	321	}

nextcloud / server

Push — master ( 2e2d1b...edf9de )

Status

Category

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