如何以同一形式将数据提交到多个表

Question

I have 2 tables: listings and listings_specifications

Listings table

• id
• type
• status
• location
• specifications_id

Listings_specifications table

• id
• listing_id
• swimming_pool
• water_well

I need to select the specifications (checkboxes) on the same form with which I add a listing. I have created all the forms, views, models, controllers but I think I got some logic wrong.

Listing.php model

protected $table = 'listings';

public function contact()
{
   return $this->BelongsTo('contacts');
}
public function specifications()
{
   return $this->BelongsTo('listings_specifications');
}

Specification.php model

protected $table = 'listings_specifications';

public function listings()
{
   return $this->BelongsTo('listings');
}

ListingsController.php (where the data gets saved in the database)

$listing = new Listing;
                $contact = new Contact;
                $listing->status = Input::get('status');
                $listing->listingfor = Input::get('listingfor');
                $listing->propertystatus = Input::get('propertystatus');
                $listing->propertytype = Input::get('propertytype');
                $listing->userid = Auth::user()->id;
                $listing->location = Input::get('location');
                $listing->contact_id = $contact_id;

                $listing->save();

                $specifications = Specification::find($id);

                if( $listings->save() ) {
                        $specifications = new Specification;
                        $specifications->listing_id = $id;
                        $specifications->swimming_pool       = Input::get('swimming_pool');
                        $specifications->water_front      = Input::get('water_front');
                        $specifications->save();
                }

I'm getting this error: Undefined variable: id

Where did I go wrong?

Thank you

Answer 1

You use $id in the line $specifications = Specification::find($id); but you don't define it before.

Answer 2

It looks like you have some logic errors.

First of all, you are never setting $id anywhere, but that's okay because you really don't need it.

Remove the $specifications = Specification::find($id); line because that's not doing anything.

Then change your last section to something like this...

            if( $listings->save() ) {
                    $specifications = new Specification;
                    $specifications->swimming_pool       = Input::get('swimming_pool');
                    $specifications->water_front      = Input::get('water_front');
                    $listing->specifications()->save($specifications);
            }

$listing->specifications()->save($specifications); will automatically save the new specification with the correct listing_id for you.

Modify your Listing model's specifications relationship to...

public function specifications()
{
   return $this->hasMany('Specification');
}

I'm assuming here one listing can have many specifications. If not, you can easily just change that to a hasOne .