бага в localcopy.ocmod.xml

[nikifalex 515]

Есть очень часто используемая модификация localcopy.ocmod.xml

так вот в ней есть такой код

 

foreach (glob($next) as $file) {
   if (is_dir($file)) {
      $path[] = $file . '/*';
   }
   $files[] = $file;
}

Они не учли что glob может работать не так как думали

http://php.net/manual/en/function.glob.php

On some systems it is impossible to distinguish between empty match and an error.

 

Т.е. иногда будет пустой массив, а иногда то будет false

 

И если в устанавливаемом дополнении будет пустой каталог внутри, то будет warning и json parser error

 

Исправляется просто добавлением 

if (glob($next)) {

вначале.

 

Исправленная версия ниже. Может кому-то пригодится.

 

localcopy.ocmod.xml

[nikifalex 515]

абсолютно тоже самое и в 

extension_uninstaller.ocmod.xml

этого же автора.

Плюс там же и еще один момент исправлен.

Файл измененный тут

 

extension_uninstaller.ocmod.xml

[nikifalex 515]

дубль

[AlexDW 1 161]

ставьте патч, потом обновляйте кеш модификаторов и пробуйте загрузить какой-нибудь архив ocmod.zip

если не помогло - значит у вас проблемы с самим движком

[Wild 19]

В обоих модификациях присутствует код заменяющий одну и ту же строку.

localcopy.ocmod.xml:

<search ><![CDATA['url'  => str_replace('&', '&', $this->url->link('extension/installer/ftp', 'token=' . $this->session->data['token'],]]></search>
<add position="replace"><![CDATA['url'  => str_replace('&', '&', $this->url->link('extension/installer/localcopy', 'token=' . $this->session->data['token'],]]></add>

extension_uninstaller.ocmod.xml:

<search ><![CDATA['url'  => str_replace('&amp;', '&', $this->url->link('extension/installer/ftp', 'token=' . $this->session->data['token'], 'SSL')),]]></search>
<add position="replace"><![CDATA['url'  => str_replace('&amp;', '&', $this->url->link('extension/installer/'. $this->rewrite_step('ftp', 'localremove'), 'token=' . $this->session->data['token'], 'SSL')),]]></add>

Есть подозрение, что установленные оба сразу, они конфликтуют, поскольку в результате строка меняется только одним модификатором и только одно расширение остается рабочим.
Кто-нить сталкивался, есть решение?

[nikifalex 515]

12 минут назад, Wild сказал:

они конфликтуют, поскольку в результате строка меняется только одним модификатором и только одно расширение остается рабочим

посмотрите что в ocmod.log

[Wild 19]

Ничего, поскольку там присутствует error="skip".
Просто кто первый поменял строку, того и тапки!

[Wild 19]

Там в URL получается либо localremove, либо localcopy.

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

[nikifalex 515]

а... ну может быть может быть

[Wild 19]

То бишь либо один, либо другой.
Как их подружить?
Ведь оба нужны и полезны!

Приходится врубать/вырубать вручную, достает эта возня.
 

[nikifalex 515]

да там вроде хороший саппорт, сейчас подскажут

[Wild 19]

Уже написал им на opencart.com только что.

 

[chukcha 1 426]

Хм, а почему у меня нет такого?

[Wild 19]

43 минуты назад, chukcha сказал:

Хм, а почему у меня нет такого?

Хм...

Стоят и активированы оба модуля?
Можно поинтересоваться что у вас в system/modification/admin/controller/extension/installer.php ?
В блоке:

// FTP
$json['step'][] = array(
	'text' => $this->language->get('text_ftp'),
	'url'  => str_replace('&', '&', $this->url->link('extension/installer/'. $this->rewrite_step('ftp', 'localremove'), 'token=' . $this->session->data['token'], 'SSL')),
	'path' => $path
);

 

[chukcha 1 426]

Я без понятия,
Я как скачал его при появленни, так и остался..

 

смотрю, там версия 2016 года

 

[Wild 19]

Вы скачали и установили и включили ОБА модуля local fix + uninstall?

[chukcha 1 426]

я вообще анинсталом не пользуюсь, ибо это не нужно

[Wild 19]

Тогда понятно.
Речь шла об обоих одновременно.
Я тоже пользуюсь анинсталлом не часто, но когда ставишь модуль для проверки и последующим удалением, заколупывает лазанье по папкам и чистка вручную.
По этому и возник вопрос.

[Satriani 0]

Я все ручакми делаю. Анинстал юзал будучи начинающим.

 

[Wild 19]

Баг пофиксили и выложили новые версии модификаторов.
Кому надо, качаем...

Спойлер

 

QuickFix: Extensions Installer issue when FTP support disabled

ExtensionUninstaller - Safely remove extensions from your store

 

 

[nikifalex 515]

тогда закройте эту тему. Пусть там и скачивают.