Изменение данных модели перед сохранением поведения при использовании saveAll

Я пытаюсь написать мета-поведение для проекта, над которым я работаю, что позволит мне назначать настраиваемые переменные/атрибуты модели, аналогично тому, как вы можете создавать настраиваемые поля в сообщении WordPress.

Я создал поведение Meta, которое связывает метамодель с моделью, на которую она воздействует, а также имеет обратный вызов beforeSave, который перебирает переменную данных модели и помещает имя модели в метамассив.

Все сохраняется, но когда я проверяю базу данных, поле модели возвращается пустым.

Структура базы данных для мета:

id - A unique if for the meta
model - The name of the model that this meta entry is associated with
foreign_id - The id of the above model that this meta entry is associated with
key - Name of the meta variable
value - Value of the meta variable

Данные, которые поступают в функцию saveAll из формы,

Array
(
    [Page] => Array
        (
            [id] => 12
            [name] => Test
            [slug] => a/b/c/d
            [layout] => 0
            [body] => Test multilevel
        )

    [Meta] => Array
        (
            [0] => Array
                (
                    [id] => 1
                    [key] => page_title
                    [value] => About Us
                )

            [1] => Array
                (
                    [id] => 6
                    [key] => test4
                    [value] => test
                )

            [2] => Array
                (
                    [key] => test3
                    [value] => lala
                )

        )

)

и после того, как он выполнил поведение перед сохранением, это

Array
(
    [Page] => Array
        (
            [id] => 12
            [name] => Test
            [slug] => a/b/c/d
            [layout] => 0
            [body] => Test multilevel
            [modified] => 2010-05-04 15:56:54
        )

    [Meta] => Array
        (
            [0] => Array
                (
                    [id] => 1
                    [key] => page_title
                    [value] => About Us
                    [model] => Page
                )

            [1] => Array
                (
                    [id] => 6
                    [key] => test4
                    [value] => test
                    [model] => Page
                )

            [2] => Array
                (
                    [key] => test3
                    [value] => lala
                    [model] => Page
                )

        )

)

Код поведения

<?php
/**
 * Meta Model Behavior
 * 
 * Adds custom variables to models
 *
 **/
class MetaBehavior extends ModelBehavior {

/**
 * Contains configuration settings for use with individual model objects.
 * Individual model settings should be stored as an associative array, 
 * keyed off of the model name.
 *
 * @var array
 * @access public
 * @see Model::$alias
 */
    var $__settings = array();

    var $__defaults = array(
        'class' => 'Meta',
        'foreign_key' => 'foreign_id',
        'dependent' => true,
        'auto_bind' => true
    );

/**
 * Initiate Meta Behavior
 *
 * @param object $model
 * @param array $config
 * @return void
 * @access public
 */
    function setup(&$model, $settings = array()) {
        $default = $this->__defaults;
        $default['conditions'] = array('Meta.model' => $model->alias);

         if (!isset($this->__settings[$model->alias])) {
            $this->__settings[$model->alias] = $default;
        }

        $this->__settings[$model->alias] = array_merge($this->__settings[$model->alias], ife(is_array($settings), $settings, array()));

        if ($this->__settings[$model->alias]['auto_bind']) {
            $hasManyMeta = array(
                'Meta' => array(
                    'className' => $this->__settings[$model->alias]['class'],
                    'foreignKey' => $this->__settings[$model->alias]['foreign_key'],
                    'dependent' => $this->__settings[$model->alias]['dependent'],
                    'conditions' => $this->__settings[$model->alias]['conditions']
                )
            );
            $metaBelongsTo = array(
                $model->alias => array(
                    'className' => $model->alias,
                    'foreignKey' => $this->__settings[$model->alias]['foreign_key']
                )
            );
            $model->bindModel(array('hasMany' => $hasManyMeta), false);
            $model->Meta->bindModel(array('belongsTo' => $metaBelongsTo), false);
        }
    }

    function beforeSave(&$model) {
        foreach($model->data[$this->__settings['class']] as $key => $value) {
            $model->data[$this->__settings['class']][$key]['model'] = $model->alias;
        }
        return true;
    }

} // End of MetaBehavior

?>

У меня такое чувство, что это может быть из-за отношений, и saveall использует переданные данные (оригинал) при сохранении ассоциаций.

Единственным другим способом, который я придумал, было бы удалить отношения и поместить некоторый код в функцию afterSave поведения для обработки сохранения, а затем добавить какой-то другой код в afterFind для их извлечения.

Любые идеи?

Привет, Дин


person Dean    schedule 04.05.2010    source источник
comment
Почему бы вам просто не связать Meta с вашей моделью страницы через обычную ассоциацию HasMany? Вам нужно связать Meta с другими моделями?   -  person Darren Newton    schedule 03.02.2011


Ответы (1)


Я не верю, что вам повезет с этим, поскольку Model::saveAll() вообще не вызывает beforeSave(). Фактически, он загружает ассоциации перед вызовом $this->save().

В строке 1652 источника модели показаны ассоциации, загружаемые до любого реального вызова. в Model::__save(), который не просто проверяет данные.

Это означает, что на первый взгляд магический вызов bindModel() в вашем MetaBehavior не окажет никакого влияния. Честно говоря, я бы не стал заморачиваться с чем-то подобным, поскольку проще установить ассоциацию в определении класса модели страницы.

Конечно, вы могли бы:

  • Создайте afterSave в вашем MetaBehavior, который облегчает сохранение данных метамодели или,
  • Переопределите saveAll() в модели, чтобы добавить привязку, а затем вызовите parent::saveAll()

В конечном счете, я считаю более разумным использовать Model::saveAll() и в любом случае установить реальную ассоциацию в определении класса Model. Использование поведения для продвижения СУХИХ практик очень благородно, но это не означает, что вам нужно делать вещи менее тривиальными в рамках методологии разработки, которой следуете вы или другой человек.

TLDR; Вы уже добавили Meta в свой массив поведений, добавление ассоциации — это всего лишь еще несколько строк кода в том же файле.

person connrs    schedule 18.02.2011