cakephp: saving to multiple models using one form

Question

I have one form that saves a person with many parts and many colors but it's not saving the parts and the colors; only saving the person(main model).

On my list view, part of script:

echo $this->Form->input('Person.person_name', array('required' => true, 'placeholder' => 'Name Your Person', 'label' => false) );
echo $this->Form->input('Part.0.part_name', array('required' => true, 'placeholder' => 'Add Part', 'label' => false) ); 
echo $this->Form->input('Color.0.color_name', array('required' => true, 'placeholder' => 'Enter a Color', 'label' => false) );

On my list controller:

if ($this->request->is('post')) {
            $created = $this->Person->saveAll($this->request->data, array('validate'=>'first'));
            if (!empty($created)) {
                $this->Session->setFlash(__('The person and his/her parts and colors have been saved.'));
                return $this->redirect(array('action' => 'index'));                 
            } else {
                $this->Session->setFlash(__('The person could not be saved. Please, try again.'));
            }
        }

On my Person Model:

public $hasMany = array(
    'PersonParts' => array(
        'className' => 'Part',
        'foreignKey' => 'part_person_id'
    ),
    'PersonColors' => array(
        'className' => 'Color',
        'foreignKey' => 'color_person_id'
    )
);

Tried replacing saveAll with saveAssociated but it's the same. What am I doing wrong here? Any help is greatly appreciated.

do you have three models right ? and values should store in different tables — Manish Patel
– Manish Patel, Commented Dec 23, 2013 at 10:27
It would be a good idea to maintain your model aliases as singular PersonParts -> PersonPart - otherwise certain aspects of your app will be confusing (such as needing form fields containing the name PersonParts) — AD7six
– AD7six, Commented Dec 23, 2013 at 10:42

Ahmed Siouani · Accepted Answer · 2013-12-23 13:26:17Z

3

You must add original name of alias in form (alias like in hasMany variable):

echo $this->Form->input('Person.person_name');
echo $this->Form->input('PersonParts.0.part_name'); 
echo $this->Form->input('PersonColors.0.color_name');

edited Dec 23, 2013 at 13:26

Ahmed Siouani

13.9k12 gold badges63 silver badges73 bronze badges

answered Dec 23, 2013 at 10:33

kicaj

2,9785 gold badges44 silver badges77 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

Leah Over a year ago

Thanks. Gosh. Didn't know I have to do that. It's not in the cakephp doc eh?

Moyed Ansari · Accepted Answer · 2013-12-23 10:29:12Z

1

Try this

echo $this->Form->input('Person.person_name', array('required' => true, 'placeholder' => 'Name Your Person', 'label' => false) );
echo $this->Form->input('PersonParts.0.part_name', array('required' => true, 'placeholder' => 'Add Part', 'label' => false) ); 
echo $this->Form->input('PersonColors.0.color_name', array('required' => true, 'placeholder' => 'Enter a Color', 'label' => false) );

answered Dec 23, 2013 at 10:29

Moyed Ansari

8,4612 gold badges38 silver badges57 bronze badges

Comments

Moyed Ansari · Accepted Answer · 2013-12-23 11:00:53Z

1

Update your form to

echo $this->Form->input('Part.part_name',...)
echo $this->Form->input('Color.color_name',...)

And in your controller, use

**saveAssociated**

edited Dec 23, 2013 at 11:00

Moyed Ansari

8,4612 gold badges38 silver badges57 bronze badges

answered Dec 23, 2013 at 10:25

XuDing

2,07219 silver badges27 bronze badges

1 Comment

AD7six Over a year ago

SaveAll is in context an alias for SaveAssociated. The form structure you propose isn't appropriate for a has-many relationship.

Manish Patel · Accepted Answer · 2013-12-23 10:33:32Z

0

To save you could use something like:

$this->Model->saveMany($data, array('deep'=>TRUE));

Note that the "deep" option requires CakePHP 2.1. Without it the associated Comment records would not be saved.

All this is documented in http://book.cakephp.org/2.0/en/models/saving-your-data.html

Also correct it

echo $this->Form->input('Person.person_name', array('required' => true, 'placeholder' => 'Name Your Person', 'label' => false) );
echo $this->Form->input('Part.part_name', array('required' => true, 'placeholder' => 'Add Part', 'label' => false) ); 
echo $this->Form->input('Color.color_name', array('required' => true, 'placeholder' => 'Enter a Color', 'label' => false) );

answered Dec 23, 2013 at 10:33

Manish Patel

1,9771 gold badge14 silver badges15 bronze badges

1 Comment

AD7six Over a year ago

SaveMany is for saving many rows of the same type. That's the wrong method for the question.

Carl0s1z · Accepted Answer · 2014-10-20 09:56:44Z

0

I think you need to save everything independently.

What do you think of this possibility?

if yes then go through.

$this->Person->Part->save();
$this->Person->Color->save();

edited Oct 20, 2014 at 9:56

Carl0s1z

4,7237 gold badges34 silver badges49 bronze badges

answered Dec 23, 2013 at 10:29

Badru

412 bronze badges

Collectives™ on Stack Overflow

cakephp: saving to multiple models using one form

5 Answers 5

1 Comment

Comments

1 Comment

1 Comment

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

5 Answers 5

1 Comment

Comments

1 Comment

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related