Inspection of "Merge pull request #35618 from nextcloud/fix/migra..." - nextcloud/server - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 182171...7c9c0d )

by Julius

created 2022-12-06 12:24 UTC

Status

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

@@ -34,47 +34,47 @@
 block discarded – undo
  * @since 13.0.0
  */
 interface IMigrationStep {
-	/**
-	 * Human-readable name of the migration step
-	 *
-	 * @return string
-	 * @since 14.0.0
-	 */
-	public function name(): string;
+    /**
+     * Human-readable name of the migration step
+     *
+     * @return string
+     * @since 14.0.0
+     */
+    public function name(): string;
 
-	/**
-	 * Human-readable description of the migration step
-	 *
-	 * @return string
-	 * @since 14.0.0
-	 */
-	public function description(): string;
+    /**
+     * Human-readable description of the migration step
+     *
+     * @return string
+     * @since 14.0.0
+     */
+    public function description(): string;
 
-	/**
-	 * @param IOutput $output
-	 * @param Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
-	 * @psalm-param Closure():ISchemaWrapper $schemaClosure
-	 * @param array $options
-	 * @since 13.0.0
-	 */
-	public function preSchemaChange(IOutput $output, Closure $schemaClosure, array $options);
+    /**
+     * @param IOutput $output
+     * @param Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
+     * @psalm-param Closure():ISchemaWrapper $schemaClosure
+     * @param array $options
+     * @since 13.0.0
+     */
+    public function preSchemaChange(IOutput $output, Closure $schemaClosure, array $options);
 
-	/**
-	 * @param IOutput $output
-	 * @param Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
-	 * @psalm-param Closure():ISchemaWrapper $schemaClosure
-	 * @param array $options
-	 * @return null|ISchemaWrapper
-	 * @since 13.0.0
-	 */
-	public function changeSchema(IOutput $output, Closure $schemaClosure, array $options);
+    /**
+     * @param IOutput $output
+     * @param Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
+     * @psalm-param Closure():ISchemaWrapper $schemaClosure
+     * @param array $options
+     * @return null|ISchemaWrapper
+     * @since 13.0.0
+     */
+    public function changeSchema(IOutput $output, Closure $schemaClosure, array $options);
 
-	/**
-	 * @param IOutput $output
-	 * @param Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
-	 * @psalm-param Closure():ISchemaWrapper $schemaClosure
-	 * @param array $options
-	 * @since 13.0.0
-	 */
-	public function postSchemaChange(IOutput $output, Closure $schemaClosure, array $options);
+    /**
+     * @param IOutput $output
+     * @param Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
+     * @psalm-param Closure():ISchemaWrapper $schemaClosure
+     * @param array $options
+     * @since 13.0.0
+     */
+    public function postSchemaChange(IOutput $output, Closure $schemaClosure, array $options);
 }

Please login to merge, or discard this patch.

lib/public/Migration/SimpleMigrationStep.php 1 patch

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

@@ -35,55 +35,55 @@
 block discarded – undo
  * @since 13.0.0
  */
 abstract class SimpleMigrationStep implements IMigrationStep {
-	/**
-	 * Human-readable name of the migration step
-	 *
-	 * @return string
-	 * @since 14.0.0
-	 */
-	public function name(): string {
-		return '';
-	}
+    /**
+     * Human-readable name of the migration step
+     *
+     * @return string
+     * @since 14.0.0
+     */
+    public function name(): string {
+        return '';
+    }
 
-	/**
-	 * Human-readable description of the migration step
-	 *
-	 * @return string
-	 * @since 14.0.0
-	 */
-	public function description(): string {
-		return '';
-	}
+    /**
+     * Human-readable description of the migration step
+     *
+     * @return string
+     * @since 14.0.0
+     */
+    public function description(): string {
+        return '';
+    }
 
-	/**
-	 * @param IOutput $output
-	 * @param Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
-	 * @psalm-param Closure():ISchemaWrapper $schemaClosure
-	 * @param array $options
-	 * @since 13.0.0
-	 */
-	public function preSchemaChange(IOutput $output, \Closure $schemaClosure, array $options) {
-	}
+    /**
+     * @param IOutput $output
+     * @param Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
+     * @psalm-param Closure():ISchemaWrapper $schemaClosure
+     * @param array $options
+     * @since 13.0.0
+     */
+    public function preSchemaChange(IOutput $output, \Closure $schemaClosure, array $options) {
+    }
 
-	/**
-	 * @param IOutput $output
-	 * @param Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
-	 * @psalm-param Closure():ISchemaWrapper $schemaClosure
-	 * @param array $options
-	 * @return null|ISchemaWrapper
-	 * @since 13.0.0
-	 */
-	public function changeSchema(IOutput $output, \Closure $schemaClosure, array $options) {
-		return null;
-	}
+    /**
+     * @param IOutput $output
+     * @param Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
+     * @psalm-param Closure():ISchemaWrapper $schemaClosure
+     * @param array $options
+     * @return null|ISchemaWrapper
+     * @since 13.0.0
+     */
+    public function changeSchema(IOutput $output, \Closure $schemaClosure, array $options) {
+        return null;
+    }
 
-	/**
-	 * @param IOutput $output
-	 * @param Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
-	 * @psalm-param Closure():ISchemaWrapper $schemaClosure
-	 * @param array $options
-	 * @since 13.0.0
-	 */
-	public function postSchemaChange(IOutput $output, \Closure $schemaClosure, array $options) {
-	}
+    /**
+     * @param IOutput $output
+     * @param Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
+     * @psalm-param Closure():ISchemaWrapper $schemaClosure
+     * @param array $options
+     * @since 13.0.0
+     */
+    public function postSchemaChange(IOutput $output, \Closure $schemaClosure, array $options) {
+    }
 }

Please login to merge, or discard this patch.

		@@ -34,47 +34,47 @@
		block discarded – undo
34	34	* @since 13.0.0
35	35	*/
36	36	interface IMigrationStep {
37		- /**
38		- * Human-readable name of the migration step
39		- *
40		- * @return string
41		- * @since 14.0.0
42		- */
43		- public function name(): string;
	37	+ /**
	38	+ * Human-readable name of the migration step
	39	+ *
	40	+ * @return string
	41	+ * @since 14.0.0
	42	+ */
	43	+ public function name(): string;
44	44
45		- /**
46		- * Human-readable description of the migration step
47		- *
48		- * @return string
49		- * @since 14.0.0
50		- */
51		- public function description(): string;
	45	+ /**
	46	+ * Human-readable description of the migration step
	47	+ *
	48	+ * @return string
	49	+ * @since 14.0.0
	50	+ */
	51	+ public function description(): string;
52	52
53		- /**
54		- * @param IOutput $output
55		- * @param Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
56		- * @psalm-param Closure():ISchemaWrapper $schemaClosure
57		- * @param array $options
58		- * @since 13.0.0
59		- */
60		- public function preSchemaChange(IOutput $output, Closure $schemaClosure, array $options);
	53	+ /**
	54	+ * @param IOutput $output
	55	+ * @param Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
	56	+ * @psalm-param Closure():ISchemaWrapper $schemaClosure
	57	+ * @param array $options
	58	+ * @since 13.0.0
	59	+ */
	60	+ public function preSchemaChange(IOutput $output, Closure $schemaClosure, array $options);
61	61
62		- /**
63		- * @param IOutput $output
64		- * @param Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
65		- * @psalm-param Closure():ISchemaWrapper $schemaClosure
66		- * @param array $options
67		- * @return null\|ISchemaWrapper
68		- * @since 13.0.0
69		- */
70		- public function changeSchema(IOutput $output, Closure $schemaClosure, array $options);
	62	+ /**
	63	+ * @param IOutput $output
	64	+ * @param Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
	65	+ * @psalm-param Closure():ISchemaWrapper $schemaClosure
	66	+ * @param array $options
	67	+ * @return null\|ISchemaWrapper
	68	+ * @since 13.0.0
	69	+ */
	70	+ public function changeSchema(IOutput $output, Closure $schemaClosure, array $options);
71	71
72		- /**
73		- * @param IOutput $output
74		- * @param Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
75		- * @psalm-param Closure():ISchemaWrapper $schemaClosure
76		- * @param array $options
77		- * @since 13.0.0
78		- */
79		- public function postSchemaChange(IOutput $output, Closure $schemaClosure, array $options);
	72	+ /**
	73	+ * @param IOutput $output
	74	+ * @param Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
	75	+ * @psalm-param Closure():ISchemaWrapper $schemaClosure
	76	+ * @param array $options
	77	+ * @since 13.0.0
	78	+ */
	79	+ public function postSchemaChange(IOutput $output, Closure $schemaClosure, array $options);
80	80	}

		@@ -35,55 +35,55 @@
		block discarded – undo
35	35	* @since 13.0.0
36	36	*/
37	37	abstract class SimpleMigrationStep implements IMigrationStep {
38		- /**
39		- * Human-readable name of the migration step
40		- *
41		- * @return string
42		- * @since 14.0.0
43		- */
44		- public function name(): string {
45		- return '';
46		- }
	38	+ /**
	39	+ * Human-readable name of the migration step
	40	+ *
	41	+ * @return string
	42	+ * @since 14.0.0
	43	+ */
	44	+ public function name(): string {
	45	+ return '';
	46	+ }
47	47
48		- /**
49		- * Human-readable description of the migration step
50		- *
51		- * @return string
52		- * @since 14.0.0
53		- */
54		- public function description(): string {
55		- return '';
56		- }
	48	+ /**
	49	+ * Human-readable description of the migration step
	50	+ *
	51	+ * @return string
	52	+ * @since 14.0.0
	53	+ */
	54	+ public function description(): string {
	55	+ return '';
	56	+ }
57	57
58		- /**
59		- * @param IOutput $output
60		- * @param Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
61		- * @psalm-param Closure():ISchemaWrapper $schemaClosure
62		- * @param array $options
63		- * @since 13.0.0
64		- */
65		- public function preSchemaChange(IOutput $output, \Closure $schemaClosure, array $options) {
66		- }
	58	+ /**
	59	+ * @param IOutput $output
	60	+ * @param Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
	61	+ * @psalm-param Closure():ISchemaWrapper $schemaClosure
	62	+ * @param array $options
	63	+ * @since 13.0.0
	64	+ */
	65	+ public function preSchemaChange(IOutput $output, \Closure $schemaClosure, array $options) {
	66	+ }
67	67
68		- /**
69		- * @param IOutput $output
70		- * @param Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
71		- * @psalm-param Closure():ISchemaWrapper $schemaClosure
72		- * @param array $options
73		- * @return null\|ISchemaWrapper
74		- * @since 13.0.0
75		- */
76		- public function changeSchema(IOutput $output, \Closure $schemaClosure, array $options) {
77		- return null;
78		- }
	68	+ /**
	69	+ * @param IOutput $output
	70	+ * @param Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
	71	+ * @psalm-param Closure():ISchemaWrapper $schemaClosure
	72	+ * @param array $options
	73	+ * @return null\|ISchemaWrapper
	74	+ * @since 13.0.0
	75	+ */
	76	+ public function changeSchema(IOutput $output, \Closure $schemaClosure, array $options) {
	77	+ return null;
	78	+ }
79	79
80		- /**
81		- * @param IOutput $output
82		- * @param Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
83		- * @psalm-param Closure():ISchemaWrapper $schemaClosure
84		- * @param array $options
85		- * @since 13.0.0
86		- */
87		- public function postSchemaChange(IOutput $output, \Closure $schemaClosure, array $options) {
88		- }
	80	+ /**
	81	+ * @param IOutput $output
	82	+ * @param Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
	83	+ * @psalm-param Closure():ISchemaWrapper $schemaClosure
	84	+ * @param array $options
	85	+ * @since 13.0.0
	86	+ */
	87	+ public function postSchemaChange(IOutput $output, \Closure $schemaClosure, array $options) {
	88	+ }
89	89	}

nextcloud / server

Push — master ( 182171...7c9c0d )

Status

Category

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

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