ORM в Kohana 3.1?

Question

[Oreolek]

Схема довольно нетривиальная, и перевести её в ORM я не могу. Писать же через Query Builder не получится, так как придётся перелопачивать сотни строк с ORM объектами.







Поле Owner ссылается на таблицу users модуля Auth, если интересно.



Проблема в трёх полях таблицы events, которые ссылаются на PRIMARY KEY таблицы days. Как их описать? Постоянно выскакивает ошибка при попытке записи.



Вот модели без чёртовых полей:

class Model_Event extends ORM {<br/>
 protected $_table_columns = array(<br/>
 'id' =&gt; array('data_type' =&gt; 'int', 'is_nullable' =&gt; FALSE),<br/>
 'owner' =&gt; array('data_type' =&gt; 'int', 'is_nullable' =&gt; FALSE),<br/>
 'name' =&gt; array('data_type' =&gt; 'tinytext', 'is_nullable' =&gt; FALSE),<br/>
 'starting_time' =&gt; array('data_type' =&gt; 'time', 'is_nullable' =&gt; FALSE),<br/>
 'ending_time' =&gt; array('data_type' =&gt; 'time', 'is_nullable' =&gt; FALSE),<br/>
 'place' =&gt; array('data_type' =&gt; 'tinytext', 'is_nullable' =&gt; FALSE),<br/>
 'person_in_charge' =&gt; array('data_type' =&gt; 'tinytext'),<br/>
 'seats' =&gt; array('data_type' =&gt; 'int'),<br/>
 'info' =&gt; array('data_type' =&gt; 'mediumtext'),<br/>
 'program' =&gt; array('data_type' =&gt; 'mediumblob'),<br/>
 'verified' =&gt; array('data_type' =&gt; 'bool'),<br/>
 );<br/>
 protected $_has_many = array(<br/>
 'owner' =&gt; array(<br/>
 'model' =&gt; 'user',<br/>
 'foreign_key' =&gt; 'owner')<br/>
 );<br/>
};

class Model_Day extends ORM {<br/>
 protected $_table_columns = array(<br/>
 'day' =&gt; array('data_type' =&gt; 'date', 'is_nullable' =&gt; FALSE),<br/>
 'max_events' =&gt; array('data_type' =&gt; 'int', 'is_nullable' =&gt; FALSE, 'default' =&gt; 10),<br/>
 );<br/>
 protected $_primary_key = 'day';<br/>
};<br/>

Вот упрощённый код записи:

$event-&gt;values(Arr::extract($_POST, array('name','starting_time','ending_time','place','person_in_charge','seats','info'), NULL));<br/>
$event-&gt;owner = Auth::instance()-&gt;get_user()-&gt;id;<br/>
 $starting_day = new Model_Day;<br/>
 $ending_day = new Model_Day;<br/>
 $starting_day -&gt; day = Request::current()-&gt;post('starting_day');<br/>
 $ending_day -&gt; day = Request::current()-&gt;post('ending_day');<br/>
 $opened_until = new Model_Day;<br/>
 $opened_until -&gt; day = Request::current()-&gt;post('opened_until');<br/>
 if (!($starting_day-&gt;find())) $starting_day -&gt; create();<br/>
 if (!($ending_day-&gt;find())) $ending_day -&gt; create();<br/>
 if (!($opened_until-&gt;find())) $opened_until -&gt; create();<br/>
 $event-&gt;add('starting_day',$starting_day);<br/>
 $event-&gt;add('ending_day',$ending_day);<br/>
 $event-&gt;add('opened_until', $opened_until);<br/>
 $event-&gt;create();<br/>

Обращаю внимание на то, что версия Kohana — 3.1.



Пробовал описывать через belongs_to, has_many, соединял оба подхода — не работает. Либо ругается на FOREIGN KEY, либо на отсутсвующие поля starting_day и иже с ним, либо на пустое имя таблицы. Затык происходит именно на строке $event->add('starting_day',$starting_day);, что характерно.

Answer 1

[Иван]

Метод add() работает только для ManyToMany связей. Для односвязных полей (HasOne, BelongsTo) надо использовать что-то вроде $event->starting_day = $starting_day

Comments

[Oreolek]

Тогда он начинает ругаться на FOREIGN_KEY constraint fail потому что не вставляет:
Cannot add or update a child row: a foreign key constraint fails (`web-site`.`events`, CONSTRAINT `fk_events_2` FOREIGN KEY (`starting_day`) REFERENCES `days` (`day`) ON DELETE NO ACTION ON UPDATE NO ACTION) [ INSERT INTO `events` (`name`, `starting_time`, `ending_time`, `place`, `person_in_charge`, `seats`, `owner`, `starting_day`, `ending_day`, `opened_until`) VALUES ('Трям', '11:00:00', '14:00:00', 'Неивестно', 'А. Н. Оним 12-34-56', '2', '4', '', '', '') ]

В коде вот что:
if (!($starting_day->find())) $starting_day -> create(); //чтобы $starting_day имел реальный ID
$event->starting_day = $starting_day;

Если же подавать $event->starting_day = $starting_day->day, то starting_day вообще не попадает в запрос INSERT (это поле попросту пропускается), что тоже противоречит FOREIGN_KEY.

[Иван]

Там же в ошибке все написано! Пустой внешний ключ `starting_day` видите? ORM воспринимает ID как autoincrement, и после создания новой записи пытается запихать в ID последний сгенерированный ИД. Сделайте для таблицы days нормальный автоинкрементный идентификатор и ссылайтесь на него.

[Oreolek]

То есть, всё-таки ID должен быть INT и auto_increment? На #kohana мне сказали, что это вообще не проблема.

[NeX]

id создавать не обязательно. Oreolek, вы поле $_primary_key у Day заполнили?

Если же подавать $event->starting_day = $starting_day->day, то starting_day вообще не попадает в запрос INSERT (это поле попросту пропускается)

Этого быть не должно
Как-то так:
$starting_day->day = "NEW_DATE_VALUE";
$starting_day->save();
$event->starting_day = $starting_day->day;


[Иван]

Ну, тут может и не будет проблем — мы передаем конкретное значение ключа. Но, в целом, я бы все равно добавил автоинкремент. Не уверен, что ORM способен полноценно работать без автоинкрементных ID, надо проверять

[Oreolek]

Пока что проблема в другом — как убедиться, что запись в таблице day существует? У меня здесь тоже была ошибка. В случае ID это относительно просто:
if ($starting_day->id == 0) $starting_day -> create(); else $starting_day -> update();
Кстати, save() в 3.1 убрали.
Отпишусь позже, когда решу эту проблемку. Спасибо за советы!

[Иван]

Не надо сравнивать ID с нулем. Вообще-то должно быть что-то вроде:
$starting_day = ORM::factory('day', $day);
if ( ! $starting_day->loaded()) { // запись не найдена}

А save() не убирали, там должна идти переадресация на create() или update().

[Oreolek]

Поправка: save() есть, но офиц. дока намекает на то, что уже в 3.2 его может не остаться.
Вот что получилось, если интересно:

$starting_day = new Model_Day(Request::current()->post('starting_day'));
if ($starting_day->day == '0000-00-00') $starting_day -> create(); else $starting_day -> update();
$ending_day = new Model_Day(Request::current()->post('ending_day'));
if ($ending_day->day == '0000-00-00') $ending_day -> create(); else $ending_day -> update();
$opened_until = new Model_Day(Request::current()->post('opened_until'));
if (Request::current()->post('opened_until') == '') $opened_until = new Model_Day(date("Y-m-d", strtotime(Request::current()->post('starting_day'))-10080));
$opened_until -> save();
$event->starting_day = $starting_day;
$event->ending_day = $ending_day;
$event->opened_until = $opened_until;
$event->create();
Что интересно: если для $opened_until тоже ввести проверку на существование, то он выдаёт либо cannot update: model not loaded, либо DUPLICATE PRIMARY KEY (т.е. 0000-00-00, такой ключ остался от предыдущих прогонов). Пока save() работает, попробую так.

Спасибо!

[Иван]

Совет: используйте один и тот же ранее созданный экземпляр модели. Не надо создавать их несколько раз (особенно если в цикле), есть ведь метод reset() для сброса состояния. Сейчас это может показаться мелочью, но потом может сказаться на скорости работы.

Answer 2

[NeX]

public function add($alias, $far_keys)
указывается имя АЛИАСА а не поле. Покажите код моделей (интересуют belongs_to И has_many) day и event

Comments

[Oreolek]

Код моделей, собственно, у меня и не работает. Я его менял много раз… в общем, всё сводится либо к следующему в Event:

protected $_has_many = array(
'starting_day' => array(
'model' => 'day',
'foreign_key' => 'starting_day'),
'ending_day' => array(
'model' => 'day',
'foreign_key' => 'ending_day'),
'opened_until' => array(
'model' => 'day',
'foreign_key' => 'opened_until')
);

либо к следующему в Day:

protected $_belongs_to = array(
'starting_day' => array('model' => 'event', 'far_key' => 'starting_day'),
'ending_day' => array('model' => 'event', 'far_key' => 'ending_day'),
'opened_until' => array('model' => 'event', 'far_key' => 'opened_until'),
);

[NeX]

Связь 1 day — ко многим event
Соответственно, должны быть заполнены $_has_many у Day и $_belongs_to у Event. Названия связей не должны совпадать с именами полей в таблице.

[NeX]

brotkin.ru/2011/02/21/orm-in-kohana-v31/

[NeX]

brotkin.ru/2009/09/21/ko3-orm-izuchaem-zanovo/

[Oreolek]

Не работает.
Переписал связь с User так же. Вот что получилось в Event:

protected $belongs_to = array (
'creator' => array('model' => 'user','foreign_key' => 'owner'),
'start' => array( 'model' => 'day', 'foreign_key' => 'starting_day'),
'end' => array( 'model' => 'day', 'foreign_key' => 'ending_day'),
'closing' => array('model' => 'day', 'foreign_key' => 'opened_until'),
);

А вот что в User:

'creator' => array('model' => 'event','foreign_key' => 'owner'),

На строчке $event->add('creator', ORM::factory('user',Auth::instance()->get_user()->id)); выдаёт Undefined index для creator.

Поля в Event прописал все, включая owner.

Вроде всё правильно. В чём может быть ошибка?

[NeX]

см. ниже