Kohana 3 и модули?

Question

[igamity]

Добрый день!


Начинаю изучать kohan-у на примере, написанном другим программистом. Часть сайта, написанная им, работает нормально и в ней все понятно, могу вносить свои изменения/улучшения. Однако, при попытке добавить свой функционал столкнулся со следующей проблемой.


Пробую совместить новый функционал с существующим на сайте «движком» авторизации. Создан контроллер «class Controller_Auth extends Controller» и соответствующая модель «class Model_Auth extends Kohana_Model». Если контроллер вызывается из основного приложения, для проверки прав посетителя, все работает нормально. Но при попытке использовать данный контроллер напрямую, собственно для авторизации, наступаю на грабли — не загружается модуль Database и соответственно все попытки обратится к базе вызывают ошибку.


Для основного приложения указан роутинг:

Route::set('forum', 'forum(/(/page))')-&gt;defaults(array('controller' =&gt; 'forum', 'id' =&gt; 1, 'page' =&gt; 1));<br/>
<br/>
<br/>
В соответствие с этим был добавлен новый роут:<br/>
<code>Route::set('auth', 'auth(/)')-&gt;defaults(array('controller' =&gt; 'auth', 'action' =&gt; 'login'));<br/>
<br/>
<br/>
Форма отправляется по URL: &quot;/auth/login&quot; и POST-ом отправляются данные. Контроллер и соответствующая модель работают, но возникает ошибка при обращении к БД. При этом в списке загруженных модулей, который kohana показывает на странице с ошибкой, видно, что модуль database не загружен.<br/>
<br/>
Посоветуйте, где найти эти чертовы грабли?</code>

Comments

[igamity]

Kohana::modules(array(
   'database'  => MODPATH.'database',  // Database access
  ));

Route::set('forum', 'forum(/<id>(/page<page>))')->defaults(array('controller' => 'forum', 'id' => 1, 'page' => 1));
Route::set('auth', 'auth(/<action>)')->defaults(array('controller' => 'auth', 'action' => 'login'));

Route::set('default', '(<controller>(/<action>(/<id>)))', array('id' => '\d{1,10}'))
  ->defaults(array(
    'controller' => 'welcome',
    'action'   => 'index',
    'id'     => 1
  ));

* This source code was highlighted with Source Code Highlighter.

[WebSpider]

Не могли бы вы уточнить что значит «контроллер вызывается из основного приложения» и «использовать данный контроллер напрямую»?
Кстати, модель лучше наследовать от Model, но это так, мелочи

[igamity]

Т. к. я не большой знаток, могу сказать только что по модели HMVC контроллер может вызываться не только напрямую, но и из другого контроллера. Я про это. Т. е. из контроллера основного приложения я вызываю «Request::factory('auth/checkAuth')->execute()->response;» и это работает, т. е. есть предположение, что с самим контроллером авторизации все в порядке.

На счет наследования, спасибо, не знал, но как я описал в начале — делал по аналогии.

[igamity]

Да, под вызовом контроллера напрямую я подразумеваю его использование… хм… по-умолчанию, когда путь к нему задан в URL. Прошу простить, если в терминологии напутал.

[WebSpider]

Приведите, пожалуйста, текст ошибки

[igamity]

ErrorException [ Fatal Error ]: Call to a member function query() on a non-object
на строчке: $u = $this->_db->query(Database :: SELECT, «select * from Users limit 1», false)->as_array();
Сам по себе запрос тестовый.

Answer 1

[WebSpider]

Похоже вы в конструкторе модели забыли написать parent::__construct(), если переопределяли его.
Какая версия фреймворка, кстати? 3.0 или 3.1? Если 3.1, то модель надо наследовать от класса Model_Database.

P.S. $this->_db надеюсь из модели вызывается? Что показывает var_dump($this->_db)? А вообще следует делать запросы через DB::query(...)

Comments

[igamity]

Точно, переопределял конструктор. Спасибо огромное за помощь!

На счет метода запросов к БД, я пока еще не решил для себя, что лучше. Для INSERT, однозначно через билдер, а для селектов… В любом случае спасибо за науку, прислушаюсь к вашим советам.

[WebSpider]

Не за что

Для INSERT, однозначно через билдер, а для селектов…

В данном случае речь идёт не о билдере, а об обёртке вокруг класса Database. Т.е. вы можете написать так:

DB::query(Database :: SELECT, "select * from Users limit 1")->execute()->as_array();

Это будет полностью аналогично вашему примеру. Кроме того этот способ поддерживает именованные параметры и т.д.:

DB::query(Database :: SELECT, "select * from Users where id = :id limit 1")
  ->param(':id', 1)
  ->execute()
  ->as_array();

[igamity]

В таком случае логичный вопрос, а почему так? ;) В смысле, почему DB::query предпочтительнее $this->_db->query? Видимо разница в том, что DB — независимый класс и вызывается он напрямую, а $this->_db через механизм наследования. Умозрительно представляю разницу, но описать нормально не могу.