Alert Feb 7 2012 at 16:50

Модуль профайлинга «ProfilerToolbar»

3 min

3.6K

Kohana *

+41

Comments 46

ivaer Feb 7 2012 at 17:14

отличная вещь
у вас есть в дальнейшем в планах сделать и для фреймворка Yii?

JiLiZART Feb 7 2012 at 17:47

Не он так другой, в yii поболее информации можно вывести через LogRoute :) И цепляться автоматически будет в лайоут.

Alert Feb 7 2012 at 18:06

Всё хочу попробовать Yii, но никак не попадается такой проект на котором можно его потестить.
Как только подвернёться, попробую портировать.

Chvanikoff Feb 7 2012 at 17:20

Код еще не смотрел, но то, что Вы не следуете Kohana Conventions and Coding Style — это плохо. Вижу, Вы и сами в курсе, но быть может пара-тройка таких комментариев подтолкнет Вас к тому, чтобы придерживаться соглашений впоследствии ;)
А так модуль полезный, только добавьте еще в статью информацию о совместимости с версиями фреймворка, если есть ограничения (хотя если нет — тоже скажите об этом)

Alert Feb 7 2012 at 18:09

Понимаю что соответствовать стилю надо, но если данный модуль особо никому не понадобится, то и тратить время на это не хочется.
Ежели им будут пользоваться, то и код приведу к требуемому стилю и userguide организую)

Alert Feb 7 2012 at 18:29

На счет поддерживаемой версии фреймворка.
На панели слева (первый скриншот), как раз и указана версия самого модуля + версия Kohana для которой он написан.

Tesla Feb 8 2012 at 04:05

Модуль нужен :) И, когда будете форматировать код, замените пробелы на табы.

mrxak Feb 7 2012 at 21:10

Возможно я чего-то не понимаю, но у них очень странные стандарты. Всю жизнь писал if (TRUE == $var), просто чтобы сразу получить ошибку, если вдруг забуду второй знак равенства, и тут какой-то фреймворк заставляет меня писать if ($var == TRUE), аргументируя это тем, что мы читаем слева направо. Писать AND вместо && тоже как-то «необычно».

А автору огромное спасибо за модуль.

Tesla Feb 8 2012 at 03:58

Просто так естественнее, мы говорим «если это — правда», а не «если правда — это». Кстати, я к AND вместо && тоже долго не мог привыкнуть, но в итоге код читать легче. Там ведь вся соль этого стиля, чтобы читалось буквально «как говорится», просто и естественно.

nonexistent Feb 7 2012 at 17:23

Небольшой баг-репорт. Если модуль рендерится (<?php ProfilerToolbar::render(true); ?>) после инклуда скриптов гуглмапы (echo html::script('http://maps.google.com/maps/api/js?v=3.5&sensor=true');) — мапа не отображается (без каких-либо ошибок). Если перед — все отлично.

А так — симпатично. Спасибо. :)

Alert Feb 7 2012 at 18:11

Спасибо, учту в следующей версии.

Cresh Feb 7 2012 at 17:41

Фреймворк очень интересный — но хабараэфект не держит =)

student_ivan Feb 7 2012 at 18:27

А почему бы не сделать решение автономным? Т.е. не зависящим от фреймворка? Чтобы каждый мог взять и «допилить» под любимый фреймворк/сайт, сам недавно думал написать подобное.

egor_bulychev Feb 7 2012 at 18:36

www.php-debug.com

student_ivan Feb 7 2012 at 19:32

Спасибо конечно, но это не совсем то, что я ожидал. Во первых Copy the css/js/images folders of /your_pear_root/data/PHP_Debug in your project directory — это мне не нравится. Думаю что подобного рода плагины должны быть единоличны, сразу с css/js и как можно меньше внешних файлов. Можно обойтись и вообще без картинок. Ну и эта PEAR-установка, тоже можно обойтись без нее, имхо много лишнего, всё таки наверное напишу свой велосипед.

egor_bulychev Feb 7 2012 at 19:37

если что могу быть тестером :)

Alert Feb 7 2012 at 20:06

Идея хорошая, но по-моему утопичная. Я лично не вижу как такое можно реализовать.
Если только сделать шаблон html+css+js, а все данные каждый будет собирать сам, но это можно и из ProfilerToolbar шаблон выдернуть.

student_ivan Feb 7 2012 at 21:18

Dependency Injection?

StraNNikk Feb 7 2012 at 18:34

Для Zend Framework помнится был аналогичный тулбар: habrahabr.ru/blogs/zend_framework/62023/
Правда по моему начиная с какой-то версии его развитие приостановилось

werdender Feb 7 2012 at 20:18

Выглядит удобно. Я как раз и попробую на днях, как раз с коханой вожусь.
Единственное, не очень понял с драйвером БД — только мускуль? ПДО отвалится?

werdender Feb 7 2012 at 20:22

А, хотел же еще предложение предложить — добавить в конфиг флаг на окружение. Типа ставишь true, если вывод нужен только при Kohana::DEVELOPMENT === Kohana::$environment — чтобы во вьюхах и т.д. с условиями не заморачиваться.

Alert Feb 7 2012 at 20:43

Думаю добавлять это в репозиторий смысла нет. Я, например, панель и в продакшене какое-то время держу включенной, только она рендерится если юзер авторизован под admin'ом.
А если вам так удобней, то копируйте конфиг в /applications/config/profilertoolbar.php и извращайтесь как угодно =)

werdender Feb 7 2012 at 20:54

А там одним конфигом не отделаешься — флаг в render обрабатывать надо.
Тут вообще можно проще поступить:

 ProfilerToolbar::render(Kohana::DEVELOPMENT === Kohana::$environment);

Alert Feb 7 2012 at 21:13

Да, тоже вариант, но вообще по хорошему в продакшене этот модуль надо вообще отключать.
Но если его отключить, то ваш код вызовет ошибку и по-этому правильней будет:

<?php if(Kohana::$environment === Kohana::DEVELOPMENT) ProfilerToolbar::render(true);?>

А если лениво постоянно удалять его из bootstrap.php то можно сделать так:

$modules = array();
if(Kohana::$environment === Kohana::DEVELOPMENT){
  $modules['profilertoolbar'] = MODPATH.'profilertoolbar';
}
$modules['cache']       = MODPATH.'cache';
$modules['database'] = MODPATH.'database';
...
Kohana::modules($modules);

werdender Feb 7 2012 at 21:26

Если убирать из бутстрапа, то и во вьюхах/контроллерах такое-же условие нужно. Наверное лучше просто в render() проверку аргумента сделать сверху, а не перед выводом. Тогда только Kohana::find_file() поработает, а методы модуля не дернутся — все экономичней будет, бутстрап можно будет и не трогать.

Alert Feb 7 2012 at 21:48

Ну это масло маслянное получится. Там уже есть проверка.

if(!self::cfg('html.enabled')) return '';

Для вашей задачи надо всего лишь в конфиге написать:

  'html'=>array(
    'enabled' => Kohana::$environment === Kohana::DEVELOPMENT, 
    ...
  ),
  'firebug'=>array(
    'enabled' => Kohana::$environment === Kohana::DEVELOPMENT,
    ...

werdender Feb 7 2012 at 21:58

А, ну да, так лучше всего будет.

Alert Feb 7 2012 at 20:35

Я PDO не использую и забыл про него =)
Добавлю в следующей версии.

Ekstazi Feb 7 2012 at 21:56

Спасибо, как будет время, портирую на yii. Вы не против?

Alert Feb 7 2012 at 22:14

Я только за! Сообщите как сделаете, размещу информацию на странице модуля

Ekstazi Feb 7 2012 at 22:23

Ок, это только как будет время, но, уже есть пару идей по улучшению… В частности, не трогать главный шаблон и так далее…

Big_Shark Feb 8 2012 at 02:22

А почему Вы не хотите помогать развивать существующий, а решили сделать свой собственный?

Alert Feb 8 2012 at 08:39

Потому что мне в существующем не нравится всё, от дизайна до функционала.
Не могу же я отправить Pull Request на замену 100% кода =)

dohlik Feb 8 2012 at 18:24

Видел на официальном форуме, не знал, что это наши люди сделали )) ИМХО, это немного другой подход к DebugToolbar, В первую очередь из-за внесения изменений в Cache и Database. Правильно сделали, что оформили как самостоятельный проект.

pewpew Feb 8 2012 at 05:53

Спасибо! Как раз делаю в свободное от работы время на кохане небольшой проектик.
DebugToolbar также не устроил, а велосипеды писать не нашлось времени.

dohlik Feb 8 2012 at 18:16

Если не трудно, опишите недостатки KDT. Может, удастся его немного улучшить в свободное время ;)

pewpew Feb 9 2012 at 03:55

Как уже отметил автор, отладка Ajax-запросов и поддержка Firebug.
Плюс, прозрачно описана отладка собственных переменных.

UFO landed and left these words here

Alert Feb 8 2012 at 08:46

Частично дублируется функционал стандартного профайлера

Это плохо? В стандартном профайлере информация выводится в неудобочитаемом виде. Я же сделал её приятной для восприятия и не портящей дизайн сайта + дополнительные плюшки.

UFO landed and left these words here

sunburn Feb 8 2012 at 11:01

Спасибо большое! Уже начали использовать в своем проекте

maxnag Mar 18 2012 at 12:01

Модуль хороший, молодец! Но как обычно есть дёготь, так пришлось что надо было заюзать транзакции. Но увы сами запросы транзакции не показываются, раз ты уже переопределил класс БД, так может подправишь этот недочет?

Alert Mar 18 2012 at 13:47

Хорошо, потестирую в ближайшее время.
Каким драйвером пользуетесь: mysql, pdo, ...?

p.s.
Если есть какие-то предложения по улучшению или нашли баги, то пишите лучше в Issue на github.com. Там удобней подобные вопросы решать.
Либо на форум kohana в тему ProfilerToolbar for kohana 3.2

maxnag Mar 18 2012 at 18:29

окей, опишу проблему на гитхабе.

maxnag Dec 5 2012 at 07:24

Уже вышла Кохана 3.3, будите допиливать свою тулзу под эту версию? Я сам поставил, все завелось, но есть один минус, когда возникает ошибка, чтобы её увидеть приходиться отключать ваш модуль.

Alert Dec 5 2012 at 07:47

Да, буду. Сейчас времени не очень много, но в течении месяца точно сделаю.