nilsteampassnet / TeamPass

api/Controller/Api/ItemController.php

Spacing +25 added lines, -25 removed lines

@@ -40,11 +40,11 @@ 
      */
     private function getUserPrivateKey(array $userData): ?string
     {
-        include_once API_ROOT_PATH . '/inc/jwt_utils.php';
+        include_once API_ROOT_PATH.'/inc/jwt_utils.php';
 
         // Verify session_key exists in JWT payload
         if (!isset($userData['session_key']) || empty($userData['session_key'])) {
-            error_log('getUserPrivateKey: Missing session_key in JWT token for user ID ' . $userData['id']);
+            error_log('getUserPrivateKey: Missing session_key in JWT token for user ID '.$userData['id']);
             return null;
         }
 
@@ -88,7 +88,7 @@ 
             // SQL where clause with folders list
             if (isset($arrQueryStringParams['folders']) === true) {
                 // convert the folders to an array
-                $arrQueryStringParams['folders'] = explode(',', str_replace( array('[',']') , ''  , $arrQueryStringParams['folders']));
+                $arrQueryStringParams['folders'] = explode(',', str_replace(array('[', ']'), '', $arrQueryStringParams['folders']));
 
                 // ensure to only use the intersection
                 $foldersList = implode(',', array_intersect($arrQueryStringParams['folders'], $userData['folders_list']));
@@ -243,7 +243,7 @@ 
                             (string) $arrQueryStringParams['description'] ?? '',
                             (string) $arrQueryStringParams['login'],
                             (string) $arrQueryStringParams['email'] ?? '',
-                            (string) $arrQueryStringParams['url'] ?? '' ,
+                            (string) $arrQueryStringParams['url'] ?? '',
                             (string) $arrQueryStringParams['tags'] ?? '',
                             (int) $arrQueryStringParams['anyone_can_modify'] ?? 0,
                             (string) $arrQueryStringParams['icon'] ?? '',
@@ -257,7 +257,7 @@ 
                 } else {
                     // Gérer le cas où les paramètres ne sont pas un tableau
                     $strErrorDesc = 'Data not consistent';
-                    $strErrorHeader = 'Expected array, received ' . gettype($arrQueryStringParams);
+                    $strErrorHeader = 'Expected array, received '.gettype($arrQueryStringParams);
                 }
             }
         } else {
@@ -309,11 +309,11 @@ 
             // SQL where clause with item id
             if (isset($arrQueryStringParams['id']) === true) {
                 // build sql where clause by ID
-                $sqlExtra = ' WHERE i.id = '.$arrQueryStringParams['id'] . $sql_constraint;
+                $sqlExtra = ' WHERE i.id = '.$arrQueryStringParams['id'].$sql_constraint;
             } else if (isset($arrQueryStringParams['label']) === true) {
                 // build sql where clause by LABEL
-                $sqlExtra = ' WHERE i.label '.(isset($arrQueryStringParams['like']) === true && (int) $arrQueryStringParams['like'] === 1 ? ' LIKE "%'.$arrQueryStringParams['label'].'%"' : ' = '.$arrQueryStringParams['label']) . $sql_constraint;
-                $sqlLimit = isset($arrQueryStringParams['limit']) === true && (int) $arrQueryStringParams['limit'] > 0 ? $arrQueryStringParams['limit'] : 50;   // let's limit to 50 by default
+                $sqlExtra = ' WHERE i.label '.(isset($arrQueryStringParams['like']) === true && (int) $arrQueryStringParams['like'] === 1 ? ' LIKE "%'.$arrQueryStringParams['label'].'%"' : ' = '.$arrQueryStringParams['label']).$sql_constraint;
+                $sqlLimit = isset($arrQueryStringParams['limit']) === true && (int) $arrQueryStringParams['limit'] > 0 ? $arrQueryStringParams['limit'] : 50; // let's limit to 50 by default
             } else if (isset($arrQueryStringParams['description']) === true) {
                 // build sql where clause by DESCRIPTION
                 $sqlExtra = ' WHERE i.description '.(isset($arrQueryStringParams['like']) === true && (int) $arrQueryStringParams['like'] === 1 ? ' LIKE '.$arrQueryStringParams['description'] : ' = '.$arrQueryStringParams['description']).$sql_constraint;
@@ -396,9 +396,9 @@ 
             }*/
 
             // Build SQL constraint for accessible folders
-            $sql_constraint = ' AND (i.id_tree IN (' . $userData['folders_list'] . ')';
+            $sql_constraint = ' AND (i.id_tree IN ('.$userData['folders_list'].')';
             if (!empty($userData['restricted_items_list'])) {
-                $sql_constraint .= ' OR i.id IN (' . $userData['restricted_items_list'] . ')';
+                $sql_constraint .= ' OR i.id IN ('.$userData['restricted_items_list'].')';
             }
             $sql_constraint .= ')';
 
@@ -416,9 +416,9 @@ 
                     $rows = DB::query(
                         "SELECT i.id, i.label, i.login, i.url, i.id_tree, 
                                 CASE WHEN o.enabled = 1 THEN 1 ELSE 0 END AS has_otp
-                        FROM " . prefixTable('items') . " AS i
-                        LEFT JOIN " . prefixTable('items_otp') . " AS o ON (o.item_id = i.id)
-                        WHERE i.url LIKE %s" . $sql_constraint . "
+                        FROM " . prefixTable('items')." AS i
+                        LEFT JOIN " . prefixTable('items_otp')." AS o ON (o.item_id = i.id)
+                        WHERE i.url LIKE %s" . $sql_constraint."
                             AND i.deleted_at IS NULL
                         ORDER BY i.label ASC",
                         "%".$searchUrl."%"
@@ -429,7 +429,7 @@ 
                         // Get user's sharekey for this item
                         $shareKey = DB::queryfirstrow(
                             'SELECT share_key
-                            FROM ' . prefixTable('sharekeys_items') . '
+                            FROM ' . prefixTable('sharekeys_items').'
                             WHERE user_id = %i AND object_id = %i',
                             $userData['id'],
                             $row['id']
@@ -462,7 +462,7 @@ 
                     }
                 }
             } catch (Error $e) {
-                $strErrorDesc = $e->getMessage() . '. Something went wrong! Please contact support.';
+                $strErrorDesc = $e->getMessage().'. Something went wrong! Please contact support.';
                 $strErrorHeader = 'HTTP/1.1 500 Internal Server Error';
             }
         } else {
@@ -522,7 +522,7 @@ 
 
                 // Load item basic info to check folder access
                 $itemInfo = DB::queryFirstRow(
-                    'SELECT id_tree FROM ' . prefixTable('items') . ' WHERE id = %i',
+                    'SELECT id_tree FROM '.prefixTable('items').' WHERE id = %i',
                     $itemId
                 );
 
@@ -546,7 +546,7 @@ 
                     } else {
                         // Load OTP data
                         $otpData = DB::queryFirstRow(
-                            'SELECT secret, enabled FROM ' . prefixTable('items_otp') . ' WHERE item_id = %i',
+                            'SELECT secret, enabled FROM '.prefixTable('items_otp').' WHERE item_id = %i',
                             $itemId
                         );
 
@@ -577,7 +577,7 @@ 
                                         'item_id' => $itemId
                                     ]);
                                 } catch (\RuntimeException $e) {
-                                    $strErrorDesc = 'Failed to generate OTP code: ' . $e->getMessage();
+                                    $strErrorDesc = 'Failed to generate OTP code: '.$e->getMessage();
                                     $strErrorHeader = 'HTTP/1.1 500 Internal Server Error';
                                 }
                             } else {
@@ -588,7 +588,7 @@ 
                     }
                 }
             } catch (\Error $e) {
-                $strErrorDesc = $e->getMessage() . '. Something went wrong! Please contact support.';
+                $strErrorDesc = $e->getMessage().'. Something went wrong! Please contact support.';
                 $strErrorHeader = 'HTTP/1.1 500 Internal Server Error';
             }
         } else {
@@ -653,7 +653,7 @@ 
                         try {
                             // Load item info to check access rights
                             $itemInfo = DB::queryFirstRow(
-                                'SELECT id, id_tree, label FROM ' . prefixTable('items') . ' WHERE id = %i',
+                                'SELECT id, id_tree, label FROM '.prefixTable('items').' WHERE id = %i',
                                 $itemId
                             );
 
@@ -714,13 +714,13 @@ 
                                 }
                             }
                         } catch (Error $e) {
-                            $strErrorDesc = $e->getMessage() . '. Something went wrong! Please contact support.';
+                            $strErrorDesc = $e->getMessage().'. Something went wrong! Please contact support.';
                             $strErrorHeader = 'HTTP/1.1 500 Internal Server Error';
                         }
                     }
                 } else {
                     $strErrorDesc = 'Data not consistent';
-                    $strErrorHeader = 'HTTP/1.1 400 Bad Request - Expected array, received ' . gettype($arrQueryStringParams);
+                    $strErrorHeader = 'HTTP/1.1 400 Bad Request - Expected array, received '.gettype($arrQueryStringParams);
                 }
             }
         } else {
@@ -778,7 +778,7 @@ 
                         try {
                             // Load item info to check access rights
                             $itemInfo = DB::queryFirstRow(
-                                'SELECT id, id_tree, label FROM ' . prefixTable('items') . ' WHERE id = %i',
+                                'SELECT id, id_tree, label FROM '.prefixTable('items').' WHERE id = %i',
                                 $itemId
                             );
 
@@ -816,13 +816,13 @@ 
                                 }
                             }
                         } catch (Error $e) {
-                            $strErrorDesc = $e->getMessage() . '. Something went wrong! Please contact support.';
+                            $strErrorDesc = $e->getMessage().'. Something went wrong! Please contact support.';
                             $strErrorHeader = 'HTTP/1.1 500 Internal Server Error';
                         }
                     }
                 } else {
                     $strErrorDesc = 'Data not consistent';
-                    $strErrorHeader = 'HTTP/1.1 400 Bad Request - Expected array, received ' . gettype($arrQueryStringParams);
+                    $strErrorHeader = 'HTTP/1.1 400 Bad Request - Expected array, received '.gettype($arrQueryStringParams);
                 }
             }
         } else {