From 1752c32eecadb896f3f1fd6fb9da63a1017a6ffd Mon Sep 17 00:00:00 2001
From: Ahmed Bouhuolia <a.bouhuolia@gmail.com>
Date: Wed, 21 Feb 2024 17:17:34 +0200
Subject: [PATCH] feat: code review notes

---
 .../BalanceSheet/BalanceSheetNetIncome.ts                 | 1 -
 .../FinancialStatements/BalanceSheet/BalanceSheetTable.ts | 5 +++--
 .../FinancialStatements/FinancialSheetStructure.ts        | 8 ++++----
 .../ProfitLossSheet/ProfitLossSheetExportInjectable.ts    | 7 -------
 .../src/services/FinancialStatements/TableSheetPdf.ts     | 1 +
 .../BalanceSheet/BalanceSheetTable.tsx                    | 1 -
 .../hooks/query/FinancialReports/use-AP-aging-summary.ts  | 2 +-
 .../hooks/query/FinancialReports/use-AR-aging-summary.ts  | 2 +-
 .../src/hooks/query/FinancialReports/use-balance-sheet.ts | 2 +-
 .../hooks/query/FinancialReports/use-cashflow-sheet.ts    | 2 +-
 .../FinancialReports/use-customer-balance-summary.ts      | 2 +-
 .../query/FinancialReports/use-customer-transactions.ts   | 2 +-
 .../hooks/query/FinancialReports/use-general-ledger.ts    | 2 +-
 .../query/FinancialReports/use-inventory-item-details.ts  | 2 +-
 .../query/FinancialReports/use-inventory-valuation.ts     | 2 +-
 .../src/hooks/query/FinancialReports/use-journal-sheet.ts | 2 +-
 .../hooks/query/FinancialReports/use-profit-loss-sheet.ts | 2 +-
 .../query/FinancialReports/use-purchases-by-items.ts      | 2 +-
 .../hooks/query/FinancialReports/use-sales-by-items.ts    | 2 +-
 .../FinancialReports/use-sales-tax-liabilities-summary.ts | 2 +-
 .../query/FinancialReports/use-trial-balance-sheet.ts     | 2 +-
 .../query/FinancialReports/use-vendor-balance-summary.ts  | 2 +-
 .../query/FinancialReports/use-vendor-transactions.ts     | 2 +-
 packages/webapp/src/hooks/query/creditNote.tsx            | 4 +++-
 packages/webapp/src/hooks/query/estimates.tsx             | 4 +++-
 packages/webapp/src/hooks/query/invoices.tsx              | 7 ++++---
 packages/webapp/src/hooks/query/paymentReceives.tsx       | 3 ++-
 packages/webapp/src/hooks/query/receipts.tsx              | 7 +++++--
 packages/webapp/src/hooks/query/transactionsLocking.tsx   | 2 --
 packages/webapp/src/hooks/useRequestPdf.tsx               | 1 +
 30 files changed, 43 insertions(+), 42 deletions(-)

diff --git a/packages/server/src/services/FinancialStatements/BalanceSheet/BalanceSheetNetIncome.ts b/packages/server/src/services/FinancialStatements/BalanceSheet/BalanceSheetNetIncome.ts
index a78624ae2..a6695b370 100644
--- a/packages/server/src/services/FinancialStatements/BalanceSheet/BalanceSheetNetIncome.ts
+++ b/packages/server/src/services/FinancialStatements/BalanceSheet/BalanceSheetNetIncome.ts
@@ -31,7 +31,6 @@ export const BalanceSheetNetIncome = (Base: any) =>
     /**
      * Retrieves the closing balance of income accounts.
      * @returns {number}
-     * 
      */
     private getIncomeTotal = () => {
       const closeingBalance = this.repository.incomeLedger.getClosingBalance();
diff --git a/packages/server/src/services/FinancialStatements/BalanceSheet/BalanceSheetTable.ts b/packages/server/src/services/FinancialStatements/BalanceSheet/BalanceSheetTable.ts
index 66ab87cd5..8e6a1f92b 100644
--- a/packages/server/src/services/FinancialStatements/BalanceSheet/BalanceSheetTable.ts
+++ b/packages/server/src/services/FinancialStatements/BalanceSheet/BalanceSheetTable.ts
@@ -41,13 +41,14 @@ export default class BalanceSheetTable extends R.compose(
   BalanceSheetBase
 )(FinancialSheet) {
   /**
-   * @param {}
+   * Balance sheet data.
+   * @param {IBalanceSheetStatementData}
    */
   private reportData: IBalanceSheetStatementData;
 
   /**
    * Balance sheet query.
-   * @parma {}
+   * @parma {BalanceSheetQuery}
    */
   private query: BalanceSheetQuery;
 
diff --git a/packages/server/src/services/FinancialStatements/FinancialSheetStructure.ts b/packages/server/src/services/FinancialStatements/FinancialSheetStructure.ts
index 722635d9a..5e2fbba55 100644
--- a/packages/server/src/services/FinancialStatements/FinancialSheetStructure.ts
+++ b/packages/server/src/services/FinancialStatements/FinancialSheetStructure.ts
@@ -61,14 +61,14 @@ export const FinancialSheetStructure = (Base: Class) =>
       });
     };
 
-    findNodeDeep = (nodes, callback) => {
+    public findNodeDeep = (nodes, callback) => {
       return findValueDeep(nodes, callback, {
         childrenPath: 'children',
         pathFormat: 'array',
       });
     };
 
-    mapAccNodesDeep = (nodes, callback) => {
+    public mapAccNodesDeep = (nodes, callback) => {
       return reduceDeep(
         nodes,
         (acc, value, key, parentValue, context) => {
@@ -97,11 +97,11 @@ export const FinancialSheetStructure = (Base: Class) =>
       });
     };
 
-    getTotalOfChildrenNodes = (node) => {
+    public getTotalOfChildrenNodes = (node) => {
       return this.getTotalOfNodes(node.children);
     };
 
-    getTotalOfNodes = (nodes) => {
+    public getTotalOfNodes = (nodes) => {
       return sumBy(nodes, 'total.amount');
     };
   };
diff --git a/packages/server/src/services/FinancialStatements/ProfitLossSheet/ProfitLossSheetExportInjectable.ts b/packages/server/src/services/FinancialStatements/ProfitLossSheet/ProfitLossSheetExportInjectable.ts
index 4fa9762d3..ba2371797 100644
--- a/packages/server/src/services/FinancialStatements/ProfitLossSheet/ProfitLossSheetExportInjectable.ts
+++ b/packages/server/src/services/FinancialStatements/ProfitLossSheet/ProfitLossSheetExportInjectable.ts
@@ -40,11 +40,4 @@ export class ProfitLossSheetExportInjectable {
 
     return tableCsv;
   }
-
-  public async pdf(
-    tenantId: number,
-    query: IProfitLossSheetQuery
-  ): Promise<Buffer> {
-    const table = await this.profitLossSheetTable.table(tenantId, query);
-  }
 }
diff --git a/packages/server/src/services/FinancialStatements/TableSheetPdf.ts b/packages/server/src/services/FinancialStatements/TableSheetPdf.ts
index 44317978c..280adf187 100644
--- a/packages/server/src/services/FinancialStatements/TableSheetPdf.ts
+++ b/packages/server/src/services/FinancialStatements/TableSheetPdf.ts
@@ -52,6 +52,7 @@ export class TableSheetPdf {
       landscape,
     });
   }
+
   /**
    * Converts the table columns to pdf columns.
    * @param {ITableColumn[]} columns
diff --git a/packages/webapp/src/containers/FinancialStatements/BalanceSheet/BalanceSheetTable.tsx b/packages/webapp/src/containers/FinancialStatements/BalanceSheet/BalanceSheetTable.tsx
index b4623e4b2..f10da57ba 100644
--- a/packages/webapp/src/containers/FinancialStatements/BalanceSheet/BalanceSheetTable.tsx
+++ b/packages/webapp/src/containers/FinancialStatements/BalanceSheet/BalanceSheetTable.tsx
@@ -9,7 +9,6 @@ import { useBalanceSheetContext } from './BalanceSheetProvider';
 import { useBalanceSheetColumns } from './components';
 import { defaultExpanderReducer, tableRowTypesToClassnames } from '@/utils';
 
-
 /**
  * Balance sheet table.
  */
diff --git a/packages/webapp/src/hooks/query/FinancialReports/use-AP-aging-summary.ts b/packages/webapp/src/hooks/query/FinancialReports/use-AP-aging-summary.ts
index 65e458cea..029f2733d 100644
--- a/packages/webapp/src/hooks/query/FinancialReports/use-AP-aging-summary.ts
+++ b/packages/webapp/src/hooks/query/FinancialReports/use-AP-aging-summary.ts
@@ -58,7 +58,7 @@ export const useAPAgingSheetCsvExport = (query, args) => {
  */
 export function useAPAgingSummaryPdf(query = {}) {
   return useRequestPdf({
-    url: `/api/financial_statements/payable_aging_summary`,
+    url: `/financial_statements/payable_aging_summary`,
     params: query,
   });
 }
diff --git a/packages/webapp/src/hooks/query/FinancialReports/use-AR-aging-summary.ts b/packages/webapp/src/hooks/query/FinancialReports/use-AR-aging-summary.ts
index 61a519b34..b2f8b289b 100644
--- a/packages/webapp/src/hooks/query/FinancialReports/use-AR-aging-summary.ts
+++ b/packages/webapp/src/hooks/query/FinancialReports/use-AR-aging-summary.ts
@@ -57,7 +57,7 @@ export const useARAgingSheetCsvExport = (query, args) => {
  */
 export function useARAgingSummaryPdf(query = {}) {
   return useRequestPdf({
-    url: `/api/financial_statements/receivable_aging_summary`,
+    url: `/financial_statements/receivable_aging_summary`,
     params: query,
   });
 }
diff --git a/packages/webapp/src/hooks/query/FinancialReports/use-balance-sheet.ts b/packages/webapp/src/hooks/query/FinancialReports/use-balance-sheet.ts
index 2eb19959d..ed60756e5 100644
--- a/packages/webapp/src/hooks/query/FinancialReports/use-balance-sheet.ts
+++ b/packages/webapp/src/hooks/query/FinancialReports/use-balance-sheet.ts
@@ -76,7 +76,7 @@ export const useBalanceSheetCsvExport = (query, args) => {
  */
 export function useBalanceSheetPdf(query = {}) {
   return useRequestPdf({
-    url: `/api/financial_statements/balance_sheet`,
+    url: `/financial_statements/balance_sheet`,
     params: query,
   });
 }
diff --git a/packages/webapp/src/hooks/query/FinancialReports/use-cashflow-sheet.ts b/packages/webapp/src/hooks/query/FinancialReports/use-cashflow-sheet.ts
index cdd2a33d8..992431ac2 100644
--- a/packages/webapp/src/hooks/query/FinancialReports/use-cashflow-sheet.ts
+++ b/packages/webapp/src/hooks/query/FinancialReports/use-cashflow-sheet.ts
@@ -77,7 +77,7 @@ export const useCashFlowStatementCsvExport = (query, args) => {
  */
 export function useCashflowSheetPdf(query = {}) {
   return useRequestPdf({
-    url: `/api/financial_statements/cash-flow`,
+    url: `/financial_statements/cash-flow`,
     params: query,
   });
 }
diff --git a/packages/webapp/src/hooks/query/FinancialReports/use-customer-balance-summary.ts b/packages/webapp/src/hooks/query/FinancialReports/use-customer-balance-summary.ts
index 0dbc034d1..4e692832e 100644
--- a/packages/webapp/src/hooks/query/FinancialReports/use-customer-balance-summary.ts
+++ b/packages/webapp/src/hooks/query/FinancialReports/use-customer-balance-summary.ts
@@ -65,7 +65,7 @@ export const useCustomerBalanceSummaryCsvExport = (query, args) => {
  */
 export function useCustomerBalanceSummaryPdf(query = {}) {
   return useRequestPdf({
-    url: `/api/financial_statements/customer-balance-summary`,
+    url: `/financial_statements/customer-balance-summary`,
     params: query,
   });
 }
diff --git a/packages/webapp/src/hooks/query/FinancialReports/use-customer-transactions.ts b/packages/webapp/src/hooks/query/FinancialReports/use-customer-transactions.ts
index a108720c8..fd6925fff 100644
--- a/packages/webapp/src/hooks/query/FinancialReports/use-customer-transactions.ts
+++ b/packages/webapp/src/hooks/query/FinancialReports/use-customer-transactions.ts
@@ -69,7 +69,7 @@ export const useCustomersTransactionsCsvExport = (query, args) => {
  */
 export const useCustomersTransactionsPdfExport = (query = {}) => {
   return useRequestPdf({
-    url: '/api/financial_statements/transactions-by-customers',
+    url: '/financial_statements/transactions-by-customers',
     params: query,
   });
 };
diff --git a/packages/webapp/src/hooks/query/FinancialReports/use-general-ledger.ts b/packages/webapp/src/hooks/query/FinancialReports/use-general-ledger.ts
index 9ecb86258..8f03928b8 100644
--- a/packages/webapp/src/hooks/query/FinancialReports/use-general-ledger.ts
+++ b/packages/webapp/src/hooks/query/FinancialReports/use-general-ledger.ts
@@ -57,7 +57,7 @@ export const useGeneralLedgerSheetCsvExport = (query, args) => {
  */
 export function useGeneralLedgerPdf(query = {}) {
   return useRequestPdf({
-    url: `/api/financial_statements/general_ledger`,
+    url: `/financial_statements/general_ledger`,
     params: query
   });
 }
diff --git a/packages/webapp/src/hooks/query/FinancialReports/use-inventory-item-details.ts b/packages/webapp/src/hooks/query/FinancialReports/use-inventory-item-details.ts
index 4ee534717..38f0c92d0 100644
--- a/packages/webapp/src/hooks/query/FinancialReports/use-inventory-item-details.ts
+++ b/packages/webapp/src/hooks/query/FinancialReports/use-inventory-item-details.ts
@@ -69,7 +69,7 @@ export const useInventoryItemDetailsCsvExport = (query, args) => {
  */
 export function useInventoryItemDetailsPdf(query = {}) {
   return useRequestPdf({
-    url: `/api/financial_statements/inventory-item-details`,
+    url: `/financial_statements/inventory-item-details`,
     params: query,
   });
 }
diff --git a/packages/webapp/src/hooks/query/FinancialReports/use-inventory-valuation.ts b/packages/webapp/src/hooks/query/FinancialReports/use-inventory-valuation.ts
index bed2dd68c..43c39885c 100644
--- a/packages/webapp/src/hooks/query/FinancialReports/use-inventory-valuation.ts
+++ b/packages/webapp/src/hooks/query/FinancialReports/use-inventory-valuation.ts
@@ -77,7 +77,7 @@ export const useInventoryValuationCsvExport = (query, args) => {
  */
 export function useInventoryValuationPdf(query = {}) {
   return useRequestPdf({
-    url: `/api/financial_statements/inventory-valuation`,
+    url: `/financial_statements/inventory-valuation`,
     params: query,
   });
 }
diff --git a/packages/webapp/src/hooks/query/FinancialReports/use-journal-sheet.ts b/packages/webapp/src/hooks/query/FinancialReports/use-journal-sheet.ts
index f83531a75..90c6cb447 100644
--- a/packages/webapp/src/hooks/query/FinancialReports/use-journal-sheet.ts
+++ b/packages/webapp/src/hooks/query/FinancialReports/use-journal-sheet.ts
@@ -58,7 +58,7 @@ export const useJournalSheetCsvExport = (query, args) => {
  */
 export const useJournalSheetPdf = (query = {}) => {
   return useRequestPdf({
-    url: `/api/financial_statements/journal`,
+    url: `/financial_statements/journal`,
     params: query,
   });
 };
diff --git a/packages/webapp/src/hooks/query/FinancialReports/use-profit-loss-sheet.ts b/packages/webapp/src/hooks/query/FinancialReports/use-profit-loss-sheet.ts
index cbc9d9cf3..b254e4ba2 100644
--- a/packages/webapp/src/hooks/query/FinancialReports/use-profit-loss-sheet.ts
+++ b/packages/webapp/src/hooks/query/FinancialReports/use-profit-loss-sheet.ts
@@ -58,7 +58,7 @@ export const useProfitLossSheetCsvExport = (query, args) => {
  */
 export function useProfitLossSheetPdf(query = {}) {
   return useRequestPdf({
-    url: `/api/financial_statements/profit_loss_sheet`,
+    url: `/financial_statements/profit_loss_sheet`,
     params: query,
   });
 }
diff --git a/packages/webapp/src/hooks/query/FinancialReports/use-purchases-by-items.ts b/packages/webapp/src/hooks/query/FinancialReports/use-purchases-by-items.ts
index df391f5da..882aaf578 100644
--- a/packages/webapp/src/hooks/query/FinancialReports/use-purchases-by-items.ts
+++ b/packages/webapp/src/hooks/query/FinancialReports/use-purchases-by-items.ts
@@ -73,7 +73,7 @@ export const usePurchasesByItemsXlsxExport = (query, args) => {
  */
 export const usePurchasesByItemsPdfExport = (query = {}) => {
   return useRequestPdf({
-    url: '/api/financial_statements/purchases-by-items',
+    url: '/financial_statements/purchases-by-items',
     params: query,
   });
 };
diff --git a/packages/webapp/src/hooks/query/FinancialReports/use-sales-by-items.ts b/packages/webapp/src/hooks/query/FinancialReports/use-sales-by-items.ts
index 1aae74058..c4feda108 100644
--- a/packages/webapp/src/hooks/query/FinancialReports/use-sales-by-items.ts
+++ b/packages/webapp/src/hooks/query/FinancialReports/use-sales-by-items.ts
@@ -72,7 +72,7 @@ export const useSalesByItemsXlsxExport = (query, args) => {
 
 export const useSalesByItemsPdfExport = (query = {}) => {
   return useRequestPdf({
-    url: '/api/financial_statements/sales-by-items',
+    url: '/financial_statements/sales-by-items',
     params: query,
   });
 };
diff --git a/packages/webapp/src/hooks/query/FinancialReports/use-sales-tax-liabilities-summary.ts b/packages/webapp/src/hooks/query/FinancialReports/use-sales-tax-liabilities-summary.ts
index cf79f4f14..85cf0eda5 100644
--- a/packages/webapp/src/hooks/query/FinancialReports/use-sales-tax-liabilities-summary.ts
+++ b/packages/webapp/src/hooks/query/FinancialReports/use-sales-tax-liabilities-summary.ts
@@ -58,7 +58,7 @@ export const useSalesTaxLiabilitySummaryCsvExport = (query, args) => {
  */
 export function useSalesTaxLiabilitySummaryPdf(query = {}) {
   return useRequestPdf({
-    url: `/api/financial_statements/sales-tax-liability-summary`,
+    url: `/financial_statements/sales-tax-liability-summary`,
     params: query,
   });
 }
diff --git a/packages/webapp/src/hooks/query/FinancialReports/use-trial-balance-sheet.ts b/packages/webapp/src/hooks/query/FinancialReports/use-trial-balance-sheet.ts
index 1fe324cd4..96afe488c 100644
--- a/packages/webapp/src/hooks/query/FinancialReports/use-trial-balance-sheet.ts
+++ b/packages/webapp/src/hooks/query/FinancialReports/use-trial-balance-sheet.ts
@@ -58,7 +58,7 @@ export const useTrialBalanceSheetCsvExport = (query, args) => {
  */
 export function useTrialBalanceSheetPdf(query = {}) {
   return useRequestPdf({
-    url: `/api/financial_statements/trial_balance_sheet`,
+    url: `/financial_statements/trial_balance_sheet`,
     params: query,
   });
 }
diff --git a/packages/webapp/src/hooks/query/FinancialReports/use-vendor-balance-summary.ts b/packages/webapp/src/hooks/query/FinancialReports/use-vendor-balance-summary.ts
index 9986e0e28..840a5c08a 100644
--- a/packages/webapp/src/hooks/query/FinancialReports/use-vendor-balance-summary.ts
+++ b/packages/webapp/src/hooks/query/FinancialReports/use-vendor-balance-summary.ts
@@ -65,7 +65,7 @@ export const useVendorBalanceSummaryCsvExport = (args) => {
 
 export const useVendorBalanceSummaryPdfExport = (query = {}) => {
   return useRequestPdf({
-    url: '/api/financial_statements/vendor-balance-summary',
+    url: 'financial_statements/vendor-balance-summary',
     params: query,
   });
 };
diff --git a/packages/webapp/src/hooks/query/FinancialReports/use-vendor-transactions.ts b/packages/webapp/src/hooks/query/FinancialReports/use-vendor-transactions.ts
index d52900f04..7c9dc5f45 100644
--- a/packages/webapp/src/hooks/query/FinancialReports/use-vendor-transactions.ts
+++ b/packages/webapp/src/hooks/query/FinancialReports/use-vendor-transactions.ts
@@ -68,7 +68,7 @@ export const useVendorsTransactionsCsvExport = (query, args) => {
  */
 export function useTransactionsByVendorsPdf(query = {}) {
   return useRequestPdf({
-    url: `/api/financial_statements/transactions-by-vendors`,
+    url: `financial_statements/transactions-by-vendors`,
     params: query,
   });
 }
diff --git a/packages/webapp/src/hooks/query/creditNote.tsx b/packages/webapp/src/hooks/query/creditNote.tsx
index 0fb31f873..fce2f7302 100644
--- a/packages/webapp/src/hooks/query/creditNote.tsx
+++ b/packages/webapp/src/hooks/query/creditNote.tsx
@@ -354,5 +354,7 @@ export function useRefundCreditTransaction(id, props, requestProps) {
  * Retrieve the credit note pdf document data,
  */
 export function usePdfCreditNote(creditNoteId) {
-  return useRequestPdf(`sales/credit_notes/${creditNoteId}`);
+  return useRequestPdf({
+    url: `sales/credit_notes/${creditNoteId}`,
+  });
 }
diff --git a/packages/webapp/src/hooks/query/estimates.tsx b/packages/webapp/src/hooks/query/estimates.tsx
index 9ab867857..ed5f11ef4 100644
--- a/packages/webapp/src/hooks/query/estimates.tsx
+++ b/packages/webapp/src/hooks/query/estimates.tsx
@@ -181,7 +181,9 @@ export function useRejectEstimate(props) {
  */
 
 export function usePdfEstimate(estimateId) {
-  return useRequestPdf(`sales/estimates/${estimateId}`);
+  return useRequestPdf({
+    url: `sales/estimates/${estimateId}`,
+  });
 }
 
 export function useRefreshEstimates() {
diff --git a/packages/webapp/src/hooks/query/invoices.tsx b/packages/webapp/src/hooks/query/invoices.tsx
index 886b04e7d..72a8bc7b1 100644
--- a/packages/webapp/src/hooks/query/invoices.tsx
+++ b/packages/webapp/src/hooks/query/invoices.tsx
@@ -24,7 +24,7 @@ const commonInvalidateQueries = (queryClient) => {
 
   // Invalidate financial reports.
   queryClient.invalidateQueries(t.FINANCIAL_REPORT);
-  
+
   // Invalidate transactions by reference.
   queryClient.invalidateQueries(t.TRANSACTIONS_BY_REFERENCE);
 
@@ -188,7 +188,9 @@ export function useInvoice(invoiceId, props, requestProps) {
  * Retrieve the invoice pdf document data.
  */
 export function usePdfInvoice(invoiceId) {
-  return useRequestPdf(`sales/invoices/${invoiceId}`);
+  return useRequestPdf({
+    url: `sales/invoices/${invoiceId}`,
+  });
 }
 
 /**
@@ -339,4 +341,3 @@ export function useSaleInvoiceDefaultOptions(invoiceId, props) {
     },
   );
 }
-
diff --git a/packages/webapp/src/hooks/query/paymentReceives.tsx b/packages/webapp/src/hooks/query/paymentReceives.tsx
index 5ed0118ee..5fbd03a28 100644
--- a/packages/webapp/src/hooks/query/paymentReceives.tsx
+++ b/packages/webapp/src/hooks/query/paymentReceives.tsx
@@ -233,9 +233,10 @@ export function usePaymentReceiveSMSDetail(
 
 /**
  * Retrieve the payment receive pdf document data.
+ * @param {number} paymentReceiveId - Payment receive id.
  */
 export function usePdfPaymentReceive(paymentReceiveId) {
-  return useRequestPdf(`sales/payment_receives/${paymentReceiveId}`);
+  return useRequestPdf({ url: `sales/payment_receives/${paymentReceiveId}` });
 }
 
 export function useSendPaymentReceiveMail(props) {
diff --git a/packages/webapp/src/hooks/query/receipts.tsx b/packages/webapp/src/hooks/query/receipts.tsx
index fed4ad5af..2797008d3 100644
--- a/packages/webapp/src/hooks/query/receipts.tsx
+++ b/packages/webapp/src/hooks/query/receipts.tsx
@@ -162,9 +162,12 @@ export function useReceipt(id, props) {
 
 /**
  * Retrieve the receipt pdf document data.
+ * @param {number} receiptId -
  */
-export function usePdfReceipt(ReceiptId) {
-  return useRequestPdf(`sales/receipts/${ReceiptId}`);
+export function usePdfReceipt(receiptId: number) {
+  return useRequestPdf({
+    url: `sales/receipts/${receiptId}`,
+  });
 }
 
 export function useRefreshReceipts() {
diff --git a/packages/webapp/src/hooks/query/transactionsLocking.tsx b/packages/webapp/src/hooks/query/transactionsLocking.tsx
index 0be24a26a..e28555fcf 100644
--- a/packages/webapp/src/hooks/query/transactionsLocking.tsx
+++ b/packages/webapp/src/hooks/query/transactionsLocking.tsx
@@ -1,9 +1,7 @@
 // @ts-nocheck
 import { useQueryClient, useMutation } from 'react-query';
 import { useRequestQuery } from '../useQueryRequest';
-import { transformPagination } from '@/utils';
 import useApiRequest from '../useRequest';
-import { useRequestPdf } from '../utils';
 import t from './types';
 
 // Common invalidate queries.
diff --git a/packages/webapp/src/hooks/useRequestPdf.tsx b/packages/webapp/src/hooks/useRequestPdf.tsx
index c729ed185..55d797b2f 100644
--- a/packages/webapp/src/hooks/useRequestPdf.tsx
+++ b/packages/webapp/src/hooks/useRequestPdf.tsx
@@ -16,6 +16,7 @@ export const useRequestPdf = (httpProps) => {
         headers: { accept: 'application/pdf' },
         responseType: 'blob',
         ...httpProps,
+        url: `/api/${httpProps?.url}`,
       })
       .then((response) => {
         // Create a Blob from the PDF Stream.