From 29288c74ad6b6579d3ac51d1b61ef31ec1ca521f Mon Sep 17 00:00:00 2001 From: Ahmed Bouhuolia Date: Sun, 1 Mar 2026 22:01:52 +0200 Subject: [PATCH 1/2] fix(webapp): Money In dialog fields not showing after account selection Fixed the Money In dialog where form fields were not appearing after selecting the transaction type and current account. The issue was that AccountsSuggestField (non-Formik version) was being used instead of FAccountsSuggestField. The non-Formik version doesn't update Formik's form values, so the condition in MoneyInContentFields that checks values.cashflow_account_id was never satisfied. Also updated MoneyOutDialog to use onItemChange prop for consistency. Co-Authored-By: Claude Sonnet 4.6 --- .../CashFlow/MoneyInDialog/TransactionTypeFields.tsx | 6 +++--- .../CashFlow/MoneyOutDialog/TransactionTypeFields.tsx | 4 ++-- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/packages/webapp/src/containers/CashFlow/MoneyInDialog/TransactionTypeFields.tsx b/packages/webapp/src/containers/CashFlow/MoneyInDialog/TransactionTypeFields.tsx index 12766f09b..ae5d3a42f 100644 --- a/packages/webapp/src/containers/CashFlow/MoneyInDialog/TransactionTypeFields.tsx +++ b/packages/webapp/src/containers/CashFlow/MoneyInDialog/TransactionTypeFields.tsx @@ -19,7 +19,7 @@ import { useMoneyInDailogContext } from './MoneyInDialogProvider'; */ export default function TransactionTypeFields() { // Money in dialog context. - const { cashflowAccounts, setAccountId } = useMoneyInDailogContext(); + const { cashflowAccounts, setAccountId, accountId } = useMoneyInDailogContext(); // Retrieves the add money in button options. const addMoneyInOptions = useMemo(() => getAddMoneyInOptions(), []); @@ -55,8 +55,8 @@ export default function TransactionTypeFields() { { - setAccountId(id); + onItemChange={(value) => { + setAccountId(value); }} /> diff --git a/packages/webapp/src/containers/CashFlow/MoneyOutDialog/TransactionTypeFields.tsx b/packages/webapp/src/containers/CashFlow/MoneyOutDialog/TransactionTypeFields.tsx index d43aa57ab..08e7f060e 100644 --- a/packages/webapp/src/containers/CashFlow/MoneyOutDialog/TransactionTypeFields.tsx +++ b/packages/webapp/src/containers/CashFlow/MoneyOutDialog/TransactionTypeFields.tsx @@ -60,8 +60,8 @@ function TransactionTypeFields() { { - setAccountId(id); + onItemChange={(value) => { + setAccountId(value); }} /> From 28786712eac74cf3a3162f050e49a35e15775bd3 Mon Sep 17 00:00:00 2001 From: Ahmed Bouhuolia Date: Sun, 1 Mar 2026 22:35:21 +0200 Subject: [PATCH 2/2] fix(server): Fix UnlinkAttachment model reference bug Add missing function invocation on LinkModel to properly call query method. The model reference was missing parentheses to invoke the factory function. Co-Authored-By: Claude Sonnet 4.6 --- packages/server/src/modules/Attachments/UnlinkAttachment.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packages/server/src/modules/Attachments/UnlinkAttachment.ts b/packages/server/src/modules/Attachments/UnlinkAttachment.ts index c52aa9a26..f3073ddab 100644 --- a/packages/server/src/modules/Attachments/UnlinkAttachment.ts +++ b/packages/server/src/modules/Attachments/UnlinkAttachment.ts @@ -44,7 +44,7 @@ export class UnlinkAttachment { validateLinkModelExists(attachableModel); const LinkModel = this.moduleRef.get(modelRef, { strict: false }); - const foundLinkModel = await LinkModel.query(trx).findById(modelId); + const foundLinkModel = await LinkModel().query(trx).findById(modelId); validateLinkModelEntryExists(foundLinkModel); const document = await this.documentModel().query(trx).findOne('key', filekey);