fix: bugs sprint

packages/server/src/modules/Expenses/ExpensesImportable.ts

@@ -5,10 +5,10 @@ import { ExpensesSampleData } from './constants';
 import { CreateExpense } from './commands/CreateExpense.service';
 import { CreateExpenseDto } from './dtos/Expense.dto';
 import { ImportableService } from '../Import/decorators/Import.decorator';
-import { ManualJournal } from '../ManualJournals/models/ManualJournal';
+import { Expense } from './models/Expense.model';
 
 @Injectable()
-@ImportableService({ name: ManualJournal.name })
+@ImportableService({ name: Expense.name })
 export class ExpensesImportable extends Importable {
   constructor(private readonly createExpenseService: CreateExpense) {
     super();

packages/server/src/modules/Import/ImportFileDataTransformer.ts

@@ -18,7 +18,7 @@ import { CurrencyParsingDTOs } from './_constants';
 export class ImportFileDataTransformer {
   constructor(
     private readonly resource: ResourceService,
-  ) {}
+  ) { }
 
   /**
    * Parses the given sheet data before passing to the service layer.

packages/server/src/modules/Import/ImportableRegistry.ts

@@ -6,7 +6,7 @@ import { ContextIdFactory, ModuleRef } from '@nestjs/core';
 
 @Injectable()
 export class ImportableRegistry {
-  constructor(private readonly moduleRef: ModuleRef) {}
+  constructor(private readonly moduleRef: ModuleRef) { }
   /**
    * Retrieves the importable service instance of the given resource name.
    * @param {string} name
@@ -15,6 +15,12 @@ export class ImportableRegistry {
   public async getImportable(name: string) {
     const _name = this.sanitizeResourceName(name);
     const importable = getImportableService(_name);
+
+    if (!importable) {
+      throw new Error(
+        `No importable service found for resource "${_name}". Make sure the resource has an @ImportableService decorator registered.`,
+      );
+    }
     const contextId = ContextIdFactory.create();
 
     const importableInstance = await this.moduleRef.resolve(importable, contextId, {

packages/server/src/modules/Import/_utils.ts

@@ -253,28 +253,28 @@ export const getResourceColumns = (resourceColumns: {
 }) => {
   const mapColumn =
     (group: string) =>
-    ([fieldKey, { name, importHint, required, order, ...field }]: [
-      string,
-      IModelMetaField2,
-    ]) => {
-      const extra: Record<string, any> = {};
-      const key = fieldKey;
+      ([fieldKey, { name, importHint, required, order, ...field }]: [
+        string,
+        IModelMetaField2,
+      ]) => {
+        const extra: Record<string, any> = {};
+        const key = fieldKey;
 
-      if (group) {
-        extra.group = group;
-      }
-      if (field.fieldType === 'collection') {
-        extra.fields = mapColumns(field.fields, key);
-      }
-      return {
-        key,
-        name,
-        required,
-        hint: importHint,
-        order,
-        ...extra,
+        if (group) {
+          extra.group = group;
+        }
+        if (field.fieldType === 'collection') {
+          extra.fields = mapColumns(field.fields, key);
+        }
+        return {
+          key,
+          name,
+          required,
+          hint: importHint,
+          order,
+          ...extra,
+        };
       };
-    };
   const sortColumn = (a, b) =>
     a.order && b.order ? a.order - b.order : a.order ? -1 : b.order ? 1 : 0;
 
@@ -287,49 +287,49 @@ export const getResourceColumns = (resourceColumns: {
 // Prases the given object value based on the field key type.
 export const valueParser =
   (fields: ResourceMetaFieldsMap, tenantModels: any, trx?: Knex.Transaction) =>
-  async (value: any, key: string, group = '') => {
-    let _value = value;
+    async (value: any, key: string, group = '') => {
+      let _value = value;
 
-    const fieldKey = key.includes('.') ? key.split('.')[0] : key;
-    const field = group ? fields[group]?.fields[fieldKey] : fields[fieldKey];
+      const fieldKey = key.includes('.') ? key.split('.')[0] : key;
+      const field = group ? fields[group]?.fields[fieldKey] : fields[fieldKey];
 
-    // Parses the boolean value.
-    if (field.fieldType === 'boolean') {
-      _value = parseBoolean(value);
+      // Parses the boolean value.
+      if (field.fieldType === 'boolean') {
+        _value = parseBoolean(value);
 
-      // Parses the enumeration value.
-    } else if (field.fieldType === 'enumeration') {
-      const option = get(field, 'options', []).find(
-        (option) => option.label?.toLowerCase() === value?.toLowerCase(),
-      );
-      _value = get(option, 'key');
-      // Parses the numeric value.
-    } else if (field.fieldType === 'number') {
-      _value = multiNumberParse(value);
-      // Parses the relation value.
-    } else if (field.fieldType === 'relation') {
-      const RelationModel = tenantModels[field.relationModel];
+        // Parses the enumeration value.
+      } else if (field.fieldType === 'enumeration') {
+        const option = get(field, 'options', []).find(
+          (option) => option.label?.toLowerCase() === value?.toLowerCase(),
+        );
+        _value = get(option, 'key');
+        // Parses the numeric value.
+      } else if (field.fieldType === 'number') {
+        _value = multiNumberParse(value);
+        // Parses the relation value.
+      } else if (field.fieldType === 'relation') {
+        const RelationModel = tenantModels[field.relationModel];
 
-      if (!RelationModel) {
-        throw new Error(`The relation model of ${key} field is not exist.`);
-      }
-      const relationQuery = RelationModel.query(trx);
-      const relationKeys = castArray(field?.relationImportMatch);
+        if (!RelationModel) {
+          throw new Error(`The relation model of ${key} field is not exist.`);
+        }
+        const relationQuery = RelationModel.query(trx);
+        const relationKeys = castArray(field?.relationImportMatch);
 
-      relationQuery.where(function () {
-        relationKeys.forEach((relationKey: string) => {
-          this.orWhereRaw('LOWER(??) = LOWER(?)', [relationKey, value]);
+        relationQuery.where(function () {
+          relationKeys.forEach((relationKey: string) => {
+            this.orWhereRaw('LOWER(??) = LOWER(?)', [relationKey, value]);
+          });
         });
-      });
-      const result = await relationQuery.first();
-      _value = get(result, 'id');
-    } else if (field.fieldType === 'collection') {
-      const ObjectFieldKey = key.includes('.') ? key.split('.')[1] : key;
-      const _valueParser = valueParser(fields, tenantModels);
-      _value = await _valueParser(value, ObjectFieldKey, fieldKey);
-    }
-    return _value;
-  };
+        const result = await relationQuery.first();
+        _value = get(result, 'id');
+      } else if (field.fieldType === 'collection') {
+        const ObjectFieldKey = key.includes('.') ? key.split('.')[1] : key;
+        const _valueParser = valueParser(fields, tenantModels);
+        _value = await _valueParser(value, ObjectFieldKey, fieldKey);
+      }
+      return _value;
+    };
 
 /**
  * Parses the field key and detarmines the key path.

packages/server/src/modules/SaleInvoices/commands/SaleInvoicesImportable.ts

@@ -5,10 +5,10 @@ import { Importable } from '@/modules/Import/Importable';
 import { CreateSaleInvoiceDto } from '../dtos/SaleInvoice.dto';
 import { SaleInvoicesSampleData } from '../constants';
 import { ImportableService } from '@/modules/Import/decorators/Import.decorator';
-import { ManualJournal } from '@/modules/ManualJournals/models/ManualJournal';
+import { SaleInvoice } from '../models/SaleInvoice';
 
 @Injectable()
-@ImportableService({ name: ManualJournal.name })
+@ImportableService({ name: SaleInvoice.name })
 export class SaleInvoicesImportable extends Importable {
   constructor(private readonly createInvoiceService: CreateSaleInvoice) {
     super();

packages/server/src/modules/Tenancy/TenancyModels/Tenancy.module.ts

@@ -107,4 +107,4 @@ const modelProviders = models.map((model) => RegisterTenancyModel(model));
   imports: [...modelProviders],
   exports: [...modelProviders],
 })
-export class TenancyModelsModule {}
+export class TenancyModelsModule { }

packages/webapp/src/components/Accounts/AccountsSuggestField.tsx

@@ -102,6 +102,7 @@ function AccountsSuggestFieldRoot({
   return (
     <FSuggest
       items={filteredAccounts}
+      itemPredicate={filterAccountsPredicater}
       onCreateItemSelect={handleCreateItemSelect}
       valueAccessor="id"
       textAccessor="name"

packages/webapp/src/containers/Expenses/ExpenseForm/ExpenseForm.tsx

@@ -48,7 +48,7 @@ function ExpenseForm({
     createExpenseMutate,
     expense,
     expenseId,
-    submitPayload,
+    submitPayloadRef,
   } = useExpenseFormContext();
 
   const isNewMode = !expenseId;
@@ -86,9 +86,12 @@ function ExpenseForm({
       return;
     }
 
+    // Get submit payload from ref for synchronous access
+    const currentSubmitPayload = submitPayloadRef?.current || {};
+
     const form = {
       ...transformFormValuesToRequest(values),
-      publish: submitPayload.publish,
+      publish: currentSubmitPayload.publish,
     };
     // Handle request success.
     const handleSuccess = (response) => {
@@ -103,10 +106,10 @@ function ExpenseForm({
       });
       setSubmitting(false);
 
-      if (submitPayload.redirect) {
+      if (currentSubmitPayload.redirect) {
         history.push('/expenses');
       }
-      if (submitPayload.resetForm) {
+      if (currentSubmitPayload.resetForm) {
         resetForm();
       }
     };

packages/webapp/src/containers/Expenses/ExpenseForm/ExpenseFormPageProvider.tsx

@@ -59,8 +59,13 @@ function ExpenseFormPageProvider({ query, expenseId, ...props }) {
   const { mutateAsync: createExpenseMutate } = useCreateExpense();
   const { mutateAsync: editExpenseMutate } = useEditExpense();
 
-  // Submit form payload.
-  const [submitPayload, setSubmitPayload] = React.useState({});
+  // Submit form payload - using ref for synchronous access.
+  const submitPayloadRef = React.useRef({});
+
+  // Setter to update the ref.
+  const setSubmitPayload = React.useCallback((payload) => {
+    submitPayloadRef.current = payload;
+  }, []);
 
   // Detarmines whether the form in new mode.
   const isNewMode = !expenseId;
@@ -69,7 +74,7 @@ function ExpenseFormPageProvider({ query, expenseId, ...props }) {
   const provider = {
     isNewMode,
     expenseId,
-    submitPayload,
+    submitPayloadRef, // Expose ref for synchronous access
 
     currencies,
     customers,