Об одной ошибке в классе XoopsBlock

[LARK]

В классе XoopsBlock есть достаточно неприятная ошибка. Как известно можно выбирать, в рамках, каких модулей показывать тот или иной блок. Но проверка показывать блок или нет, происходит уже после того, как контент блока был загнан в массив методом buildBlock(). Другими словами, сначала блок формируется, а потом уже проверяется, выводить его или нет. Таким образом, мы получаем совершенно ненужные запросы к DB необходимые для построения конкретного блока. Плюс достаточно объемные и ненужные массивы с контентом блоков.

Лекарство:

Берем файл class/xoopsblock.php (RUNCMS 1.1A)

Находим метод getAllBlocksByGroup() – это 438 строка

Находим фрагмент, начинающийся на 505 строке

PHP код:

while ( $myrow = @$db->fetch_array($result) ) {
           if ( !in_array($myrow['bid'], $added) ) {
               if (!$asobject) {
                     $ret[] = $myrow['bid'];
                     } else {
                      $ret[] = new XoopsBlock($myrow);
                     }
         array_push($added, $myrow['bid']);
         }
 } 


И заменяем его на:

PHP код:

while ($myrow = @$db->fetch_array($result)) {
       if ($myrow['show_mid'] != '0') {
           if (!empty($show_mid)) {
             if (!in_array($show_mid, explode('|', $myrow['show_mid']))) continue;
           } else {
             if (!in_array('-1', explode('|', $myrow['show_mid']))) continue;
           }
       } 
         if (!in_array($myrow['bid'], $added)) {
           if (!$asobject) {
               $ret[] = $myrow['bid'];
           } else {
               $ret[] = new XoopsBlock($myrow);
           } 
           array_push($added, $myrow['bid']);
       } 
     } 


Таким образом, мы отсекаем блоки, которые не должны выводится в рамках данного модуля.

В прикрепленном файле вы найдете xoopsblock.php с вышеописанным изменением.

По идее из метода showBlock() аналогичною проверку можно вырезать.

[Zormax]

Заодно может посмотришь, почему динамически титлы у блоков не меняются?

[LARK]

К слову, статистика по запросам к DB, после установки данного хака на этом сайте (в режиме админа, понятно, что для незарегистрированных пользователей, мало что изменилось)

Главная страница было: 35/стало: 34
Архив файлов было: 35/стало: 25
Статьи было: 35/стало: 26
Партнеры было: 33/стало: 23

Это просто все к тому, что с этой ошибкой, если, допустим, на сайте (на главной странице) стоит какой-нибудь блок "мутант" вроде WhatNews с его лавинообразными запросами, то эти запросики будут отображаться и на всех других страницах, на которых этот блок не показывается.

[Zormax]

Не одного запроса не увидим уменьшенного, все как было...
С чем связано?

[LARK]

Zormax: Не одного запроса не увидим уменьшенного, все как было...

В смысле? Где не увидим? Ведь все зависит от каждого конкретного случая. И от настроек блоков - группа, модуль, тип страницы, что это за блок.

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

[Zormax]

LARK*Я не досказал свою мысль:
Небыло уменьшений на главной, на странице Файлы реальное уменьшение до 8 запросов.
На остальных не смотрел...

[clog]

В довнлоадс было 46 стало 28
В новостях было 51 стало 31
На форуме было 68 стало 54
В ссылках было 51 стало 25
В статьях было 43 стало 17
В FAQе было 43 стало 16
Просто на "самописанноый" странице без правого блока было 18 стало 14
Просто на "самописанноый" странице с правым блоком было 46 стало 19



Хух....+)

И еще раз!РЕСПЕКТ!!!

СУПЕР!

[LARK]

clog Это в режиме администратора, или незарегистрированного пользователя?

[clog]

В незарегестрированном.

Но скажу что и под админом тоже много запросов снизилось.

[LARK]

В прикрепленном файле исправление бага, из за которого сбрасывались галки доступа к блокам в настройках групп. Подробнее тут