Aliaksandr
нашел древний форк от мая 2018 - там как в 1.4 - https://github.com/webber12/evolution/blob/develop/manager/includes/document.parser.class.inc.php#L5425 . Скорее всего это именно в 2.0 уже появилось
Alexey
Всем привет. После обновления до 1.4.10 и переноса на https отвалилось дерево ресурсов. Это известная проблема?
Михаил
https://github.com/evolution-cms/evolution/commit/9681480f4384ae1bb0b8a20c8038d319affb2ba1#diff-35727b64f9d1bda7f8729b704fca9fb8R5346
Aliaksandr
Aliaksandr
в разделе "and other things" )) и главное как сразу мастера по почерку узнали))
Dark
в разделе "and other things" )) и главное как сразу мастера по почерку узнали))
Так это ещё с тех пор кочует?
Pathologic
🙃
Aliaksandr
а когда понадобилось - оказалось, что все поломалось)
Dmytro
ну пока особо никто не пользовался оно было незаметно)
так я чет не пойму, а как оно могло поломать, если ключи - строки?
Anonymous
А оно не там
Anonymous
А скорей всего в конце evalPlugin есть unset
Anonymous
Я пока с телефона не могу потестить
Aliaksandr
Dmytro
что array_merge, что $a[$k] = $v, если $k - строка, будут одинаковые массивы
Anonymous
Anonymous
Но там в конце evalPlugin есть unset ссылочной переменной на $params
Anonymous
https://github.com/evolution-cms/evolution/blob/2.0.x/core/src/Core.php#L1789
Anonymous
https://github.com/evolution-cms/evolution/blob/2.0.x/core/src/Core.php#L1836
Anonymous
Возможно, что-то полезное можете подсмотреть там:
https://github.com/mnoskov/commerce/blob/master/install/assets/snippets/Order.tpl
К сожалению, не совсем то. Это сниппет. А мне бы именно использование метода
Anonymous
@webber_12 я правильно понимаю. Что при изменении параметров в одном из плагинов, они изменённые перебрасываются в следующий?(так должно быть)
Dmytro
@webber_12 я правильно понимаю. Что при изменении параметров в одном из плагинов, они изменённые перебрасываются в следующий?(так должно быть)
Да по логике так должно быть и когда то работало 🙂
Aliaksandr
@webber_12 я правильно понимаю. Что при изменении параметров в одном из плагинов, они изменённые перебрасываются в следующий?(так должно быть)
внутри плагина просто либо модифицируются те, что есть, либо добавляются новые
Aliaksandr
Aliaksandr
и сейчас при замене на старую строчку с array_merge так и работает
Aliaksandr
а в новой версии не работает - потому что ссылка теряется на $params
Pathologic
давно пора запретить потому что pr, которые ничего не исправляют и не добавляют
Aliaksandr
тогда у нас будет рево ))
Pathologic
ну в рево на самом деле таких проебов было в разы меньше
Pathologic
а у нас только с плагинами сколько косяков было
Dmytro
Ну дык я считаю что у нас очень мало багов при переезде с 1.4 на 2.0 и мы очень легко отделались
Dmytro
осталось плагины опять поченить и вроди как из важного все
Pathologic
лол
Pathologic
у нас были баги критичные
Dmytro
А так ну понятно херячить без тестирования
Dmytro
Ну были да но поправили считай за месяц после релиза 🙂
Dmytro
а с альфа до релиза это норм что критикалы
Dmytro
как бы в разработке
Pathologic
ну, так от того что альфу назвали релизом лучше не станет
Pathologic
а тестировать тут только ставить и ждать, что же поломается
Dmytro
Ну как бы стало очень много всего исправили + получили много багрепортов что позволило еще исправить
Dmytro
а тестировать тут только ставить и ждать, что же поломается
Ну при таких глобальных вещах как бы да
Pathologic
ну а раз тестировать мы не можем, то нужно хотя бы минимизировать поломки того, что всегда работало
Aliaksandr
сейчас народ потянется на 2.0 - найдем вам еще тележку багов ))) сидите там - ничего не замечаете ))
Pathologic
ага, дитто не работает, еформ не работает, шопкипер не работает
Aliaksandr
ну у каждого свои методы работы в эво, особенно когда единых нет, а все можно сделать двадцатью способами. сейчас точно работает то, что используется теми, кто уже использует 2.0 . Вполне вероятно, что когда туда массово придут другие - мы узнаем еще о десятках способов "все поломать" ))
Pathologic
так многие просто обновят потому что цифра больше и получат проблемы с сайтом
Aliaksandr
вот решился я поставить коммерс на 2.0 и сразу вместо расплывчатого "там ничего не работает" стало понятно: в основном работает, но иногда есть нюансы )
Pathologic
уже было такое когда была кнопка обновить на 2.0 альфа
Pathologic
с новыми сайтами конечно попроще будет
Anonymous
@webber_12 а когда вернули array_merge они начали прокидываться?
Aliaksandr
ну в любом случае без более -менее массово использования никто не сможет гарантировать что оно там работает. Многие нюансы, о которых большинство может даже не догадываться )
Anonymous
Pathologic
так ты же говорил там еще unset добавился
Aliaksandr
ансета в своей версии никакого не вижу
Pathologic
тогда ладно
Dark
ну у каждого свои методы работы в эво, особенно когда единых нет, а все можно сделать двадцатью способами. сейчас точно работает то, что используется теми, кто уже использует 2.0 . Вполне вероятно, что когда туда массово придут другие - мы узнаем еще о десятках способов "все поломать" ))
Есть у меня пара сайтов, с яматегами и модификаторами, которые вообще трогать страшно - освоил новое на свою голову тогда
Anonymous
https://github.com/evolution-cms/evolution/blob/1.4.x/manager/includes/document.parser.class.inc.php#L1937
В 1.4 он тоже есть
Aliaksandr
кроме этого я больше ничего не трогал в 2.0 - и плагины заработали в коммерсе
Aliaksandr
Anonymous
Блин правильно что у меня не работают
Anonymous
Я же дёргаю события без ссылочных параметров как в комерс.
Anonymous
И правильно что ломает foreach
Aliaksandr
без ссылочных работает и так и так по идее
Aliaksandr
тут проблемы именно в &$
Anonymous
foreach переписывает старыми параметрами новый массив. А array_merge склеивает и первый считается основным
Dmytro
О блин если вы почините буду очень благодарен а то никак не доберусь
Aliaksandr
да вернуть все взад и забыть)
Aliaksandr
что там чинить - взять то, что всегда работало и заменить этим то, что яма вписал не всегда рабочее ))
Михаил
забанить яму на веки вечные)
Anonymous
Aliaksandr
ну в былые времена оттуда вроде стянули тоже пару интересных фишек, с японской версии
Aliaksandr
но точно не помню что за они уже
Pathologic
кэш по $_GET наверное