[3.3][beta] Update attachment

[Татьяна5]

Название расширения: Update attachment
Автор: Татьяна5, rxu
Описание расширения: Позволяет обновить (перезагрузить) вложения. Порт одноимённого мода
GitHub: https://github.com/Tatiana5/updateattachment
Скачать: https://github.com/Tatiana5/updateattac ... s/main.zip

[rxu]

Супер.
Моя попытка портирования в свое время (очень давно) уперлась в PLupload.

В слушателе лучше не использовать global, а передать сервис attachment.manager как зависимость.
Скрипт в overall_footer_body_after.html лучше выделить в файл js и вставить его в слушатель шаблона стандартным способом через INCLUDEJS.

Стиль в overall_header_head_append.html лучше выделить в файл css и вставить его в слушатель шаблона стандартным способом через INCLUDECSS.

[Татьяна5]

В слушателе лучше не использовать global, а передать сервис attachment.manager как зависимость.

Сначала так и попробовала сделать. Он так не передаётся

Скрипт в overall_footer_body_after.html лучше выделить в файл js и вставить его в слушатель шаблона стандартным способом через INCLUDEJS.

Сделала так для гарантии, что скрипт подключится после plupload

Стиль в overall_header_head_append.html лучше выделить в файл css и вставить его в слушатель шаблона стандартным способом через INCLUDECSS.

Там кода чуть, решила не задействовать под него целый файл

[rxu]

Он так не передаётся

А что происходит?

[Татьяна5]

А что происходит?

Пишет, что передана строка. И всё на том

[rxu]

Сделала так для гарантии, что скрипт подключится после plupload

Он гарантированно должен подключиться позже, так как подключение происходит в блоке {$SCRIPTS}

Код: Выделить всё

<!-- IF S_PLUPLOAD --><!-- INCLUDE plupload.html --><!-- ENDIF -->
{$SCRIPTS}

[Татьяна5]

Докопалась по этому вопросу до shared: false в объявлении того класса и так поняла, что передать его невозможно в принципе

Отправлено спустя 35 секунд:

Он гарантированно должен подключиться позже

Значит можно и файлом

[rxu]

Докопалась по этому вопросу до shared: false в объявлении того класса и так поняла, что передать его невозможно в принципе

Попробовал на локалке, всё передалось в виде объекта. А shared: false означает только то, что объект создается заново, а не передается уже готовый со старыми данными.

[Татьяна5]

rxu, можно тогда код в самом эксте?

[rxu]

код в самом эксте

Да вроде бы всё стандартно.
https://github.com/rxu/updateattachment ... 2a8b91d46b

[Татьяна5]

Вроде бы делала также, но у меня не работало. Где-то, значит, опечаталась

[rxu]

Ну, только если %% были вместо @

Отправлено спустя 13 минут 48 секунд:
Там еще в config/services.yml неверный формат перевода строк (Win) и в последней пустой строке лишний пробел.

[Татьяна5]

Там ещё с последовательным использованием кнопок "обновить" и "добавить" были накладки... Как временное решение - блокирую кнопки, чтобы ими невозможно было пользоваться друг за другом до перезагрузки страницы

Там еще в config/services.yml неверный формат перевода строк (Win) и в последней пустой строке лишний пробел.

Это поправила

[rxu]

Еще нюанс - с отключенным в браузере яваскриптом после выбора файла "Добавить файл" работает, "Обновить вложение" не работает.

[Татьяна5]

rxu, так и должно быть. Вариант без js не делала