Inspection of "Fix copy and paste error on activity descriptions" - nextcloud/server - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#5801)

by Joas

created 2017-07-20 07:32 UTC

Status

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

@@ -27,72 +27,72 @@
 block discarded – undo
 
 class FileDeleted implements ISetting {
 
-	/** @var IL10N */
-	protected $l;
+    /** @var IL10N */
+    protected $l;
 
-	/**
-	 * @param IL10N $l
-	 */
-	public function __construct(IL10N $l) {
-		$this->l = $l;
-	}
+    /**
+     * @param IL10N $l
+     */
+    public function __construct(IL10N $l) {
+        $this->l = $l;
+    }
 
-	/**
-	 * @return string Lowercase a-z and underscore only identifier
-	 * @since 11.0.0
-	 */
-	public function getIdentifier() {
-		return 'file_deleted';
-	}
+    /**
+     * @return string Lowercase a-z and underscore only identifier
+     * @since 11.0.0
+     */
+    public function getIdentifier() {
+        return 'file_deleted';
+    }
 
-	/**
-	 * @return string A translated string
-	 * @since 11.0.0
-	 */
-	public function getName() {
-		return $this->l->t('A file or folder has been <strong>deleted</strong>');
-	}
+    /**
+     * @return string A translated string
+     * @since 11.0.0
+     */
+    public function getName() {
+        return $this->l->t('A file or folder has been <strong>deleted</strong>');
+    }
 
-	/**
-	 * @return int whether the filter should be rather on the top or bottom of
-	 * the admin section. The filters are arranged in ascending order of the
-	 * priority values. It is required to return a value between 0 and 100.
-	 * @since 11.0.0
-	 */
-	public function getPriority() {
-		return 3;
-	}
+    /**
+     * @return int whether the filter should be rather on the top or bottom of
+     * the admin section. The filters are arranged in ascending order of the
+     * priority values. It is required to return a value between 0 and 100.
+     * @since 11.0.0
+     */
+    public function getPriority() {
+        return 3;
+    }
 
-	/**
-	 * @return bool True when the option can be changed for the stream
-	 * @since 11.0.0
-	 */
-	public function canChangeStream() {
-		return true;
-	}
+    /**
+     * @return bool True when the option can be changed for the stream
+     * @since 11.0.0
+     */
+    public function canChangeStream() {
+        return true;
+    }
 
-	/**
-	 * @return bool True when the option can be changed for the stream
-	 * @since 11.0.0
-	 */
-	public function isDefaultEnabledStream() {
-		return true;
-	}
+    /**
+     * @return bool True when the option can be changed for the stream
+     * @since 11.0.0
+     */
+    public function isDefaultEnabledStream() {
+        return true;
+    }
 
-	/**
-	 * @return bool True when the option can be changed for the mail
-	 * @since 11.0.0
-	 */
-	public function canChangeMail() {
-		return true;
-	}
+    /**
+     * @return bool True when the option can be changed for the mail
+     * @since 11.0.0
+     */
+    public function canChangeMail() {
+        return true;
+    }
 
-	/**
-	 * @return bool True when the option can be changed for the stream
-	 * @since 11.0.0
-	 */
-	public function isDefaultEnabledMail() {
-		return false;
-	}
+    /**
+     * @return bool True when the option can be changed for the stream
+     * @since 11.0.0
+     */
+    public function isDefaultEnabledMail() {
+        return false;
+    }
 }
 

Please login to merge, or discard this patch.

apps/files/lib/Activity/Settings/FileRestored.php 1 patch

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

@@ -27,72 +27,72 @@
 block discarded – undo
 
 class FileRestored implements ISetting {
 
-	/** @var IL10N */
-	protected $l;
+    /** @var IL10N */
+    protected $l;
 
-	/**
-	 * @param IL10N $l
-	 */
-	public function __construct(IL10N $l) {
-		$this->l = $l;
-	}
+    /**
+     * @param IL10N $l
+     */
+    public function __construct(IL10N $l) {
+        $this->l = $l;
+    }
 
-	/**
-	 * @return string Lowercase a-z and underscore only identifier
-	 * @since 11.0.0
-	 */
-	public function getIdentifier() {
-		return 'file_restored';
-	}
+    /**
+     * @return string Lowercase a-z and underscore only identifier
+     * @since 11.0.0
+     */
+    public function getIdentifier() {
+        return 'file_restored';
+    }
 
-	/**
-	 * @return string A translated string
-	 * @since 11.0.0
-	 */
-	public function getName() {
-		return $this->l->t('A file or folder has been <strong>restored</strong>');
-	}
+    /**
+     * @return string A translated string
+     * @since 11.0.0
+     */
+    public function getName() {
+        return $this->l->t('A file or folder has been <strong>restored</strong>');
+    }
 
-	/**
-	 * @return int whether the filter should be rather on the top or bottom of
-	 * the admin section. The filters are arranged in ascending order of the
-	 * priority values. It is required to return a value between 0 and 100.
-	 * @since 11.0.0
-	 */
-	public function getPriority() {
-		return 4;
-	}
+    /**
+     * @return int whether the filter should be rather on the top or bottom of
+     * the admin section. The filters are arranged in ascending order of the
+     * priority values. It is required to return a value between 0 and 100.
+     * @since 11.0.0
+     */
+    public function getPriority() {
+        return 4;
+    }
 
-	/**
-	 * @return bool True when the option can be changed for the stream
-	 * @since 11.0.0
-	 */
-	public function canChangeStream() {
-		return true;
-	}
+    /**
+     * @return bool True when the option can be changed for the stream
+     * @since 11.0.0
+     */
+    public function canChangeStream() {
+        return true;
+    }
 
-	/**
-	 * @return bool True when the option can be changed for the stream
-	 * @since 11.0.0
-	 */
-	public function isDefaultEnabledStream() {
-		return true;
-	}
+    /**
+     * @return bool True when the option can be changed for the stream
+     * @since 11.0.0
+     */
+    public function isDefaultEnabledStream() {
+        return true;
+    }
 
-	/**
-	 * @return bool True when the option can be changed for the mail
-	 * @since 11.0.0
-	 */
-	public function canChangeMail() {
-		return true;
-	}
+    /**
+     * @return bool True when the option can be changed for the mail
+     * @since 11.0.0
+     */
+    public function canChangeMail() {
+        return true;
+    }
 
-	/**
-	 * @return bool True when the option can be changed for the stream
-	 * @since 11.0.0
-	 */
-	public function isDefaultEnabledMail() {
-		return false;
-	}
+    /**
+     * @return bool True when the option can be changed for the stream
+     * @since 11.0.0
+     */
+    public function isDefaultEnabledMail() {
+        return false;
+    }
 }
 

Please login to merge, or discard this patch.

		@@ -27,72 +27,72 @@
		block discarded – undo
27	27
28	28	class FileDeleted implements ISetting {
29	29
30		- /** @var IL10N */
31		- protected $l;
	30	+ /** @var IL10N */
	31	+ protected $l;
32	32
33		- /**
34		- * @param IL10N $l
35		- */
36		- public function __construct(IL10N $l) {
37		- $this->l = $l;
38		- }
	33	+ /**
	34	+ * @param IL10N $l
	35	+ */
	36	+ public function __construct(IL10N $l) {
	37	+ $this->l = $l;
	38	+ }
39	39
40		- /**
41		- * @return string Lowercase a-z and underscore only identifier
42		- * @since 11.0.0
43		- */
44		- public function getIdentifier() {
45		- return 'file_deleted';
46		- }
	40	+ /**
	41	+ * @return string Lowercase a-z and underscore only identifier
	42	+ * @since 11.0.0
	43	+ */
	44	+ public function getIdentifier() {
	45	+ return 'file_deleted';
	46	+ }
47	47
48		- /**
49		- * @return string A translated string
50		- * @since 11.0.0
51		- */
52		- public function getName() {
53		- return $this->l->t('A file or folder has been <strong>deleted</strong>');
54		- }
	48	+ /**
	49	+ * @return string A translated string
	50	+ * @since 11.0.0
	51	+ */
	52	+ public function getName() {
	53	+ return $this->l->t('A file or folder has been <strong>deleted</strong>');
	54	+ }
55	55
56		- /**
57		- * @return int whether the filter should be rather on the top or bottom of
58		- * the admin section. The filters are arranged in ascending order of the
59		- * priority values. It is required to return a value between 0 and 100.
60		- * @since 11.0.0
61		- */
62		- public function getPriority() {
63		- return 3;
64		- }
	56	+ /**
	57	+ * @return int whether the filter should be rather on the top or bottom of
	58	+ * the admin section. The filters are arranged in ascending order of the
	59	+ * priority values. It is required to return a value between 0 and 100.
	60	+ * @since 11.0.0
	61	+ */
	62	+ public function getPriority() {
	63	+ return 3;
	64	+ }
65	65
66		- /**
67		- * @return bool True when the option can be changed for the stream
68		- * @since 11.0.0
69		- */
70		- public function canChangeStream() {
71		- return true;
72		- }
	66	+ /**
	67	+ * @return bool True when the option can be changed for the stream
	68	+ * @since 11.0.0
	69	+ */
	70	+ public function canChangeStream() {
	71	+ return true;
	72	+ }
73	73
74		- /**
75		- * @return bool True when the option can be changed for the stream
76		- * @since 11.0.0
77		- */
78		- public function isDefaultEnabledStream() {
79		- return true;
80		- }
	74	+ /**
	75	+ * @return bool True when the option can be changed for the stream
	76	+ * @since 11.0.0
	77	+ */
	78	+ public function isDefaultEnabledStream() {
	79	+ return true;
	80	+ }
81	81
82		- /**
83		- * @return bool True when the option can be changed for the mail
84		- * @since 11.0.0
85		- */
86		- public function canChangeMail() {
87		- return true;
88		- }
	82	+ /**
	83	+ * @return bool True when the option can be changed for the mail
	84	+ * @since 11.0.0
	85	+ */
	86	+ public function canChangeMail() {
	87	+ return true;
	88	+ }
89	89
90		- /**
91		- * @return bool True when the option can be changed for the stream
92		- * @since 11.0.0
93		- */
94		- public function isDefaultEnabledMail() {
95		- return false;
96		- }
	90	+ /**
	91	+ * @return bool True when the option can be changed for the stream
	92	+ * @since 11.0.0
	93	+ */
	94	+ public function isDefaultEnabledMail() {
	95	+ return false;
	96	+ }
97	97	}
98	98

		@@ -27,72 +27,72 @@
		block discarded – undo
27	27
28	28	class FileRestored implements ISetting {
29	29
30		- /** @var IL10N */
31		- protected $l;
	30	+ /** @var IL10N */
	31	+ protected $l;
32	32
33		- /**
34		- * @param IL10N $l
35		- */
36		- public function __construct(IL10N $l) {
37		- $this->l = $l;
38		- }
	33	+ /**
	34	+ * @param IL10N $l
	35	+ */
	36	+ public function __construct(IL10N $l) {
	37	+ $this->l = $l;
	38	+ }
39	39
40		- /**
41		- * @return string Lowercase a-z and underscore only identifier
42		- * @since 11.0.0
43		- */
44		- public function getIdentifier() {
45		- return 'file_restored';
46		- }
	40	+ /**
	41	+ * @return string Lowercase a-z and underscore only identifier
	42	+ * @since 11.0.0
	43	+ */
	44	+ public function getIdentifier() {
	45	+ return 'file_restored';
	46	+ }
47	47
48		- /**
49		- * @return string A translated string
50		- * @since 11.0.0
51		- */
52		- public function getName() {
53		- return $this->l->t('A file or folder has been <strong>restored</strong>');
54		- }
	48	+ /**
	49	+ * @return string A translated string
	50	+ * @since 11.0.0
	51	+ */
	52	+ public function getName() {
	53	+ return $this->l->t('A file or folder has been <strong>restored</strong>');
	54	+ }
55	55
56		- /**
57		- * @return int whether the filter should be rather on the top or bottom of
58		- * the admin section. The filters are arranged in ascending order of the
59		- * priority values. It is required to return a value between 0 and 100.
60		- * @since 11.0.0
61		- */
62		- public function getPriority() {
63		- return 4;
64		- }
	56	+ /**
	57	+ * @return int whether the filter should be rather on the top or bottom of
	58	+ * the admin section. The filters are arranged in ascending order of the
	59	+ * priority values. It is required to return a value between 0 and 100.
	60	+ * @since 11.0.0
	61	+ */
	62	+ public function getPriority() {
	63	+ return 4;
	64	+ }
65	65
66		- /**
67		- * @return bool True when the option can be changed for the stream
68		- * @since 11.0.0
69		- */
70		- public function canChangeStream() {
71		- return true;
72		- }
	66	+ /**
	67	+ * @return bool True when the option can be changed for the stream
	68	+ * @since 11.0.0
	69	+ */
	70	+ public function canChangeStream() {
	71	+ return true;
	72	+ }
73	73
74		- /**
75		- * @return bool True when the option can be changed for the stream
76		- * @since 11.0.0
77		- */
78		- public function isDefaultEnabledStream() {
79		- return true;
80		- }
	74	+ /**
	75	+ * @return bool True when the option can be changed for the stream
	76	+ * @since 11.0.0
	77	+ */
	78	+ public function isDefaultEnabledStream() {
	79	+ return true;
	80	+ }
81	81
82		- /**
83		- * @return bool True when the option can be changed for the mail
84		- * @since 11.0.0
85		- */
86		- public function canChangeMail() {
87		- return true;
88		- }
	82	+ /**
	83	+ * @return bool True when the option can be changed for the mail
	84	+ * @since 11.0.0
	85	+ */
	86	+ public function canChangeMail() {
	87	+ return true;
	88	+ }
89	89
90		- /**
91		- * @return bool True when the option can be changed for the stream
92		- * @since 11.0.0
93		- */
94		- public function isDefaultEnabledMail() {
95		- return false;
96		- }
	90	+ /**
	91	+ * @return bool True when the option can be changed for the stream
	92	+ * @since 11.0.0
	93	+ */
	94	+ public function isDefaultEnabledMail() {
	95	+ return false;
	96	+ }
97	97	}
98	98

nextcloud / server

Pull Request — master (#5801)

Status

Category

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

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