#498 Реализована проверка OptionalParameterBeforeRequired

[VAGoncharov]

Что сделано

• Реализована проверка "Необязательный параметр расположен перед обязательным"

Чек-лист

Общее:

• [x] ветка PR обновлена из master и нет конфликтов
• [x] Тесты-кейсы, JUnit тесты правильного и неправильного состояния
• [x] Измененные Вами исходники отформатированы в соответствии с конвенцией
• [x] Авто-аудит (SonarQube и CheckStyle) пройден, покрытие кода хорошее, ошибок нет, плохой код устранен
• [x] Добавлена запись в ИСТОРИЮ ИЗМЕНЕНИЯ, включаемая в пользовательскую документацию плагина

Если применимо:

• [ ] Пользовательская документация на доп.инструменты написана (на русском)
• [x] Описание проверок - на двух языках

Закрываемые задачи

Closes #498

@1C-Company @marmyshev прошу сделать аудит

[marmyshev]

@VAGoncharov на всякий случай глянь в com.e1c.v8codestyle.bsl.check.MethodTooManyPramsCheck чтобы не было пересечений по функциональности (там что-то на эту же тему было)

[VAGoncharov]

@VAGoncharov на всякий случай глянь в com.e1c.v8codestyle.bsl.check.MethodTooManyPramsCheck чтобы не было пересечений по функциональности (там что-то на эту же тему было)

Посмотрел. И там и тут проверяются параметры методов, поэтому есть пересечение по подписке и способу обхода коллекции параметров. Но семантически проверки разные, как и их коды. Предлагаешь объединить в одну?

[sonarcloud[bot]]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

94.9% Coverage
0.0% Duplication