fix: Refactor store and update methods in RecurringInvoices controller to merge 'issued_at' directly into the request before dispatching document creation and update actions.

This commit is contained in:
Hayatunnabi Nabil 2025-11-06 02:40:53 +06:00
parent 8fb34dc8d2
commit f720e22117
1 changed files with 6 additions and 2 deletions

View File

@ -80,7 +80,9 @@ class RecurringInvoices extends Controller
*/
public function store(Request $request)
{
$response = $this->ajaxDispatch(new CreateDocument($request->merge(['issued_at' => $request->get('recurring_started_at')])));
$request->merge(['issued_at' => $request->get('recurring_started_at')]);
$response = $this->ajaxDispatch(new CreateDocument($request));
if ($response['success']) {
$response['redirect'] = route('recurring-invoices.show', $response['data']->id);
@ -163,7 +165,9 @@ class RecurringInvoices extends Controller
*/
public function update(Document $recurring_invoice, Request $request)
{
$response = $this->ajaxDispatch(new UpdateDocument($recurring_invoice, $request->merge(['issued_at' => $request->get('recurring_started_at')])));
$request->merge(['issued_at' => $request->get('recurring_started_at')]);
$response = $this->ajaxDispatch(new UpdateDocument($recurring_invoice, $request));
if ($response['success']) {
$response['redirect'] = route('recurring-invoices.show', $response['data']->id);