Ars Longa, Vita Brevis » bbPress http://blog.sjinks.pro Quod scripsi, scripsi Sat, 19 May 2012 17:55:07 +0000 ru-RU hourly 1 SJ Hook Profiler — плагин для измерения производительности хуков http://blog.sjinks.pro/wordpress/plugins/642-sj-hook-profiler/ http://blog.sjinks.pro/wordpress/plugins/642-sj-hook-profiler/#comments Mon, 21 Sep 2009 23:01:11 +0000 Vladimir http://blog.sjinks.pro/?p=642 © 2012 Ars Longa, Vita Brevis. Все права защищены. Перепубликация материалов без разрешения автора запрещена.

При использовании материалов блога наличие активной не закрытой от индексирования ссылки на источник обязательно.

]]> Поиск слабых мест в и

Сразу оговорюсь, что речь пойдёт совсем не о боксе, а о WordPress и bbPress.

Разработчики плагинов WordPress и bbPress используют две функции для расширения функциональности WordPress/bbPress: это add_action() и add_filter(). Первая служит для установки обработчика некоторого события, вторая — для установки фильтра. Под хуком подразумевается обобщённое понятие (либо фильтр, либо обработчик).

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

В отличие от WP Tuner, которому явно нужно указывать, какие события нужно профилировать, автоматически определяет все используемые хуки и устанавливает обработчики, измеряющие их время работы. На выходе получается что-то вида данной таблицы:

Hook Name Total Time Invocations Average Time
pre_option_gmt_offset 0.000169 1 0.000169
option_blog_charset 0.000405 11 0.000037
plugins_loaded 0.014295 1 0.014295
option_siteurl 0.000810 26 0.000031
option_home 0.004852 160 0.000030
sanitize_comment_cookies 0.000117 1 0.000117
option_category_base 0.000029 1 0.000029
option_tag_base 0.000027 1 0.000027
setup_theme 0.000030 1 0.000030
wp_default_scripts 0.001951 1 0.001951
set_current_user 0.000114 1 0.000114
init 0.011603 1 0.011603
widgets_init 0.003384 1 0.003384
wp_default_styles 0.000349 1 0.000349
query_vars 0.000036 1 0.000036
sanitize_title 0.001518 17 0.000089
posts_where 0.000033 1 0.000033
the_posts 0.000127 1 0.000127
template_redirect 0.000541 1 0.000541
bloginfo 0.002297 8 0.000287
term_name 0.000621 2 0.000310
term_description 0.000982 2 0.000491
single_post_title 0.000468 2 0.000234
wp_title 0.000747 2 0.000373
wp_head 0.020773 1 0.020773
comments_open 0.000373 4 0.000093
the_title 0.011747 46 0.000255
wp_print_styles 0.000175 2 0.000088
style_loader_src 0.000330 9 0.000037
wp_print_scripts 0.000440 1 0.000440
print_scripts_array 0.000050 1 0.000050
wp_list_pages 0.000305 1 0.000305
the_author 0.000534 2 0.000267
the_content 1.004469 1 1.004469
comment_text 0.001105 1 0.001105
pings_open 0.000087 1 0.000087
comment_form 0.007084 1 0.007084
widget_title 0.001137 4 0.000284
list_cats 0.004768 22 0.000217
wp_footer 0.009798 1 0.009798
wp_print_footer_scripts 0.000115 1 0.000115

Из таблицы видно, что слабое место — это вызов фильтра the_content (вызов более одной секунды), и копать нужно именно в сторону оптимизации фильтров the_content. Пока не поддерживается профилирование каждого обработчика (то есть делаются замеры до вызова первого обработчика the_content и после вызова последнего), но это в планах.

Одна из особенностей плагина — поддержка как WordPress, так и bbPress (правда, пришлось немного поколдовать).

По умолчанию плагин активен только для администратора (WordPress) и Key Master (bbPress); изменить это можно, повесив обработчик на фильтр enable_hook_profiler (должен вернуть true, если профайлер нужно активировать и false в противном случае).

Скачать плагин SJ Hook Profiler 0.1.
Скачать плагин SJ Hook Profiler 0.2.

© 2012 Ars Longa, Vita Brevis. Все права защищены. Перепубликация материалов без разрешения автора запрещена.

При использовании материалов блога наличие активной не закрытой от индексирования ссылки на источник обязательно.

]]> http://blog.sjinks.pro/wordpress/plugins/642-sj-hook-profiler/feed/ 3 bbPress Attachments 0.2.7: решение проблемы с установкой http://blog.sjinks.pro/bbpress/641-bbpress-attachments-fixing-installer-bugs/ http://blog.sjinks.pro/bbpress/641-bbpress-attachments-fixing-installer-bugs/#comments Mon, 21 Sep 2009 20:37:21 +0000 Vladimir http://blog.sjinks.pro/?p=641 © 2012 Ars Longa, Vita Brevis. Все права защищены. Перепубликация материалов без разрешения автора запрещена.

При использовании материалов блога наличие активной не закрытой от индексирования ссылки на источник обязательно.

]]> Решение проблемы с активацией

Устанавливал заказчику  и bbPress Attachments 0.2.7. Плагин установился без проблем, но вот незадача: вложения загружаются (создаются файлы на диске), но на форуме не отображаются.

Оказалось, что всё дело портит в функции активации плагина.

Плагин хранит свои настройки непосредственно в коде:

$bb_attachments['role']['see']="read";           // minimum role to see list of attachments = read/participate/moderate/administrate
$bb_attachments['role']['inline']="read";    // minimum role to view inline reduced images = read/participate/moderate/administrate
$bb_attachments['role']['download']="participate";  // minimum role to download original = read/participate/moderate/administrate
$bb_attachments['role']['upload']="participate";  // minimum role to upload = participate/moderate/administrate (times out with post edit time)
$bb_attachments['role']['delete']="moderate";  // minimum role to delete = read/participate/moderate/administrate
// ...
$bb_attachments['db']="bb_attachments";   //   $bbdb->prefix."attachments";  // database name - force to "bb_attachments" if you need compatibility with an old install

Процедура активации выглядит так:

function bb_attachments_install() {
global $bbdb,$bb_attachments;
$bbdb->query("CREATE TABLE IF NOT EXISTS ".$bb_attachments['db']." (
    id              int(10)        UNSIGNED NOT NULL auto_increment,
    time            int(10)        UNSIGNED NOT NULL default '0',
    post_id         int(10)        UNSIGNED NOT NULL default '0',
    user_id         int(10)        UNSIGNED NOT NULL default '0',
    user_ip         int(10)        UNSIGNED NOT NULL default '0',
    status          tinyint(10)    UNSIGNED NOT NULL default '0',
    downloads       int(10)        UNSIGNED NOT NULL default '0',
    size            int(10)        UNSIGNED NOT NULL default '0',
    ext             varchar(255)            NOT NULL default '',
    mime            varchar(255)            NOT NULL default '',
    filename        varchar(255)            NOT NULL default '',
    PRIMARY KEY ( id ),
    INDEX ( post_id )
    ) CHARSET utf8  COLLATE utf8_general_ci");
//..
}

А к MySQL запрос приходит в виде

CREATE TABLE IF NOT EXISTS  ( /* ... */ ) CHARSET utf8  COLLATE utf8_general_ci

Так как имя таблицы пустое, то MySQL возвращает ошибку. Но ведь переменная $bb_attachments объявлена как глобальная и $bb_attachments['db']="bb_attachments". Проблема здесь в том, что bbPress при активации плагина подключает его код не из глобального контекста, а из функции:

function bb_activate_plugin( $plugin, $redirect = '' ) {
    $active_plugins = (array) bb_get_option( 'active_plugins' );
    $plugin = bb_plugin_basename( trim( $plugin ) );

    $valid_path = bb_validate_plugin( $plugin );
    if ( is_wp_error( $valid_path ) )
        return $valid_path;

    if ( in_array( $plugin, $active_plugins ) ) {
        return false;
    }

    if ( !empty( $redirect ) ) {
        // We'll override this later if the plugin can be included without fatal error
        wp_redirect( add_query_arg( '_scrape_nonce', bb_create_nonce( 'scrape-plugin_' . $plugin ), $redirect ) );
    }

    ob_start();
    @include( $valid_path ); /* <<< Вот оно! */
    // Add to the active plugins array
    $active_plugins[] = $plugin;
    ksort( $active_plugins );
    bb_update_option( 'active_plugins', $active_plugins );
    do_action( 'bb_activate_plugin_' . $plugin );
    ob_end_clean();

    return $valid_path;
}

Таким образом, в момент активации плагина массив $bb_attachments принадлежит не глобальному пространству имён ($GLOBALS), а функции bb_activate_plugin(). И, как следствие, global $bb_attachments адресует несуществующую на момент активации плагина глобальную переменную.

Решение: перед первым упоминанием переменной $bb_attachments добавить:

global $bb_attachments;

После чего код плагина будет выглядеть примерно так:

<?php
/*
Plugin Name: bbPress Attachments
Plugin URI: http://bbpress.org/plugins/topic/bb-attachments
Description: Gives members the ability to upload attachments on their posts.
Author: _ck_
Author URI: http://bbShowcase.org
Version: 0.2.7

License: CC-GNU-GPL http://creativecommons.org/licenses/GPL/2.0/

Donate: http://bbshowcase.org/donate/
*/

global $bb_attachments; /* <<< Вот оно! */
$bb_attachments['role']['see']="read";           // minimum role to see list of attachments = read/participate/moderate/administrate

Разработчикам (вообще всем) хочу сказать две вещи:

  1. Глобальные переменные — это плохо.
  2. Если всё же нужно использовать глобальные переменные, то их нужно явно помечать как глобальные. В противном случае при известных обстоятельствах они внезапно могут стать локальными.

© 2012 Ars Longa, Vita Brevis. Все права защищены. Перепубликация материалов без разрешения автора запрещена.

При использовании материалов блога наличие активной не закрытой от индексирования ссылки на источник обязательно.

]]> http://blog.sjinks.pro/bbpress/641-bbpress-attachments-fixing-installer-bugs/feed/ 2