Русскоязычный форум, посвященный фреймворку Kohana

Все о фреймворке Kohana. Обсуждение уроков, документации.
Текущее время: 28 мар 2024, 15:54

Часовой пояс: UTC + 4 часа [ Летнее время ]




Начать новую тему Ответить на тему  [ Сообщений: 19 ]  На страницу 1, 2  След.
Автор Сообщение
 Заголовок сообщения: Log Viewer
СообщениеДобавлено: 22 апр 2012, 22:49 
Не в сети
Бывалый

Зарегистрирован: 18 фев 2012, 12:10
Сообщения: 57
Откуда: Украина, Киев
Хороший модуль для просмотра логов
Работает только на php 5.3+
https://github.com/ajaxray/Kohana-Log-Viewer


Вернуться к началу
 Профиль  
 
 Заголовок сообщения: Re: Log Viewer
СообщениеДобавлено: 23 апр 2012, 00:05 
Не в сети
Администратор
Аватара пользователя

Зарегистрирован: 12 фев 2012, 01:02
Сообщения: 462
Нужная вещь. Спасибо.

_________________
kohanaframework.su - обучение фреймворку Kohana


Вернуться к началу
 Профиль  
 
 Заголовок сообщения: Re: Log Viewer
СообщениеДобавлено: 13 июн 2013, 14:17 
Не в сети
Новичок

Зарегистрирован: 07 июн 2013, 18:07
Сообщения: 24
Откуда: Питер
Не будет работать в Kohana 3.3
Из замеченных косяков:
неправильно названные файлы в папке logviewer/classes/controller/log.php (log c маленькой буквы)
используются устаревшие свойства и методы, например Kohana::$is_cli, Request::redirect()

PS
Решил немного поковырять модуль.
Любопытная особенность в файле инициализации модуля (init.php) есть таки строки

Код:
Route::set('logviewer', 'logs/(<year>(/<month>(/<day>(/<level>))))')
   ->defaults(array(
      'controller' => 'logs',
      'action'     => 'index',
   ));


Казалось бы должен работать адрес вида:
http://localhost/logs/
Но не тут то было...
Кохана бросает исключение Kohana_HTTP_Exception [ 404 ]: Unable to find a route to match the URI: logs

Меняем маршрут на
Код:
Route::set('logviewer', 'logs(/<year>(/<month>(/<day>(/<level>))))')
   ->defaults(array(
      'controller' => 'logs',
      'action'     => 'index',
   ));

т.е. переносим слэш в круглые скобки...
адрес http://localhost/logs/ (и даже http://localhost/logs) начинают работать...


Вернуться к началу
 Профиль  
 
 Заголовок сообщения: Re: Log Viewer
СообщениеДобавлено: 13 июн 2013, 21:50 
Не в сети
Администратор
Аватара пользователя

Зарегистрирован: 24 июл 2012, 18:00
Сообщения: 701
Откуда: Murom, Russia
Молодец :) Исправил ошибку. А переделать дальше модуль под Ко3.3?


Вернуться к началу
 Профиль  
 
 Заголовок сообщения: Re: Log Viewer
СообщениеДобавлено: 14 июн 2013, 17:22 
Не в сети
Новичок

Зарегистрирован: 07 июн 2013, 18:07
Сообщения: 24
Откуда: Питер
Мне модуль не нужен, переписывать не буду. Смотрю его только чтоб приобщиться так сказать ...
Ничего глобального менять не пришлось, чтоб он нормально заработал уже через 15 минут после того как скачал. Все очевидно

Кто будет использовать советую обратить внимание на следующие моменты:

1) Элементы в верхнем меню выглядят так <a href="/logs/2012\10/01/?mode=raw">2012\10</a>
т.е. один слэш смотрит не в ту сторону. У меня хром такие ссылки воспринимает нормально, в других браузерах не смотрел...

2) По мне так неправильно, что мы обращаемся во Вьюшках к массиву $_GET. При этом не проверяем что там содержится, а сразу выводим. На мой взгляд все данные из глобальных массивов должны проверяться в контроллере или модели, а уж затем передаваться в Вид. Или я параноик?

3) Я бы переделал пути, чтоб избавиться от ?mode=raw в адресе /logs/raw/2012/06/25?mode=raw (Больше с эстетической точки зрения)
Например так:
Код:
Route::set('logviewer', 'logs(/<year>(/<month>(/<day>(/<level>)(/<mode>))))', 
            array('mode' => 'raw|formatted'))
        ->defaults(array(
            'controller' => 'Logs',
            'action' => 'index',           
        ));


Вернуться к началу
 Профиль  
 
 Заголовок сообщения: Re: Log Viewer
СообщениеДобавлено: 14 июн 2013, 17:45 
Не в сети
Бывалый
Аватара пользователя

Зарегистрирован: 02 апр 2013, 16:26
Сообщения: 474
Откуда: Сергиев Посад
MAD_B> тебя стебут как бы... ты видимо не последнюю ветку смотрел, https://github.com/laurent22/Kohana-Log-Viewer
атам написано:
Цитата:
Notes:
BE CAREFUL ABOUT USING ON PRODUCTION!


Цитата:
1) Элементы в верхнем меню выглядят так <a href="/logs/2012\10/01/?mode=raw">2012\10</a>

это только в винде, а винде как бы похрен, что так / что так \ и браузер тут не при чем

Цитата:
2) Или я параноик?

ты агностик скорее

http://kohanaframework.org/3.3/guide-api/Kohana#init

// Sanitize all request variables
$_GET = Kohana::sanitize($_GET);
$_POST = Kohana::sanitize($_POST);
$_COOKIE = Kohana::sanitize($_COOKIE);

Цитата:
3) Я бы переделал пути

не работает твой роут и работать не будет. этот скрипт работает в бекенде, в бекенде по большей части всем насрать на вид ссылок

Итого:
1. ты много думал и мало разбирался\делал.
2. когда тебе что-то кто-то говорит, пусть даже очень авторитетный или ты так по каким-то причинам считаешь, то не ленись лишний раз проверить, а действительно ли так?
3. знаешь как сделать лучше? сделай лучше, а потом покажи всем и докажи этим. что ты круче их.

update: посмотрел детально модуль, подтверждаю, что это по коду это полное говнище, работоспособность не тестил ввиду отсутствия необходимости.

еще я не понимаю почему большинство думает, что ,например, для работы с логами нужен обязательно какой-то специальный кохана модуль или для работы с yaml нужен спец класс, кохана не был бы хорошим фреймворком если бы не старался хранить\работать с данными в общепринятых форматах - парсить логи можно любым удобным способом.

update2: времени у меня много, а почитать нечего т.ч. пожалуй прокачаю сегодня этот модуль

_________________
Майкл Джордан играет в баскетбол. Чарльз Мэнсон убивает людей. Я пишу код. У каждого свой талант.


Вернуться к началу
 Профиль  
 
 Заголовок сообщения: Re: Log Viewer
СообщениеДобавлено: 14 июн 2013, 19:14 
Не в сети
Новичок

Зарегистрирован: 07 июн 2013, 18:07
Сообщения: 24
Откуда: Питер
какой неприятный снисходительный тон, даже отвечать не хочется...

но, жду с нетерпением результат прокачки


Последний раз редактировалось MAD_B 14 июн 2013, 19:15, всего редактировалось 1 раз.

Вернуться к началу
 Профиль  
 
 Заголовок сообщения: Re: Log Viewer
СообщениеДобавлено: 15 июн 2013, 00:30 
Не в сети
Бывалый
Аватара пользователя

Зарегистрирован: 02 апр 2013, 16:26
Сообщения: 474
Откуда: Сергиев Посад
MAD_B> он скорее раздраженный, и заметь не без причин. функциональнее точно не станет, но хотя бы грамотно написано будет

_________________
Майкл Джордан играет в баскетбол. Чарльз Мэнсон убивает людей. Я пишу код. У каждого свой талант.


Вернуться к началу
 Профиль  
 
 Заголовок сообщения: Re: Log Viewer
СообщениеДобавлено: 25 июн 2013, 17:35 
Не в сети
Новичок

Зарегистрирован: 07 июн 2013, 18:07
Сообщения: 24
Откуда: Питер
Я бы взглянул на результат прокачки, если можно.


Вернуться к началу
 Профиль  
 
 Заголовок сообщения: Re: Log Viewer
СообщениеДобавлено: 28 июн 2013, 20:04 
Не в сети
Администратор
Аватара пользователя

Зарегистрирован: 24 июл 2012, 18:00
Сообщения: 701
Откуда: Murom, Russia
Господа, давайте будем конструктивны и объективны :)


Вернуться к началу
 Профиль  
 
Показать сообщения за:  Поле сортировки  
Начать новую тему Ответить на тему  [ Сообщений: 19 ]  На страницу 1, 2  След.

Часовой пояс: UTC + 4 часа [ Летнее время ]


Кто сейчас на конференции

Сейчас этот форум просматривают: нет зарегистрированных пользователей и гости: 9


Вы не можете начинать темы
Вы не можете отвечать на сообщения
Вы не можете редактировать свои сообщения
Вы не можете удалять свои сообщения
Вы не можете добавлять вложения

Найти:
Перейти:  
cron
Все о фреймворке Kohana  | 
Powered by phpBB® Forum Software © phpBB Group