Saving Data To Two Db Tables With Two Models Inyii

Yii 1.1 General Discussion
1

I’m attempting to save data to two db tables with two different models in Yii. I’ve consulted the wiki: http://www.yiiframework.com/wiki/19/how-to-use-a-single-form-to-collect-data-for-two-or-more-models/ and http://www.yiiframework.com/forum/index.php/topic/52109-save-data-with-two-models/, yet I still can’t get the data to save to both tables. I have two tables sales_rep_min_margin and sales_rep_min_margin_history:

CREATE TABLE sales_rep_min_margin (

id int(11) NOT NULL AUTO_INCREMENT,

username varchar(32) NOT NULL,

domestic int(2) NOT NULL,

overseas int(2) NOT NULL,

PRIMARY KEY (id)

) ENGINE=InnoDB DEFAULT CHARSET=utf8;

CREATE TABLE sales_rep_min_margin_history (

`id` int(11) NOT NULL AUTO_INCREMENT,


`min_margin_id` int(11) NOT NULL,


`from` int(11) DEFAULT NULL,


`to` int(11) DEFAULT NULL,


`update_username` varchar(32) NOT NULL,


`update_time` timestamp NOT NULL DEFAULT CURRENT_TIMESTAMP,


PRIMARY KEY (`id`),


KEY `min_margin_id` (`min_margin_id`),


CONSTRAINT `sales_rep_min_margin_history_ibfk_1` FOREIGN KEY (`min_margin_id`) REFERENCES `sales_rep_min_margin` (`id`)


) ENGINE=InnoDB DEFAULT CHARSET=utf8;

My SalesRepMinMarginController code (right now) is:

public function actionCreate() {

    $model = new SalesRepMinMargin;


    $model2 = new SalesRepMinMarginHistory;





    //Uncomment the following line if AJAX validation is needed


    //$this->performAjaxValidation($model);





    if (isset($_POST['SalesRepMinMargin'])) {





        $model->attributes = $_POST['SalesRepMinMargin'];





        if ($model->save())


            $this->redirect(array('view', 'id' => $model->id));


    }


    if (isset($_POST['SalesRepMinMarginHistory'])) {





        $model2->attributes = $_POST['SalesRepMinMarginHistory'];


        $model2->save();





        $this->render('create', array(


            'model' => $model,


        ));


    }


}

and ‘SalesRepMinMarginHistoryController’:

public function actionUpdate($id)

{

$model=$this->loadModel($id);





// Uncomment the following line if AJAX validation is needed


// $this->performAjaxValidation($model);





if(isset($_POST['SalesRepMinMarginHistory']))


{


    $model->attributes=$_POST['SalesRepMinMarginHistory'];





    if($model->save())


        $this->redirect(array('view','id'=>$model->id));


}





$this->render('update',array(


    'model'=>$model,


));

}

I just need to save the data to the tables, but I don’t need the ‘history’ table’s data in the view. Any help is greatly apreciated!

2

You code is pretty mixed up there. Try this:





    public function actionCreate() {

    

        $model = new SalesRepMinMargin;

        $model2 = new SalesRepMinMarginHistory;


        //Uncomment the following line if AJAX validation is needed

        //$this->performAjaxValidation($model);


        if (isset($_POST['SalesRepMinMargin'])) {


            $model->attributes = $_POST['SalesRepMinMargin'];


            if ($model->save()) {

                

                if (isset($_POST['SalesRepMinMarginHistory'])) {


                    $model2->attributes = $_POST['SalesRepMinMarginHistory'];

                    $model2->save();

                }

            

                $this->redirect(array('view', 'id' => $model->id));

            }

        }

        

        $this->render('create', array(

            'model' => $model, 'model2' => $model2,

            ));

        

    }
3

Thanks for the reply. Your code looks very nice, and I can follow the logic, however, it still won’t save to the second db. :sigh: On to the debugging again!

4

Maybe you’re running into validation errors? Try looking at what $model2->getErrors() returns after you try saving.

Also, if your model has any beforeValidate() and beforeSave() functions, make sure they return true.

5

True, check for validation errors. I encountered this several times and it turned out that my second model had some errors.

For the code, I think it would be better if you wrap it up on a db transaction so that you are not saving only for the first model but not the second model. or, you could do something like this:




public function actionCreate() {

    

        $model = new SalesRepMinMargin;

        $model2 = new SalesRepMinMarginHistory;


        //Uncomment the following line if AJAX validation is needed

        //$this->performAjaxValidation($model);


        if (isset($_POST['SalesRepMinMargin'])) {


            $model->attributes = $_POST['SalesRepMinMargin'];


            if ($model->validate()) {

                

                if (isset($_POST['SalesRepMinMarginHistory'])) {


                    $model2->attributes = $_POST['SalesRepMinMarginHistory'];

                    

                    if($model2->validate()) {

                        $model->save(false);

                        $model2->save(false);


                        $this->redirect(array('view', 'id' => $model->id));

                    }

                }

            }

        }

        

        $this->render('create', array(

            'model' => $model, 'model2' => $model2,

            ));

        

    }

By validating first before saving you would see the errors. Theoretically, if supported, db transaction is much better.

6

Thanks a lot!

7

Thank you very much.