Current populate() does not account for setting property value back to null

[crazedVic]

I know you probably don't use this code any more but just in case you do. Found an interesting bug.

public function populate($params)
    {
        $columns = self::getColumns();
        foreach ($columns as $column) {
            if (isset($params[$column['Field']])) {
                $c = $column['Field'];
                $this->$c = $params[$column['Field']];
            }
        }
        return $this;
    }

i changed it to this, i was trying to set date field back to null and it didn't store the change, this fixed it:

public function populate($params)
    {
        $columns = self::getColumns();

        foreach ($columns as $column) {
            if (array_key_exists($column['Field'], $params)) {
                $c = $column['Field'];
                $this->$c = $params[$column['Field']];
            }
        }
        return $this;
    }