Штормовскими предупреждениями - их можно настраивать уровни. Например на одном проекте тебе нужно чтобы 100% никто не писал API в шаблоне - ставишь показывать как ошибку фатальную, он красным подсвечивает, а можно и совсем отключить
Так вообще идеально. Интересно - можно ли через API частично глушить проверку вызовов нестатичных методов как статичных (например CIBlockElement::GetList, который в ядре не отмечен как static, но по сути им является - шторм всегда подсвечивает желтым)
Про это знаю, интересно глушить проверку именно конкретных функций, именно силами плагина - чтобы не надо было настраивать под каждый проект настройки инспекции, и при этом не отключать проверку совсем, потому что в рамках проекта на битриксе часто применяю сторонние библиотеки, где с определением функций все в порядке
Автовынос - это сложно, можно вынести часть, можно вынести лишее. Может перестать работать, ведь код может быть разбросан по всему шаблону. Ругаться можно на использование GetList, его выделять уветом и предлагать выделить код и вынести в резалтмодифер. Такое решение должен принимать кодер, ему будет легко и удобно вынести код.
ЗЫ: цепочка обсуждения уже вырвиглазная, не понять кто на что ответил
Idees
Если только ненавязчивые предупреждения. Стиль битрикса не располагает к идеальному коду, даже в их шаблонах где-то видел вызовы API.
Штормовскими предупреждениями - их можно настраивать уровни. Например на одном проекте тебе нужно чтобы 100% никто не писал API в шаблоне - ставишь показывать как ошибку фатальную, он красным подсвечивает, а можно и совсем отключить
Так вообще идеально.
Интересно - можно ли через API частично глушить проверку вызовов нестатичных методов как статичных (например CIBlockElement::GetList, который в ядре не отмечен как static, но по сути им является - шторм всегда подсвечивает желтым)
Крайний правый нижний угол -> жмешь на иконку. Там всплывает http://d.pr/i/Mndm потом жмешь Configure Inspections -> там ищешь static иснимаешьгалочку с нужного (или уровень регулируешь)
Про это знаю, интересно глушить проверку именно конкретных функций, именно силами плагина - чтобы не надо было настраивать под каждый проект настройки инспекции, и при этом не отключать проверку совсем, потому что в рамках проекта на битриксе часто применяю сторонние библиотеки, где с определением функций все в порядке
Не, так не получится я думаю
ну если ругаться, тогда уж былобы удобно выделить кусок, и в контекстном меню нажать "Move to result_modifier.php"
В целом идея неплохая, но мне кажется реализовать автоматический вынос будет сложно.
Выносить выделеный кусок only
Михаил это действительно сложно?
Автовынос - это сложно, можно вынести часть, можно вынести лишее. Может перестать работать, ведь код может быть разбросан по всему шаблону. Ругаться можно на использование GetList, его выделять уветом и предлагать выделить код и вынести в резалтмодифер. Такое решение должен принимать кодер, ему будет легко и удобно вынести код.
ЗЫ: цепочка обсуждения уже вырвиглазная, не понять кто на что ответил