Консолидация ресурсов

Прододжение темы про некорректную работу алгоритма косолидации

Macik

#34340 17.05.2012 12:24

Team
Thanked: 181 раз

Установил последнее исправление №914

Не помогло. Включил отладку и залез в код. Вот, что выясняется:

PHP

        
          
          
              
              if (preg_match_all('#\burl\((\'|")?([^\)"\']+)\1?\)#i', $file_code, $mt, PREG_SET_ORDER))
{
        foreach ($mt as $m)
    {
        $filename = empty($relative_path) ? $m[2] : $relative_path . '/' . $m[2];
        $filename = str_replace($current_path, '', realpath($filename));
        if (!$filename)
        {
            continue;
        }
        if ($filename[0] === '/')
        {
            $filename = mb_substr($filename, 1);
        }
        $file_code = str_replace($m[0], 'url("' . $filename . '")', $file_code);
    }
}

            

        
      

- проблема не в регулярном выражении, оно отрабатывает нормально (по крайней мере с последними правками).
- проблема в логике строк 4005-4007, т.к. на выходе имеем пустой $filename и как следствие выход из цикла без замены кода в CSS.

Теперь подробнее...

В итоге на одном (для примера) из шагов цикла имеем (строки 4005-4007) следующие входные данные:

$path = 'plugins/ratings/tpl/ratings.css';
$relative_path = '\plugins\ratings\tpl';
$m[2] = '../../../images/icons/default/delete.png';
т.е. учитывая путь в $m[2] мы на выходе должны иметь строку 'images/icons/default/delete.png', однако в строке 4006 происходит обнуление $filename (дальше естественно алгоритм уже «правильно» не работает).

Стал копать дальше пришел к выводу, что проблема в функции realpath(), для которой (по видимому) очень важно в каком формате передается путь. Т.к. у меня тестовый сервер работает под Windows, то часть путей (традиционно) имеют формат с обратным слэшем ("C:\dvp\VertrigoServ\www\sites\cot98\plugins\ratings\tpl").

В моем примере (в строке 4006) получается realpath('\plugins\ratings\tpl/../../../images/icons/default/delete.png') возвращает false.

Поэкпериментировал:

realpath('\plugins\ratings\tpl/../../../images/icons/default/delete.png') == false

realpath('plugins\ratings\tpl/../../../images/icons/default/delete.png') == false

realpath('/plugins/ratings/tpl/../../../images/icons/default/delete.png') == false

и только когда все слэши идут в одном формате и без открывающего слеша работает правильно:

realpath('plugins/ratings/tpl/../../../images/icons/default/delete.png') == 'C:\dvp\VertrigoServ\www\sites\cot98\images\icons\default\delete.png'

Так что, как решение проблемы надо проверять (и отрезать) ведущий слэш в $filename и конвертировать слэши в «прямые» перед передачей в realpath().

Добавлено 3 минуты спустя:

Да, и в $current_path слеши тоже придется заменить для единообразия и правильной работы str_replace(); в строке 4006.

https://github.com/macik
правильный хостинг — https://goo.gl/fjCa1F

Відредаговано: Macik (17.05.2012 12:35, 13 років тому)