Format control structures, classes, methods and function

To continue this formatting madness, here's a tiny patch that adds
unified formatting for control structures like if and loops as well as
classes, their methods and anonymous functions. This basically forces
the constructs to start on the same line. This is not exactly what PSR2
wants, but I think we can have a few exceptions with "our" style. The
starting of braces on the same line is pracrically standard for our
code.

This also removes and empty lines from method/function bodies at the
beginning and end.

Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>

6 files changed, 2 insertions, 7 deletions

diff --git a/lib/public/WorkflowEngine/GenericEntityEvent.php b/lib/public/WorkflowEngine/GenericEntityEvent.php
index 08786aecad4..2b8cbe04716 100644
--- a/lib/public/WorkflowEngine/GenericEntityEvent.php
+++ b/lib/public/WorkflowEngine/GenericEntityEvent.php
@@ -47,10 +47,10 @@ class GenericEntityEvent implements IEntityEvent {
 	 * @since 18.0.0
 	 */
 	public function __construct(string $displayName, string $eventName) {
-		if(trim($displayName) === '') {
+		if (trim($displayName) === '') {
 			throw new \InvalidArgumentException('DisplayName must not be empty');
 		}
-		if(trim($eventName) === '') {
+		if (trim($eventName) === '') {
 			throw new \InvalidArgumentException('EventName must not be empty');
 		}
 
diff --git a/lib/public/WorkflowEngine/IComplexOperation.php b/lib/public/WorkflowEngine/IComplexOperation.php
index 476175187f7..679a5c1e078 100644
--- a/lib/public/WorkflowEngine/IComplexOperation.php
+++ b/lib/public/WorkflowEngine/IComplexOperation.php
@@ -54,5 +54,4 @@ interface IComplexOperation extends IOperation {
 	 * @since 18.0.0
 	 */
 	public function getTriggerHint(): string;
-
 }
diff --git a/lib/public/WorkflowEngine/IEntity.php b/lib/public/WorkflowEngine/IEntity.php
index 2aec5e76e40..173e696a6e4 100644
--- a/lib/public/WorkflowEngine/IEntity.php
+++ b/lib/public/WorkflowEngine/IEntity.php
@@ -83,5 +83,4 @@ interface IEntity {
 	 * @since 18.0.0
 	 */
 	public function isLegitimatedForUserId(string $userId): bool;
-
 }
diff --git a/lib/public/WorkflowEngine/IEntityCheck.php b/lib/public/WorkflowEngine/IEntityCheck.php
index 88b9352424d..46dab0d5691 100644
--- a/lib/public/WorkflowEngine/IEntityCheck.php
+++ b/lib/public/WorkflowEngine/IEntityCheck.php
@@ -49,5 +49,4 @@ interface IEntityCheck {
 	 * @since 18.0.0
 	 */
 	public function setEntitySubject(IEntity $entity, $subject): void;
-
 }
diff --git a/lib/public/WorkflowEngine/IFileCheck.php b/lib/public/WorkflowEngine/IFileCheck.php
index d11a5011367..f8f4bf2dd0c 100644
--- a/lib/public/WorkflowEngine/IFileCheck.php
+++ b/lib/public/WorkflowEngine/IFileCheck.php
@@ -43,5 +43,4 @@ interface IFileCheck extends IEntityCheck {
 	 * @since 18.0.0
 	 */
 	public function setFileInfo(IStorage $storage, string $path, bool $isDir = false): void;
-
 }
diff --git a/lib/public/WorkflowEngine/IManager.php b/lib/public/WorkflowEngine/IManager.php
index 0eb504242df..b9ce157c222 100644
--- a/lib/public/WorkflowEngine/IManager.php
+++ b/lib/public/WorkflowEngine/IManager.php
@@ -32,7 +32,6 @@ namespace OCP\WorkflowEngine;
  * @since 9.1
  */
 interface IManager {
-
 	const SCOPE_ADMIN = 0;
 	const SCOPE_USER = 1;