Русскоязычный форум, посвященный фреймворку Kohana
http://forum.kohanaframework.su/

Log Viewer
http://forum.kohanaframework.su/viewtopic.php?f=20&t=140
Страница 1 из 2

Автор:  vomchik [ 22 апр 2012, 22:49 ]
Заголовок сообщения:  Log Viewer

Хороший модуль для просмотра логов
Работает только на php 5.3+
https://github.com/ajaxray/Kohana-Log-Viewer

Автор:  admin [ 23 апр 2012, 00:05 ]
Заголовок сообщения:  Re: Log Viewer

Нужная вещь. Спасибо.

Автор:  MAD_B [ 13 июн 2013, 14:17 ]
Заголовок сообщения:  Re: Log Viewer

Не будет работать в 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) начинают работать...

Автор:  Sergey [ 13 июн 2013, 21:50 ]
Заголовок сообщения:  Re: Log Viewer

Молодец :) Исправил ошибку. А переделать дальше модуль под Ко3.3?

Автор:  MAD_B [ 14 июн 2013, 17:22 ]
Заголовок сообщения:  Re: Log Viewer

Мне модуль не нужен, переписывать не буду. Смотрю его только чтоб приобщиться так сказать ...
Ничего глобального менять не пришлось, чтоб он нормально заработал уже через 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',           
        ));

Автор:  WinterSilence [ 14 июн 2013, 17:45 ]
Заголовок сообщения:  Re: Log Viewer

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: времени у меня много, а почитать нечего т.ч. пожалуй прокачаю сегодня этот модуль

Автор:  MAD_B [ 14 июн 2013, 19:14 ]
Заголовок сообщения:  Re: Log Viewer

какой неприятный снисходительный тон, даже отвечать не хочется...

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

Автор:  WinterSilence [ 15 июн 2013, 00:30 ]
Заголовок сообщения:  Re: Log Viewer

MAD_B> он скорее раздраженный, и заметь не без причин. функциональнее точно не станет, но хотя бы грамотно написано будет

Автор:  MAD_B [ 25 июн 2013, 17:35 ]
Заголовок сообщения:  Re: Log Viewer

Я бы взглянул на результат прокачки, если можно.

Автор:  Sergey [ 28 июн 2013, 20:04 ]
Заголовок сообщения:  Re: Log Viewer

Господа, давайте будем конструктивны и объективны :)

Страница 1 из 2 Часовой пояс: UTC + 4 часа [ Летнее время ]
Powered by phpBB® Forum Software © phpBB Group
http://www.phpbb.com/