Деинсталляция компонентов

[Arkadiy]

Столкнулся с такой проблемой: joostina 1.3.0 не удаляет таблицы компонента из БД при удалении компонента. Конкретно Виртуемарт.
за удаление таблиц отвечает функция com_uninstall() в файле \administrator\components\com_virtuemart\uninstall.virtuemart.php

Так вот, чтобы ее подключить надо проинклюдить этот файл. в Joostina за это
отвечает строка

Код:

// Try to find the uninstall file
$filesindir = mosReadDirectory($config->config_absolute_path.DS.ADMINISTRATOR_DIRECTORY.DS.'components'.DS.$row->option,'uninstall');
if(count($filesindir) > 0) {
$uninstall_file = $filesindir[0];
if(file_exists($mosConfig_absolute_path.DS.ADMINISTRATOR_DIRECTORY.DS.'components'.DS.$row->option.DS.$uninstall_file)) {
require_once ($mosConfig_absolute_path.DS.ADMINISTRATOR_DIRECTORY.DS.'components'.DS.$row->option.DS.$uninstall_file);
$uninstallret = com_uninstall();
}
}

в файле
\administrator\components\com_installer\component\component.class.php


Так вот здесь думаю собака и порылась, конкретно в
$uninstall_file = $filesindir[0];

Почему uninstall_file должен быть первым в списке файлов?

[smart]

Ну если файлов, содержащих в имени uninstall несколько, то вполне можно взять первый. Хотя правильнее брать не файл по имени, а на основании информации в xml-файле компонента.

[Arkadiy]

Извиняюсь, написал не разобравшись с функцией, в таком случае должно работать, но почему-то не цепляет файл. В чем может быть еще причина?

[Arkadiy]

Проверил, все определяется нормально, значит не там роюсь.

Smart спасибо.

[Arkadiy]

$filesindir = mosReadDirectory($config->config_absolute_path
if(file_exists($mosConfig_absolute_path

Почему-то в первом случае абсолютный путь $config->config_absolute_path, во втором $mosConfig_absolute_path.
А объявления глобальной переменной $mosConfig_absolute_path нету в функции. Может здесь косяк?

[smart]

А объявления глобальной переменной $mosConfig_absolute_path нету в функции. Может здесь косяк?

вполне возможно, надо послушать, что скажет boston.

[boston]

Хех, наш косяк, спасибо что заметили, исправим

[Arkadiy]

Поправил

Код:

// Try to find the uninstall file
$filesindir = mosReadDirectory($config->config_absolute_path.DS.ADMINISTRATOR_DIRECTORY.DS.'components'.DS.$row->option,'uninstall');
if(count($filesindir) > 0) {
$uninstall_file = $filesindir[0];
if(file_exists($config->config_absolute_path.DS.ADMINISTRATOR_DIRECTORY.DS.'components'.DS.$row->option.DS.$uninstall_file)) {
require_once ($config->config_absolute_path.DS.ADMINISTRATOR_DIRECTORY.DS.'components'.DS.$row->option.DS.$uninstall_file);
$uninstallret = com_uninstall();
}
}

Заработало.

[smart]

Хех, наш косяк, спасибо что заметили, исправим

сделай ревизию, на использование таких переменных? А вообще, случаи использования необъявленных переменных (а в данном случае именно так и получается) очень легко увидеть в Zend Studio, там удобнейший Code Analyzer для этого есть

[boston]

Уже качаю )))

[boston]

Исправлено!

[smart]

Исправлено!

позволь полюбопытствовать - много нашлось подобных ситуаций? в смысле насколько полезна и своевременна была ревизия?

[boston]

Ревизию сделал пока тока начальную, избавился от глобальной переменной $mosConfig_absolute_path и добавил константу JPATH_BASE

Но полная ревизия тока начата, таких штук еще много может быть в коде.