f=
в карту?Я исправил в phpBBex 2.0 каноничные ссылки на ссылки с параметром
f=
, так как ещё во времена 1.х ссылки были такими.Достаточно проверять параметр конфигурации
phpbbex_version
, чтобы он был больше 2.0.0.Заранее спасибо.
f=
в карту?f=
, так как ещё во времена 1.х ссылки были такими.phpbbex_version
, чтобы он был больше 2.0.0.f=
в phpBBex? Там ссылки на темы вроде наоборот без f.Shredder писал(а):Если вид канонических ссылок и ссылок в топиках не отличается от стандартного движка
Shredder писал(а):Никакой нагрузки от переписи каноникла расширением нет, поверьте - никакой.
Код: Выделить всё
Time: 0.044s | Queries: 23 | Peak Memory Usage: 3.79 МБ | GZIP: Off | SQL Explain
Код: Выделить всё
Time: 0.043s | Queries: 23 | Peak Memory Usage: 3.77 МБ | GZIP: Off | SQL Explain
class_loader__
обзаводится ещё одной строкой, и т.д. и т.п.Shredder писал(а):усложняя её код
Код: Выделить всё
if ($this->phpbb_extension_manager->is_enabled('shredder/seo_topic_url'))
{
$seo_topic_url = (!$this->config['stc_mode']) ? true : false;
}
Код: Выделить всё
$f_part = (!isset($seo_topic_url) || (isset($seo_topic_url) && $seo_topic_url)) ? "" : "f=$f_id&";
Код: Выделить всё
$with_f_parameter = false;
if ($this->phpbb_extension_manager->is_enabled('shredder/seo_topic_url'))
{
$with_f_parameter = (!$this->config['stc_mode']) ? false : true;
}
elseif (isset($this->config['phpbbex_version']))
{
$with_f_parameter = true;
}
$f_part = $with_f_parameter ? "f=$f_id&" : '';
seo_topic_url
устанавливается всегда, даже когда даром не нужна, и её срок жизни большой, она оторвана от места первого использования. Так же проверка её существования идёт два раза.with_f_parameter
инициализируется только когда нужно, и сразу же используется.Shredder писал(а):Тем более:
Sumanai писал(а):С расширением
Sumanai писал(а):Без него
Sumanai писал(а):Каноникал как раз и отличается, он сразу идёт правильный.
Sumanai писал(а):Я исправил в phpBBex 2.0 каноничные ссылки на ссылки с параметромf=
, так как ещё во времена 1.х ссылки были такими.
f=
, как и в оригинальном движке, а потом пишете, что "каноникал отличается от оригинала". Противоречие.Shredder писал(а):Вы это серьёзно?
Shredder писал(а):Т.е. вы пишете, что по умолчанию в phpBBEx ссылки идут без f=,
Sumanai писал(а):Перечитайте пожалуйста процитированное ещё раз, и вы поймёте. Или посмотрите на мой форум.
f=
, в отличие от стандартного движка. Но в таком случае эти слова:Sumanai писал(а):Можно ли встроить поддержку phpBBex и добавлять для него параметр f= в карту?
Я исправил в phpBBex 2.0 каноничные ссылки на ссылки с параметромf=
, так как ещё во времена 1.х ссылки были такими.
f=
, то раньше его в phpBBEx 2.0 не было, как нет и в оригинальном движке. Может, вы всё-таки в карте исправили ссылки, а не в phpBBEx? Shredder писал(а):Из цитаты получается, что раз вы добавили в phpBBEx 2.0 к каноникалу параметр f=, то раньше его в phpBBEx 2.0 не было, как нет и в оригинальном движке.
f=
только на своём личном форуме.Sumanai писал(а):Итого- у вас переменная seo_topic_url устанавливается всегда, даже когда даром не нужна, и её срок жизни большой, она оторвана от места первого использования. Так же проверка её существования идёт два раза.
У меня же переменная with_f_parameter инициализируется только когда нужно, и сразу же используется.
$this->phpbb_extension_manager->is_enabled('shredder/seo_topic_url')
внутрь цикла while
- сократит некую мифическую нагрузку, а наоборот её не увеличит? Ведь в этом случае проверка phpbb_extension_manager->is_enabled
будет выполняться не один раз в начале функции, а во время генерации каждой страницы для каждой темы на форуме.Shredder писал(а):Если вы - один из разработчиков phpBBEx или из тех людей, правки которых добавляют в дистрибутив phpBBEx, то об этом стоило написать сразу, я же не экстрасенс.
Shredder писал(а):И форумы phpBBEx не читаю,
Shredder писал(а):внутрь циклаwhile
Sumanai писал(а):вечер на дворе, а то и ночь.
$with_f_parameter
чуть более прозрачен, даже если перенести инициализацию обратно.Sumanai писал(а):Я и на phpbbguru отмечался.
Shredder писал(а):Теперь это место почти не читаю.