Comments 9
$this->user_roles = Model::factory('User')->user_roles($this->user->id);
а разве нельзя
$this->user_roles = $this->user->roles;
?
isset($config_security[$this->class_id][$action])
тоже слабенько
В конфигах необходимо постоянно писать этот бойлерплейт «Controller_ *», да и вообще создается впечатление, что с фреймворком недавно работаете,
foreach($this->configs as $config)
{
$this->template->set_global($config, Kohana::$config->load($config)->as_array());
}
Странное решение имхо.
если интересно узнать все огрехи — стучите в личку, если нет — сэкономим друг другу время)
UPD
не обижайтесь, но весь метод user_roles() это просто свалка говнокода.
UPD2
Ну и ценность статьи на мой взгляд ничтожна
а разве нельзя
$this->user_roles = $this->user->roles;
?
isset($config_security[$this->class_id][$action])
тоже слабенько
В конфигах необходимо постоянно писать этот бойлерплейт «Controller_ *», да и вообще создается впечатление, что с фреймворком недавно работаете,
foreach($this->configs as $config)
{
$this->template->set_global($config, Kohana::$config->load($config)->as_array());
}
Странное решение имхо.
если интересно узнать все огрехи — стучите в личку, если нет — сэкономим друг другу время)
UPD
не обижайтесь, но весь метод user_roles() это просто свалка говнокода.
UPD2
Ну и ценность статьи на мой взгляд ничтожна
В конфигах необходимо постоянно писать этот бойлерплейт «Controller_ *», да и вообще создается впечатление, что с фреймворком недавно работаете,
В конфигах питшется полный путь до контроллера, если у Вас контроллер будет в директории admin, то писать например так Controller_Admin_Index.
не обижайтесь, но весь метод user_roles() это просто свалка говнокода.
Почитайте документацию на офф сайте, по вашему мнению там тоже говонокод в примерах моделий?
>В конфигах питшется полный путь до контроллера, если у Вас контроллер будет в директории admin, то писать например так Controller_Admin_Index
Вы видимо читали невнимательно, зачем везде писать «Controller_», если он есть у каждого элемента 1-го уровня?
>Почитайте документацию на офф сайте, по вашему мнению там тоже говонокод в примерах моделий?
При чем тут документация? Или Вы этот метод оттуда взяли? Если да — потрудитесь ссылочку указать. У Вас тут куча ненужного кода и Вы явно не понимаете, как работает ORM и как к ней нужно писать методы.
Самый простой способ забрать роли — $user->roles->find_all(). Если уж так хочется свой метод и «переоформить» вывод
public function user_roles(){
return $roles = $this->roles->find_all()->as_array(NULL,'name');
}
Легко заметить, что так же исчез ненужный параметр и из 10-ти строк осталось 1. Если результат один и тот же результат в 10 раза больше кода занимает — что это, если не говнокод?
И да, реагируйте пожалуйста адекватно
Вы видимо читали невнимательно, зачем везде писать «Controller_», если он есть у каждого элемента 1-го уровня?
>Почитайте документацию на офф сайте, по вашему мнению там тоже говонокод в примерах моделий?
При чем тут документация? Или Вы этот метод оттуда взяли? Если да — потрудитесь ссылочку указать. У Вас тут куча ненужного кода и Вы явно не понимаете, как работает ORM и как к ней нужно писать методы.
Самый простой способ забрать роли — $user->roles->find_all(). Если уж так хочется свой метод и «переоформить» вывод
public function user_roles(){
return $roles = $this->roles->find_all()->as_array(NULL,'name');
}
Легко заметить, что так же исчез ненужный параметр и из 10-ти строк осталось 1. Если результат один и тот же результат в 10 раза больше кода занимает — что это, если не говнокод?
И да, реагируйте пожалуйста адекватно
Чем не подошёл ACL-модуль для данного фреймворка?
forum.kohanaframework.org/discussion/5274/a2-kohanas-acl-module/p1
Там вполне удобно можно создать роли и прописать доступ к ресурсам. По сути — это порт Zend ACL модуля для разграничения доступа.
forum.kohanaframework.org/discussion/5274/a2-kohanas-acl-module/p1
Там вполне удобно можно создать роли и прописать доступ к ресурсам. По сути — это порт Zend ACL модуля для разграничения доступа.
А мы просто прикрутили Zend_Acl — прекрасный компонент.
user_roles заменяется одной строкой
$roles = $auth->get_user()->roles->find_all();
$roles = $auth->get_user()->roles->find_all();
Sign up to leave a comment.
Разграничение прав доступа Kohana 3.3