Laravel 5.2从动态表单插入和更新数据

问题描述:

So far, I have found answers in here and never needed to ask anything. But this issue is really bugging me. Here is the situation. I have a dynamic invoice form. Posts from the form will be saved to two different tables

  1. invoices(where details for invoice is stored like member_id, discount, duedate, remarks and the timestamps)
  2. invoiceitems(invoice_id, itemdescription, quantity, rate, timestamps)

To achieve this, here is my method to store invoice

public function store(InvoiceRequest $request)
{

//get necessary details for invoice table
$datainvoice = array(
'member_id' => $request->input('member_id'),
'duedate' => $request->input('duedate'),
'discount' => $request->input('discount'),
'remarks' => $request->input('remarks'),
//'gtotal' => $request->input ('gtotal'),
);

//save result of invoice
$resultforinvoice = Invoice::create($datainvoice);


// ready all data of invoice items
$datainvoiceitem = array(
'description' => $request->input('description'),
'rate' => $request->input('rate'),
'quantity' => $request->input('quantity'),
'invoice_id' => $resultforinvoice->id,
);


 // insert invoice along with invoice_id
Invoiceitem::create($datainvoiceitem);
return redirect('members');

}

Additional Information:

Relationship between members and invoices:: member-has-many-invoices, invoice-belongs-to-single-member

Relationship between invoice and invoiceitems:: invoice-has-many-items, an-item-belongs-to-one-invoice

partial of the dynamic form:

<td>  1   </td>
<td><textarea type="text" name='description[0][description]'   class="form-control"/></textarea> </td>
<td><input type="text" name='quantity[0][quantity]'  class="form-control"/></td>
<td> <input type="text" name='rate[0][rate]'  class="form-control"/>   </td>

Errors im receiving right now Argument 1 passed to Illuminate\Database\Grammar::parameterize() must be of the type array, integer given, called in C:\xampp\htdocs\study\vendor\laravel\framework\src\Illuminate\Database\Query\Grammars\Grammar.php on line 719 and defined

Update

when i try to return the inputs for invoiceitems to check whats going on, i get an array like below

{
"description": [
{
    "description": "desc1"
},
{
    "description": "desc2"
}
],
"rate": [
{
    "rate": "11"
},
{
    "rate": "22"
}
],
"quantity": [
{
    "quantity": "1"
},
{
    "quantity": "2"
}
],
"invoice_id": 41

}

Guys, please tell let me know where im missing. I have tried asking this in laracast as well, its been couple of hours nothing at the moment, hopefully someone will point me to right direction. Here is the laracast link.

I ended up updating majority of the code and now it looks like this

 $items = [];
    foreach ($request->items as $field => $values) {
        foreach ($values as $index => $value) {
            $items[$index][$field] = $value;
        }
    }


    Invoice::create($invoice)
        ->invoiceitem()->createMany($items);

and it just worked. Let me know anyone else needs help with this. I would be happy to explain

May I suggest

public function store(InvoiceRequest $request)
{
    $invoice_fields = ['member_id', 'duedate', 'discount', 'remarks', 'gtotal'];

    $items = [];

    foreach ($request->items as $field => $values) {
        foreach ($values as $index => $value) {
            $items[$index][$field] = $value;
        }
    }        

    Invoice::create($request->only($invoice_fields))
        ->items()->createMany($items);

    return redirect('members');    
}

(If you set up the relation properly, you don't need to manually assign the invoice_id to each item, just have the Invoice create the Item).


You may abstract the foreachs into a separate method, a Collection macro may be a good idea as the tutorial suggests (although I implement it differently; my suggestion doesn't need index numbers in the original HTML). The HTML for your form should look like

<input type="text" name="items[description][]">
<input type="text" name="items[quantity][]">
<input type="text" name="items[rate][]">

<input type="text" name="items[description][]">   <!-- as many of --> 
<input type="text" name="items[quantity][]">      <!-- these      -->
<input type="text" name="items[rate][]">          <!-- as needed  -->