Support passing extra fields to update_attributes

[ext]

Problem

Given a class such as this:

class MyModel extends BasicObject {
  public function __get($key){
    switch($key){
      case 'bar':
        return $this->foo - 12;
      default:
        return parent::__get($key);
      }
  }

  public function __set($key, $value){
    switch($key){
      case 'bar':
        $this->foo = $value + 12;
        break;
      default:
        return parent::__get($key);
  }
}

It would be tempting (and useful) to pass bar directly to update_attributes:

$data = ['bar' => 1234, ...];
$model = MyModel::update_attributes($data);

Since BO only allows setting values existing in database columns other values are silently ignored (which is probably a good thing).

Workaround

$data = [...];
$model = MyModel::update_attributes($data, ['commit' => false]);
$model->foo = $data['bar']; // ignoring error-checking, e.g. array_key_exists
$model->commit();

Solution

$data = ['bar' => 1234, ...];
$model = MyModel::update_attributes($data, ['extra_fields' => 'bar']);

Questions

• Should extra_fields populate permit array too? As of now one must set both both when using permit.

[ext]

Updated with BasicObject::extra_fields() for specifying allowed extra fields, permit works as expected.