Свое правило проверки

[bemulima]

Здравтвуйте, сделал свое правило проверки, но не работает:

Код: Выделить всё

public function ValidatorRelated()
    {

        $result = PostOptionRelated::model()->find(array(
            'condition' => 'page_id = :page_id AND category_id='. $_POST['category_id'],
            'params'    => array(':page_id' => $_POST['page_id']),
        ));
        if($result->option_id == $_POST['option_id'])
        $this->addError('page_id','Такая связь с категорией уже сушествует!');
        
    }

в рулес написал:

Код: Выделить всё

array('page_id, category_id, option_id', 'ValidatorRelated'),

Что не так, подскажите?

[notgosu]

замените $_POST['column'] на $this->column

[bemulima]

Странно, сработала, я думал с $_POST будет работать

[masson]

1. Ваш валидатор вызовется 3 раза (для 3-х полей). Зачем вам это надо? Сделайте руль для одного поля.
2. Вы почему-то page_id привязываете через params, а category_id втыкаете в condition напрямик. Наверное есть смысл сделать одинаково

[Neuromance]

я бы еще добавил, что надо стараться избегать в моделях различных $_POST, $_GET и т.д. Пусть этим всем рулят контроллеры, а модели работают со своими полями.
И я что-то не очень понял, валидатор типа для определения уникальности чтоли? Зачем тогда изобретать велосипед?

[bemulima]

И я что-то не очень понял, валидатор типа для определения уникальности чтоли? Зачем тогда изобретать велосипед?

Тут я кусок кода вложил, на самом деле код правило так выглядит:

Код: Выделить всё

public function ValidatorRelated()
    {

        if(!empty($this->page_id) && !empty($this->category_id)){
        $result = self::model()->find(array(
            'condition' => 'page_id = :page_id AND category_id='. $this->category_id,
            'params'    => array(':page_id' => $this->page_id),
        ));
        if($result->option_id == $this->option_id)
        $this->addError('page_id','Такая связь с категорией уже сушествует!');
        }elseif(!empty($this->page_id) && empty($this->category_id)){
        $result = self::model()->find(array(
            'condition' => 'page_id = :page_id',
            'params'    => array(':page_id' => $this->page_id),
        ));
        if($result->option_id == $this->option_id)
        $this->addError('page_id','Такая связь со страницей уже сушествует!');
        }
        
    } 

во общем я тут проверяю в таблице нет ли одинаковые связи, у меня есть много различные опции, их можно связать к страницам или к категориям, и вот я тут проверяю не связан ли данные уже раньше или нет: . а таблица в таком порядке | id | page_id | category_id | option_id |,

[bemulima]

1. Ваш валидатор вызовется 3 раза (для 3-х полей). Зачем вам это надо? Сделайте руль для одного поля.

Да потому что мне всех надо проверять)

Код: Выделить всё

2. Вы почему-то page_id привязываете через params, а  category_id втыкаете в condition напрямик. Наверное есть смысл сделать одинаково :) 

а я думал дополнительные можно писать напрямую т.е. не через params

[masson]

Прощще надо быть

Код: Выделить всё

if(!empty($this->page_id)) {
    $condition= 'page_id = :page_id AND option_id = :option_id';
    $params = array(':page_id' => $this->page_id, ':option_id' => $this->option_id),

    if(!empty($this->category_id)) {
        $condition .= ' AND category_id= :category_id';
        $params[':category_id'] = $this->category_id;
    }

    if(self::model()->exists($condition,$params)
        $this->addError('page_id', empty($this->category_id) ? 'Связь со страницей уже существует' : 'Связь с категорией уже существует!');
} 

Да потому что мне всех надо проверять)

Так они за один заход все проверяются
array('page_id', 'ValidatorRelated'),

а я думал дополнительные можно писать напрямую т.е. не через params

Все можно писать напрямую, если уверен в безопасности содержимого.

[bemulima]

Прощще надо быть

Круто! Зато мой ошибку дает если со страницей связь существует

[masson]

про текст ошибки? Уже поправил

[bemulima]

Прощще надо быть

Но я ваш код возьму Мне нравится потому что он, как нить я туда прикручу ошибку связь со страницей

[masson]

Исправил уже код, все там есть

[bemulima]

про текст ошибки? Уже поправил

Круто!!! Теперь я так всегда буду делать)

[bemulima]

Исправил уже код, все там есть

Спасибо!