changing object in loop in Datamapper ORM to save

[eluser]mak.gnu[/eluser]
I'm trying to save a long form in Codeigniter's Datamapper. I'm able to save the form if I pass the value like this

Code:

$t->brandName = $this->input->post('brandName');  
$t->specialNotes = $this->input->post('specialNotes');
$t->name = $this->input->post('name')

;

Now if I call the save method it works fine.

$t->save();

Since the form is big I tried to add object values in foreach

Code:

$a = get_object_vars($t);
foreach ($a['stored'] as $k => $val){
      $t->$k = $this->input->post("$k");
}

however if I call the $t->save() it doesn't work.
Any input into this.

[eluser]toopay[/eluser]
Maybe there is something wrong in foreach statement. How your forms looks like?

[eluser]mak.gnu[/eluser]
My form is simple with many fields in it. the name of each field is same as column name. it works when I do

foreach($_POST as $k=>$val){
$t->$k = $this->input->post($k);
}

Can't I just iterate through the object ?

[eluser]WanWizard[/eluser]
For starters, not every Datamapper object property is a table column, so using get_object_vars() doesn't work. If you need a list of table columns, use the $object->fields array.

And second, a statement like "save() doesn't work" isn't really informative. What doesn't work? Any error messages? What have you tried to debug it?

[eluser]mak.gnu[/eluser]
thanks for reply

Code:

function save() {
        
        $t = new Truck();
        foreach ($t->fields as $k => $val){
            $t->$val = $this->input->post("$val");
        }
        $t->save();
        echo $this->db->last_query();
    }

this displays

Code:

SELECT * FROM `trucks` LIMIT 1

and also its not saving the data.
If I use $_POST instead of $t->fields it saves and also gives insert query

[eluser]WanWizard[/eluser]
Try to use code blocks, that would make your posts much cleaner. Also, you're answer doesn't answer any of my questions.

Check for errors first

Code:

if ( ! $t->save() )
{
    // or we can loop through the error's all list
    foreach ($t->error->all as $error)
    {
        echo $error;
   }
}

If you say that using $_POST['field'] works, but $this->input->post("$val") doesn't, wouldn't it be obvious that that's where the problem is, and not in the Datamapper code?

For starters, try $this->input->post($var), because I haven't got a clue why you would use double quotes there. Also, you should check if that actually returns a value. If it doesn't exist, the post() method returns false, and setting an 'id' to false is not something Datamapper likes.

[eluser]mak.gnu[/eluser]
When I run the following code it gives error stating

Code:

The id field must contain an integer.

Code:

$t = new Truck();
        foreach ($t->fields as $k => $val){
            $t->$val = $this->input->post("$val");
        }
        if (!$t->save()) {
            // or we can loop through the error's all list
            foreach ($t->error->all as $error) {
                echo $error;
            }
        }

And whenI run following code it works like a charm. and saves the data.

Code:

$t = new Truck();
        foreach ($_POST as $k => $val){
            $t->$k = $this->input->post($k);
        }
        if (!$t->save()) {
            // or we can loop through the error's all list
            foreach ($t->error->all as $error) {
                echo $error;
            }
        }

What is wrong in using $t->fields

[eluser]WanWizard[/eluser]
There is nothing wrong with the Datamapper code, nor with the fields array.
The error is in your code and logic. This error suggests you have a form field called 'id'. You can not set a Datamapper id manually. Every Datamapper model validates the 'id' field before attempting a save operation.

Learn to do some debugging before you ask for help. If you had done that here, you would have spotted the mistake.

[eluser]mak.gnu[/eluser]
Thanks for suggestion will do that in future