Pathologic / SimpleGallery

MODX Evo plugin to manage images
20 stars 14 forks source link

Modx 1.1-d8.1.3 #78

Closed bagi091 closed 7 years ago

bagi091 commented 8 years ago

после этого коммита в modx используется jQuery 3.1, а из-за этого плагин SimpleGallery не может отобразиться, но если подключить jQuery migrate то все равно некорректно работает

  1. нет прогресс бара при загрузке
  2. не сворачивается диалоговое окно после загрузки
  3. не генерируется превью в админке (пока не понял почему, возможно мой косяк)
bagi091 commented 8 years ago

up.

  1. Развернул пустой modx на другом хостинге, тоже самое, превью не создается
bagi091 commented 8 years ago

up

  1. Однажды мы на эту тему с вами разговаривали, дело в том, что после обновления MODx даже пустая строка передается в параметры плагина и соответственно неверно проходит проверку на isset().

Чтобы превью начало формироваться нужно заменить isset на !empty в этой строке

Pathologic commented 8 years ago

Параметры, как я понял, так и не починили, вот и получается абы что. С jQuery я точно ничего не могу сказать, потому что даже на фронте ни разу не использовал 3ю версию. В теории проблемы могут вылезти не только в моем говнокоде, но и в компонентах, что еще хуже, потому что код того же EasyUI закрыт. Если есть возможность, то попробуйте вернуть тот jQuery, который был в прошлых версиях.

bagi091 commented 8 years ago

Не говнокод вовсe) пойду к Дмитрию