Изменение данных модели в beforeSave of Behavior при использовании saveAll - PullRequest
3 голосов
/ 04 мая 2010

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

Я создал мета-поведение, которое связывает метамодель с моделью, на которую она действует, а также имеет обратный вызов 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
                )

        )

)

и после того, как он прошел через поведение beforeSave, это

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 для их получения.

Есть идеи?

Cheers, Декан

1 Ответ

0 голосов
/ 19 февраля 2011

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

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

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

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

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

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

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

...