Как обновить файлы публички при установлении обновлений модуля через файл updater.php?

Question

[Павел Сидоров]

Как обновить файлы публички при установлении обновлений модуля через файл updater.php?

Сейчас вот так. Обновляю только файлы шаблона

if(IsModuleInstalled('name_module'))
{
	if (is_dir(dirname(__FILE__).'/install/components'))
		$updater->CopyFiles("install/components", "components/");

	if (is_dir(dirname(__FILE__).'/install/js'))
		$updater->CopyFiles("install/js", "js/name_module/");
	
	$updater->CopyFiles("install/wizards/module/swp/site/templates/profimontag", "templates/profimontag");
	$updater->CopyFiles("install/wizards/module/swp/site/public/ru", "/");  //не работает 
}

второе правило не работает

Comments

[Алий Кунашев]

Почему не __DIR__?

[Павел Сидоров]

Алий Кунашев, а не могли бы написать полную конструкцию?

$updater->CopyFiles("install/wizards/module/swp/site/public/ru", __DIR__); // не работает

[PetrPo]

Павел Сидоров,

$updater->CopyFiles("install/wizards/module/swp/site/public/ru", "/");  //не работает

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

[Павел Сидоров]

PetrPo, да, верно

[Павел Сидоров]

PetrPo, при многосайтовости.

[PetrPo]

Павел Сидоров, В функцию CopyFiles передаются 2 параметра $fromDir, $toDir = откуда, куда. Потом к этим переменным дописывается $_SERVER["DOCUMENT_ROOT"]
Вот... если у тебя многосайтовость на разных доменах, то значит $_SERVER["DOCUMENT_ROOT"] не правильно определяется, а если на папках, то надо вторым параметром папку сайта указать - /папка сайта/

Если не поможет поставь AddMessage2Log($updater->errorMessage); может чего интересного в логе будет.
В публичке индексные файлы обновляешь?

[Павел Сидоров]

PetrPo, второй сайт просто в папке.
Понял. Надо через папку. Это будет по хорошему /#SITE_DIR#/? Или ?

[Павел Сидоров]

PetrPo, В публичке индексные файлы обновляешь? Ты имеешь ввиду что я хочу сделать ? Если да , то да.

[PetrPo]

Вот тут начинаются нюансы
1. Если есть многосайтовость, то и публичек несколько. Соответственно при обновлении модуля надо обновить несколько публичек.
Может быть есть какой-то изящный способ, я знаю только что-то типа такого

$arSites = array();
$dbRes = CSite::GetList($by='sort', $order='desc'));
while($arRes = $dbRes->Fetch()){
	$arSites[$arRes['LID']] = $arRes;
}

foreach($arSites as $arSite) {
	// тут обновляешь публичку
	// как пример
	$updater->CopyFiles("install/wizards/module/swp/site/public/ru", $arSite['DIR']);
}

2. Второй момент - не во всех публичках может использоваться твое решение, но и в твоем решении, и в стороннем могут совпадать имена публичных папок/файлов, надо как-то этот момент учесть, чтобы не затереть чужие файлы
3. В файлах типа index.php скорее всего стоят компоненты, пользователь твоего модуля в таких файлах мог внести изменения, даже не подозревая что сделал это (например перенастроив компонент из публички), если ты ему просто перезапишешь файл, это будет ой как не хорошо! Но это уже тема отдельного вопроса))

[Павел Сидоров]

PetrPo, благодарю за развернутый ответ. Оч помог.
Да, вот боль). Аспро я видел не перезаписывает файл. Они оставляют предыдущую копию с названием версии.

[Павел Сидоров]

PetrPo, но вот вопрос. Получается разработчики максимально не трогают файлы публички что ли? Всю логику запихивают в компоненты и вне ядра обновляют.
Кстати теперь видимо понимаю почему у аспро index.php и его вариации лежат в шаблоне сайта.

[Павел Сидоров]

PetrPo, вот с сохранением предыдущей версии файла не видел у них функционал ? Можешь скинуть ?

[PetrPo]

Павел Сидоров, не знаю как у аспро ...
Сохранение предыдущей версии файла - это наверное просто копия файла перед перезаписью? Тогда думаю тот же $updater->CopyFiles подойдет, если нет напиши свою простенькую функцию, путь к файлу ты знаешь, php copy и готово ссылка.
А вот как изменить сам файл, тут по ситуации надо смотреть, что за файл и что конкретно меняешь. Думаю, если есть возможность не трогать публичку, лучше не трогать. Приведи пример, что именно хочешь изменить в публичном файле?

Там ты вопрос задал про SITE_DIR, уточни для какой цели? если для установщика этот вариант не подойдет, у тебя не будет этой константы

[Павел Сидоров]

PetrPo, Спасибо еще раз. Кстати вот нашел статью у парня.

Можно собирать обновления. Например аспро и смотреть, что обновляют детально. А также и сам файл updater.php

[Павел Сидоров]

PetrPo, "Приведи пример, что именно хочешь изменить в публичном файле?"

Слушай, ну банально добавить компонент на index.php, например новости, или слайдер, который изначально не был предусмотрен.

[Павел Сидоров]

PetrPo, "про SITE_DIR, уточни для какой цели?"
Да, updater не понимает этой константы. Да я думаю, как быть) обновлять то хочется при многосайтовости). Ты дал вариант, когда много публичек, но в нем же проблема. Он обновит все публички)

[Павел Сидоров]

PetrPo, яндекс диск файлы обновления от аспро , последняя версия updater.php с названием версии.

[PetrPo]

Павел Сидоров, пишет вирусняк при скачивании архива
updater файлик посмотрю отпишусь, остальное открывать не буду))

[Павел Сидоров]

PetrPo, странно)) да, остальное там не нужно. вот отдельно

[Павел Сидоров]

PetrPo, Мне кажется вот это ключевое. Просто получают файл и делают замену $content = @file_get_contents($fileTo); строка 360, а выше много всяких замен str_replace

[Павел Сидоров]

PetrPo,

// remove old bak files
RemoveOldBakFiles();

// create bak files
CreateBakFile($_SERVER['DOCUMENT_ROOT'].'/bitrix/modules/'.MODULE_NAME.'/classes/general/CNext.php');

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

[PetrPo]

Павел Сидоров, ну как видишь все это сплошной костыль))

RemoveOldBakFiles();
CreateBakFile();
GetSites();
GetIBlocks();

Это все какие-то не битриксовые функции, значит они где-то в модуле аспро зашиты, поищи. Дальше идут изменения файлов публички через обычные replace, никаких копий файлов вроде не создается.