MODX маминой подруги - Исправление бага с непослушным menuindex
Привет сообществу!
Из-за сильной занятости над проектами я и не заметил, что еще в Апреле был смерджен мой PR по исправлению бага с непослушным menuindex.
О баге на GitHub сообщил @Руслан Алеев, но о нём было известно и до этого. По крайней мере я точно знал, потому как очень часто клиенты которые пользуются панелью управления сообщали, что при каких-то мистических обстоятельствах поле menuindex не изменяется или ведет себя странно. Выяснилось, что происходит это в тех обстоятельствах, когда ресурс открыт пользователем на редактирование и он находясь там же сортирует ресурсы в дереве ресурсов.
Пару слов о сортировке в MODX
Стоит еще заметить, что при сортировке одного ресурса (если речь идёт о перетаскивании в дереве) процессор затрагивает и вносит изменения не только сортируемому ресурсу, но и его соседним ресурсам, так сказать сдвигает их при необходимости.
Пример ситуации для того, чтобы вам было ясно почему MODX так себя ведет. Имеем 3 ресурса, в скобках menuindex:
Исправление бага с непослушным menuindex
Теперь при сортировке ресурсов в дереве, если открыт ресурс на редактирование, его значение будет обновляться в форме редактирования.
Как итог, баг исправлен, PR смерджен, Я молодец.
Более детально можно посмотреть по ссылкам:
Сам PR и Issue
☕ Угостить чашкой кофе
Из-за сильной занятости над проектами я и не заметил, что еще в Апреле был смерджен мой PR по исправлению бага с непослушным menuindex.
О баге на GitHub сообщил @Руслан Алеев, но о нём было известно и до этого. По крайней мере я точно знал, потому как очень часто клиенты которые пользуются панелью управления сообщали, что при каких-то мистических обстоятельствах поле menuindex не изменяется или ведет себя странно. Выяснилось, что происходит это в тех обстоятельствах, когда ресурс открыт пользователем на редактирование и он находясь там же сортирует ресурсы в дереве ресурсов.
Пару слов о сортировке в MODX
Процессор сортировки ресурсов в дереве
Стоит еще заметить, что при сортировке одного ресурса (если речь идёт о перетаскивании в дереве) процессор затрагивает и вносит изменения не только сортируемому ресурсу, но и его соседним ресурсам, так сказать сдвигает их при необходимости.
Пример ситуации для того, чтобы вам было ясно почему MODX так себя ведет. Имеем 3 ресурса, в скобках menuindex:
- Ресурс (0)
- Ресурс (1)
- Ресурс (2)
Исправление бага с непослушным menuindex
Changing the menuindex of the updating resource after sorting in tree
Теперь при сортировке ресурсов в дереве, если открыт ресурс на редактирование, его значение будет обновляться в форме редактирования.
Как итог, баг исправлен, PR смерджен, Я молодец.
Более детально можно посмотреть по ссылкам:
Сам PR и Issue
☕ Угостить чашкой кофе
Комментарии: 33
Авторизуйтесь или зарегистрируйтесь, чтобы оставлять комментарии.
Еще смущает то, что версии для 2-ой ветки не заканчиваются, вроде планировалась MODX 2.7.0 быть финальной, потом 2.7.1, теперь 2.7.2 :) И из-за этого PR часто висят в неопределенном состоянии.
Хотя по поводу 2.7.* они видимо подумали что логичнее было бы исправить ошибки которые возникли после 2.7.0
С другой стороны правильно вроде делают, чтобы новые PR были уже после рефакторинга, им будет легче чтобы с конфликтами не возиться
Я не согласен! Человек захотел написать об этом заметку, почему нет? Сообщество итак редко пополняется, а часто бывают заметки, которые просто трудно читать из-за корявого форматирования. А тут вам и оформление, и гифка — читать приятно, короче. К тому-же, если это мотивирует Баху, то блин, пусть хоть каждую неделю пишет!
P.S. Если взять за основу психологическую модель «Треугольник Карпмана», то оч забавно наблюдать, как притеснители, высказывая своё недовольство и получив жёсткий ответ, становятся в роль жертвы…
Так держать!
Спасибо за твой труд, за вклад в MODX 3, за статьи и за помощь, в которой ты не отказываешь!
А то че это ты, 24 строчки добавил и думаешь что тебе теперь все можно?
Я вообще был бы только за то, чтобы каждое более-менее важное изменение или починка бага были так подробно расписаны — это позволяет лучше понять как и что работает в системе.
За статью спасибо — было интересно читать.
Пиши почаще — напоминает о необходимости общего вклада.
А так — у данного сообщества есть свой модератор, и если он решает когда статья или дополнение не соответствует правилам. Он может диктовать правила, а не кто-то с «улицы». Кто-то с улицы может сделать своё дополнение, написать свою статью, создать своё дополнение или своё сообщество в конце концов. А лезть к другим с осуждением…
А подстраиваться под мнение и ожидания каждого человека — это гнаться за ветром пытаясь его поймать… Угодить всем нельзя и смысла в этом никакого.
Это мысли вслух. Может быть зря я их сюда написал — не хочется очередного бессмысленного холивара.
Ничего особенного, флуд да и только, я вспоминал про твой снег)