Passed
Branch master (a7d334)
by Andreas
21:22
created
src/midcom/dba/privileges.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -64,7 +64,7 @@  discard block
 block discarded – undo
64 64
      */
65 65
     public function set_privilege($privilege, $assignee = null, int $value = MIDCOM_PRIVILEGE_ALLOW, string $classname = '') : bool
66 66
     {
67
-        if (   !$this->can_do('midgard:update')
67
+        if (!$this->can_do('midgard:update')
68 68
             || !$this->can_do('midgard:privileges')) {
69 69
             debug_add("Failed to set a privilege, midgard:update or midgard:privileges on " . static::class . " {$this->guid} not granted for the current user.",
70 70
             MIDCOM_LOG_ERROR);
@@ -94,7 +94,7 @@  discard block
 block discarded – undo
94 94
      */
95 95
     public function unset_privilege($privilege, $assignee = null, string $classname = '') : bool
96 96
     {
97
-        if (   !$this->can_do('midgard:update')
97
+        if (!$this->can_do('midgard:update')
98 98
             || !$this->can_do('midgard:privileges')) {
99 99
             debug_add("Failed to unset a privilege, midgard:update or midgard:privileges on " . static::class . " {$this->guid} not granted for the current user.",
100 100
             MIDCOM_LOG_ERROR);
Please login to merge, or discard this patch.
src/midcom/dba/parameters.php 2 patches
Indentation   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -204,7 +204,7 @@  discard block
 block discarded – undo
204 204
         if (   !$this->can_do('midgard:update')
205 205
             || !$this->can_do('midgard:parameters')) {
206 206
             debug_add("Failed to set parameters, midgard:update or midgard:parameters on " . static::class . " {$this->guid} not granted for the current user.",
207
-                  MIDCOM_LOG_ERROR);
207
+                    MIDCOM_LOG_ERROR);
208 208
             midcom_connection::set_error(MGD_ERR_ACCESS_DENIED);
209 209
             return false;
210 210
         }
@@ -248,7 +248,7 @@  discard block
 block discarded – undo
248 248
         if (   !$this->can_do('midgard:update')
249 249
             || !$this->can_do('midgard:parameters')) {
250 250
             debug_add("Failed to delete parameters, midgard:update or midgard:parameters on " . static::class . " {$this->guid} not granted for the current user.",
251
-                  MIDCOM_LOG_ERROR);
251
+                    MIDCOM_LOG_ERROR);
252 252
             midcom_connection::set_error(MGD_ERR_ACCESS_DENIED);
253 253
             return false;
254 254
         }
Please login to merge, or discard this patch.
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -201,7 +201,7 @@  discard block
 block discarded – undo
201 201
             return false;
202 202
         }
203 203
 
204
-        if (   !$this->can_do('midgard:update')
204
+        if (!$this->can_do('midgard:update')
205 205
             || !$this->can_do('midgard:parameters')) {
206 206
             debug_add("Failed to set parameters, midgard:update or midgard:parameters on " . static::class . " {$this->guid} not granted for the current user.",
207 207
                   MIDCOM_LOG_ERROR);
@@ -245,7 +245,7 @@  discard block
 block discarded – undo
245 245
             return false;
246 246
         }
247 247
 
248
-        if (   !$this->can_do('midgard:update')
248
+        if (!$this->can_do('midgard:update')
249 249
             || !$this->can_do('midgard:parameters')) {
250 250
             debug_add("Failed to delete parameters, midgard:update or midgard:parameters on " . static::class . " {$this->guid} not granted for the current user.",
251 251
                   MIDCOM_LOG_ERROR);
Please login to merge, or discard this patch.
src/midcom/dba/attachments.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -51,7 +51,7 @@  discard block
 block discarded – undo
51 51
             return false;
52 52
         }
53 53
 
54
-        if (   !$this->can_do('midgard:update')
54
+        if (!$this->can_do('midgard:update')
55 55
             || !$this->can_do('midgard:attachments')) {
56 56
             debug_add("Failed to set parameters, midgard:update or midgard:attachments on " . static::class . " {$this->guid} not granted for the current user.",
57 57
             MIDCOM_LOG_ERROR);
@@ -71,7 +71,7 @@  discard block
 block discarded – undo
71 71
             return null;
72 72
         }
73 73
 
74
-        if (   !$this->can_do('midgard:update')
74
+        if (!$this->can_do('midgard:update')
75 75
             || !$this->can_do('midgard:attachments')) {
76 76
             debug_add("Failed to set parameters, midgard:update or midgard:attachments on the " . static::class . " {$this->guid} not granted for the current user.",
77 77
             MIDCOM_LOG_ERROR);
@@ -85,7 +85,7 @@  discard block
 block discarded – undo
85 85
         $attachment->parentguid = $this->guid;
86 86
 
87 87
         if (!$attachment->create()) {
88
-            debug_add("Could not create the attachment '{$name}' for " . static::class . " {$this->guid}: "  . midcom_connection::get_error_string(),
88
+            debug_add("Could not create the attachment '{$name}' for " . static::class . " {$this->guid}: " . midcom_connection::get_error_string(),
89 89
             MIDCOM_LOG_INFO);
90 90
             return null;
91 91
         }
Please login to merge, or discard this patch.