Comments 10
Позволю себе парочку замечаний, если не возражаете :)
1. Не по фэн-шую добавляете сторонние классы — обычно их кидают в папку vendor (в корень модуля) и далее добавляют через Kohana::find_file().
2. Вы то используете глобальные массивы напрямую ($_GET['oauth_token'] например, даже нет проверки на существование такого ключа), то переходите на «родные» для Ko3 классы, типа Cookie::get().
3. Многовато кода в контроллере. Возможно, имеет смысл создать специальные методы в библиотеке (например, получение данных о пользователе), в которые и спрятать всю логику работы Вашего Twitter-модуля.
4. Настройки подключения к Twitter лучше сохранять в конфиге (это в стиле Kohana) или в БД.
Да, и перенесите в тематический блог, пожалуйста :)
1. Не по фэн-шую добавляете сторонние классы — обычно их кидают в папку vendor (в корень модуля) и далее добавляют через Kohana::find_file().
2. Вы то используете глобальные массивы напрямую ($_GET['oauth_token'] например, даже нет проверки на существование такого ключа), то переходите на «родные» для Ko3 классы, типа Cookie::get().
3. Многовато кода в контроллере. Возможно, имеет смысл создать специальные методы в библиотеке (например, получение данных о пользователе), в которые и спрятать всю логику работы Вашего Twitter-модуля.
4. Настройки подключения к Twitter лучше сохранять в конфиге (это в стиле Kohana) или в БД.
Да, и перенесите в тематический блог, пожалуйста :)
+7
Посмотрел, как это у Вас в блоге выглядит и впал в ступор. Получается, что я (посетитель блога, не владелец) должен кликнуть по кнопке Sign in? Я предполагал, что основная цель — импорт на сайт АВТОРСКИХ твитов…
+1
Ну, располагая методом arr::get(), грех им не пользоваться :)
+2
блин, реально… Ты молодец хорошо изучил ф-ционал. Я делаю тоже самое только isset-ом Спасибо за наводку, буду юзать методы Коханы.
ЗЫ В своё оправдание, скажу, что все линки и картинки пропускаю церез хелпер ХТМЛ. Я отнаследовался и немного переделал метод добавления тега картинок, теперь если нет физически картинок, мне в лог пишется запись об этом. Ладно это уже не входит в рамки данной статьи, ну будем холиварить
ЗЫ В своё оправдание, скажу, что все линки и картинки пропускаю церез хелпер ХТМЛ. Я отнаследовался и немного переделал метод добавления тега картинок, теперь если нет физически картинок, мне в лог пишется запись об этом. Ладно это уже не входит в рамки данной статьи, ну будем холиварить
-3
Переделайте как советует biakaveron и на гитхаб. а там уже коммюнити сделает по феншую. На следующей неделе форкну, если закинете ;)
+1
Sign up to leave a comment.
Интеграция OAuth и Twitter API в фреймворк Kohana3