Passed
Push — master ( ad9515...b9d9d6 )
by Nils
11:37
created
api/Model/ItemModel.php 1 patch
Indentation   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -342,8 +342,8 @@
 block discarded – undo
342 342
         );
343 343
 
344 344
         if (DB::count() > 0 && (
345
-	     (isset($SETTINGS['duplicate_item']) && (int) $SETTINGS['duplicate_item'] === 0)
346
-	     && (int) $itemInfos['personal_folder'] === 0)
345
+            (isset($SETTINGS['duplicate_item']) && (int) $SETTINGS['duplicate_item'] === 0)
346
+         && (int) $itemInfos['personal_folder'] === 0)
347 347
         ) {
348 348
             throw new Exception('Similar item already exists. Duplicates are not allowed.');
349 349
         }
Please login to merge, or discard this patch.
sources/folders.class.php 3 patches
Indentation   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -113,7 +113,7 @@
 block discarded – undo
113 113
     private function isParentFolderAllowed($parent_id, $user_accessible_folders, $user_is_admin, $user_can_create_root_folder)
114 114
     {
115 115
         if ($parent_id == 0 && $user_can_create_root_folder == true)
116
-	    return true;
116
+        return true;
117 117
 
118 118
         if (in_array($parent_id, $user_accessible_folders) === false
119 119
             && (int) $user_is_admin !== 1
Please login to merge, or discard this patch.
Braces   +6 added lines, -4 removed lines patch added patch discarded remove patch
@@ -112,8 +112,9 @@  discard block
 block discarded – undo
112 112
      */
113 113
     private function isParentFolderAllowed($parent_id, $user_accessible_folders, $user_is_admin, $user_can_create_root_folder)
114 114
     {
115
-        if ($parent_id == 0 && $user_can_create_root_folder == true)
116
-	    return true;
115
+        if ($parent_id == 0 && $user_can_create_root_folder == true) {
116
+        	    return true;
117
+        }
117 118
 
118 119
         if (in_array($parent_id, $user_accessible_folders) === false
119 120
             && (int) $user_is_admin !== 1
@@ -438,8 +439,9 @@  discard block
 block discarded – undo
438 439
             )['count'];
439 440
 
440 441
             // Don't insert duplicates
441
-            if ($count > 0)
442
-                continue;
442
+            if ($count > 0) {
443
+                            continue;
444
+            }
443 445
 
444 446
             // Insert new background task
445 447
             DB::insert(
Please login to merge, or discard this patch.
Spacing   +16 added lines, -16 removed lines patch added patch discarded remove patch
@@ -84,7 +84,7 @@  discard block
 block discarded – undo
84 84
 
85 85
         $parentComplexity = $this->checkComplexityLevel($parentFolderData, $complexity, $parent_id);
86 86
         if (isset($parentComplexity ['error']) && $parentComplexity['error'] === true) {
87
-            return $this->errorResponse($this->lang->get('error_folder_complexity_lower_than_top_folder') . " [<b>{$this->settings['TP_PW_COMPLEXITY'][$parentComplexity['valeur']][1]}</b>]");
87
+            return $this->errorResponse($this->lang->get('error_folder_complexity_lower_than_top_folder')." [<b>{$this->settings['TP_PW_COMPLEXITY'][$parentComplexity['valeur']][1]}</b>]");
88 88
         }
89 89
 
90 90
         return $this->createFolder($params, array_merge($parentFolderData, $parentComplexity));
@@ -137,7 +137,7 @@  discard block
 block discarded – undo
137 137
         ) {
138 138
             DB::query(
139 139
                 'SELECT *
140
-                FROM ' . prefixTable('nested_tree') . '
140
+                FROM ' . prefixTable('nested_tree').'
141 141
                 WHERE title = %s AND personal_folder = 0',
142 142
                 $title
143 143
             );
@@ -161,7 +161,7 @@  discard block
 block discarded – undo
161 161
         //check if parent folder is personal
162 162
         $data = DB::queryfirstrow(
163 163
             'SELECT personal_folder, bloquer_creation, bloquer_modification
164
-            FROM ' . prefixTable('nested_tree') . '
164
+            FROM ' . prefixTable('nested_tree').'
165 165
             WHERE id = %i',
166 166
             $parent_id
167 167
         );
@@ -200,7 +200,7 @@  discard block
 block discarded – undo
200 200
             // get complexity level for this folder
201 201
             $data = DB::queryfirstrow(
202 202
                 'SELECT valeur
203
-                FROM ' . prefixTable('misc') . '
203
+                FROM ' . prefixTable('misc').'
204 204
                 WHERE intitule = %i AND type = %s',
205 205
                 $parent_id,
206 206
                 'complex'
@@ -251,11 +251,11 @@  discard block
 block discarded – undo
251 251
      */
252 252
     private function canCreateFolder($isPersonal, $user_is_admin, $user_is_manager, $user_can_manage_all_users, $user_can_create_root_folder)
253 253
     {
254
-        return (int)$isPersonal === 1 ||
255
-            (int)$user_is_admin === 1 ||
256
-            ((int)$user_is_manager === 1 || (int)$user_can_manage_all_users === 1) ||
254
+        return (int) $isPersonal === 1 ||
255
+            (int) $user_is_admin === 1 ||
256
+            ((int) $user_is_manager === 1 || (int) $user_can_manage_all_users === 1) ||
257 257
             ($this->settings['enable_user_can_create_folders'] ?? false) ||
258
-            ((int)$user_can_create_root_folder === 1);
258
+            ((int) $user_can_create_root_folder === 1);
259 259
     }
260 260
 
261 261
     /**
@@ -335,7 +335,7 @@  discard block
 block discarded – undo
335 335
         $path = '';
336 336
         $tree_path = $tree->getPath(0, false);
337 337
         foreach ($tree_path as $fld) {
338
-            $path .= empty($path) ? $fld->title : '/' . $fld->title;
338
+            $path .= empty($path) ? $fld->title : '/'.$fld->title;
339 339
         }
340 340
 
341 341
         $new_json = [
@@ -349,7 +349,7 @@  discard block
 block discarded – undo
349 349
             "is_visible_active" => 0,
350 350
         ];
351 351
 
352
-        $cache_tree = DB::queryFirstRow('SELECT increment_id, folders, visible_folders FROM ' . prefixTable('cache_tree') . ' WHERE user_id = %i', (int)$user_id);
352
+        $cache_tree = DB::queryFirstRow('SELECT increment_id, folders, visible_folders FROM '.prefixTable('cache_tree').' WHERE user_id = %i', (int) $user_id);
353 353
 
354 354
         if (empty($cache_tree)) {
355 355
             DB::insert(prefixTable('cache_tree'), [
@@ -369,7 +369,7 @@  discard block
 block discarded – undo
369 369
                 'folders' => json_encode($folders),
370 370
                 'visible_folders' => json_encode($visible_folders),
371 371
                 'timestamp' => time(),
372
-            ], 'increment_id = %i', (int)$cache_tree['increment_id']);
372
+            ], 'increment_id = %i', (int) $cache_tree['increment_id']);
373 373
         }
374 374
     }
375 375
 
@@ -379,7 +379,7 @@  discard block
 block discarded – undo
379 379
     private function manageFolderPermissions($parent_id, $newId, $user_roles, $access_rights, $user_is_admin)
380 380
     {
381 381
         if ($parent_id !== 0 && $this->settings['subfolder_rights_as_parent'] ?? false) {
382
-            $rows = DB::query('SELECT role_id, type FROM ' . prefixTable('roles_values') . ' WHERE folder_id = %i', $parent_id);
382
+            $rows = DB::query('SELECT role_id, type FROM '.prefixTable('roles_values').' WHERE folder_id = %i', $parent_id);
383 383
             foreach ($rows as $record) {
384 384
                 DB::insert(prefixTable('roles_values'), [
385 385
                     'role_id' => $record['role_id'],
@@ -387,7 +387,7 @@  discard block
 block discarded – undo
387 387
                     'type' => $record['type'],
388 388
                 ]);
389 389
             }
390
-        } elseif ((int)$user_is_admin !== 1) {
390
+        } elseif ((int) $user_is_admin !== 1) {
391 391
             foreach (array_unique(explode(';', $user_roles)) as $role) {
392 392
                 if (!empty($role)) {
393 393
                     DB::insert(prefixTable('roles_values'), [
@@ -405,7 +405,7 @@  discard block
 block discarded – undo
405 405
      */
406 406
     private function copyCustomFieldsCategories($parent_id, $newId)
407 407
     {
408
-        $rows = DB::query('SELECT id_category FROM ' . prefixTable('categories_folders') . ' WHERE id_folder = %i', $parent_id);
408
+        $rows = DB::query('SELECT id_category FROM '.prefixTable('categories_folders').' WHERE id_folder = %i', $parent_id);
409 409
         foreach ($rows as $record) {
410 410
             DB::insert(prefixTable('categories_folders'), [
411 411
                 'id_category' => $record['id_category'],
@@ -430,7 +430,7 @@  discard block
 block discarded – undo
430 430
             // Search for existing job
431 431
             $count = DB::queryFirstRow(
432 432
                 'SELECT COUNT(*) AS count
433
-                FROM ' . prefixTable('background_tasks') . '
433
+                FROM ' . prefixTable('background_tasks').'
434 434
                 WHERE is_in_progress = %i AND process_type = %s AND arguments = %s',
435 435
                 0,
436 436
                 'user_build_cache_tree',
@@ -464,7 +464,7 @@  discard block
 block discarded – undo
464 464
         return [
465 465
             'error' => true,
466 466
             'message' => $message,
467
-            'newId' => '' . $newIdSuffix,
467
+            'newId' => ''.$newIdSuffix,
468 468
         ];
469 469
     }
470 470
 }
Please login to merge, or discard this patch.