Ошибки класса Database (RUNCMS 1.6.2)

[LARK]

1) Ошибка в методе fetch_rowset() класса Database

Метод fetch_rowset() класса Database, используется в адаптации phpBB2 от SVL, однако в этом методе не работает кеширование результата запроса к BD.

Исправление:

в файле /class/database/mysql.php

функцию fetch_rowset() это 460-484 строки в RUNCMS 1.6.2

PHP код:

   function fetch_rowset($resource = 0)
   {
      if($resource === 'cache' && $this->cached)
      {
         return $this->cache;
      }
      while($this->rowset[$resource] = @mysql_fetch_array($resource))
      {
         if($this->caching)
         {
            if($this->row[$resource] === false)
            {
               $this->write_cache();
            }
            $this->cache[] = $this->row[$resource];
         }
         $result[] = $this->rowset[$resource];
      }
      if($this->caching)
      {
         $this->write_cache();
      }

      return $result;
   } 


меняем на

PHP код:

   function fetch_rowset($resource = 0)
   {
      if($resource === 'cache' && $this->cached)
      {
         return $this->cache;
      }
      while($this->rowset[$resource] = @mysql_fetch_array($resource))
      {
         if($this->caching)
         {
            if($this->rowset[$resource] === false)
            {
               $this->write_cache();
            }
            $this->cache[] = $this->rowset[$resource];
         }
         $result[] = $this->rowset[$resource];
      }
      if($this->caching)
      {
         $this->write_cache();
      }

      return $result;
   } 


[ZlydenGL]

Т.е. вся правка сводится к замене вызова this->row на this->rowset?

[LARK]

Сообщение от ZlydenGL Т.е. вся правка сводится к замене вызова this->row на this->rowset?

Да. Данный метод практически не используется, но тем кто будет оптимизировать phpBB2 и вводить кеширование запросов перед fetch_rowset(), придется делать или этот фикс или переписывать fetch_rowset на while с fetch_array/fetch_assoc

[LARK]

Так же обнаружил, что кеширование запросов будет работать только если мы используем циклические конструкции.

Т.е.

PHP код:

while ( $row = $db->fetch_array($result) ) {} 


запрос будет кеширован, а

PHP код:

if ( $row = $db->fetch_array($result) ) {} 


не будет.

Это не совсем баг, просто недоработка механизма кеширования

[LARK]

2) Ошибка некорректного удаления кеша в функции query() класса Database

Для функции query() можно задавать время жизни файла кеша с данными конкретного SQL-запроса (5-й аргумент).

По истечении заданного времени файлы имеющие префикс заданный 4-м аргументом query(), должны удаляться. Однако удаляются все файлы кеша расположенные в /cache/sql/

Так как некоторые запросы кешируются на один час (вопрос - зачем?). То каждый час, весь кеш SQL-запросов сбрасывается.

исправление (для RUNCMS 1.6.2):

в файле /class/database/mysql.php

находим функцию query() и меняем 160 строку

PHP код:

if ($time_to_cache && @file_exists($filename) && (@filemtime($filename)+$time_to_cache<time())) $this->clear_cache($this->caching); 


на

PHP код:

if ($time_to_cache && @file_exists($filename) && (@filemtime($filename)+$time_to_cache<time())) $this->clear_cache($cache); 


[pan]

Это применимо к 1.4 ?

[LARK]

Сообщение от pan Это применимо к 1.4 ?

к 1.4 это не относится, там нет такого.

[LARK]

Сообщение от LARK 2) Ошибка некорректного удаления кеша в функции query() класса Database

Тут подумалось, что имеет смысл сделать чуть по другому:

меняем 160 строку на

PHP код:

if ($time_to_cache && @file_exists($filename) && (@filemtime($filename)+$time_to_cache<time())) $this->clear_cache($hash); 


т.е. в первом варианте удаляется группа файлов, а в новом конкретный файл запроса

[ZlydenGL]

На всякий случай - в RunCMS 1.6.1 требуемая строка - номер 161 Магическое число, аднака!

[Michael-XIII]

Спасиб закоммитил