Skip to content

Conversation

@imhayatunnabi
Copy link
Contributor

Issue Resolved:

  • When creating or editing a recurring invoice, RecurringInvoices::store() and RecurringInvoices::update() dispatch CreateDocument / UpdateDocument jobs by calling new CreateDocument($request->merge([...])).
  • FormRequest::merge() mutates the request but returns void, so the expression ends up passing null into the job constructors.
  • The job boot logic sees no request payload, builds an empty mock request, and the invoice gets created/updated without any of the validated fields (type, contact, items, schedule), leading to errors or corrupt data across the recurring invoices feature and any integrations that rely on it.

#3315

…r to merge 'issued_at' directly into the request before dispatching document creation and update actions.
@cuneytsenturk cuneytsenturk added this to the 3.1.20 milestone Nov 8, 2025
@cuneytsenturk cuneytsenturk merged commit ac58f4d into akaunting:master Nov 8, 2025
@cuneytsenturk
Copy link
Member

Thanks for your help

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants