Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

Ajax bug in banlist plug #929

Closed
Dayver opened this Issue · 2 comments

2 participants

@Dayver
Collaborator

If the list of records, prohibition from one record to change something and click "Update" after you press (send Ajax request to change the record and get an answer in the form of an updated list), in the shown buttons to update the list do not work


Если в списке записей-запретов у одной записи что-то изменить и нажать "Обновить" то после нажатия(отправки аякс запроса на изменение записи и получения ответа в виде обновленного списка), в показанном списке кнопки обновить уже не работают

@trustmaster
Owner

Works fine for me.

@Dayver
Collaborator

There is a bug in Opera & Firefox. In Crome all is fine.


Баг с аяксом в банлисте в опере и лисе ... хром лишен бага ... все стандартное с чистой установки 099 ... и вот я думаю почему - банлист единственное коробочное расширение в котором есть список в котором правка происходит по отдельности для каждой строки (да и в ядре тоже такого уже не осталось) ..... то есть каждая строка этого списка это отдельная форма ... в результате когда мы заходим на страницу банлиста при $(document).ready с помощью function bindHandlers автоматом навешивается обработчик на каждую форму с классом ajax .... но вот после отправки одной из таких форм в возвращенном результате присутствует куча новых форм для которых bindHandlers уже не выполняется и соответственно обработчик не вешается (почему хром это делает а другие нет - хз) .... меня долго удивляло почему в таком отвеченном аяксом куске разметки обработчик формы не навешивается а обработчик ссылок (в данном случае "удалить") срабатывает .... но все оказалось проще в ссылках "удалить" все прописано жестко в ссылке средством атрибута onclick

потому решений данного бага несколько:
1) так же как и с ссылками навешивать обработчик жестко на каждую форму средством атрибута onsubmit
2) переделать плуг банлиста так что редактирование (кнопка "Обновить") будет одной для нескольких рядов сразу нежели для каждого ряда отдельно (чем сделать более удобно и правильно - как уже везде это сделано в движке)
3) поправить base.js + в ответе вызывалась функция ....... так что бы при получении аякс ответа опять инициировалось навешивание обработчика на формы с классом ajax

@trustmaster trustmaster was assigned
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.