Merge branch 'develop'

Author: nixel2007

README.md

@@ -1,6 +1,6 @@
 # BSL Language Server
 
-[![Build Status](https://travis-ci.org/1c-syntax/bsl-language-server.svg?branch=master)](https://travis-ci.org/1c-syntax/bsl-language-server)
+[![Build Status](https://travis-ci.org/1c-syntax/bsl-language-server.svg?branch=develop)](https://travis-ci.org/1c-syntax/bsl-language-server)
 [![JitPack](https://jitpack.io/v/1c-syntax/bsl-language-server.svg)](https://jitpack.io/#1c-syntax/bsl-language-server)
 [![Quality Gate](https://sonarcloud.io/api/project_badges/measure?project=1c-syntax_bsl-language-server&metric=alert_status)](https://sonarcloud.io/dashboard?id=1c-syntax_bsl-language-server)
 [![Maintainability](https://sonarcloud.io/api/project_badges/measure?project=1c-syntax_bsl-language-server&metric=sqale_rating)](https://sonarcloud.io/dashboard?id=1c-syntax_bsl-language-server)
@@ -17,10 +17,10 @@
 
 * Форматирование файла
 * Форматирование выбранного диапазона
-* Определение символов текущего файла (процедуры, функции, переменные, объявленные через `Перем`)
-* Сворачивание областей, блоков кода, процедур и функций, секций `#Использовать`
+* Определение символов текущего файла (области, процедуры, функции, переменные, объявленные через `Перем`)
 * Определение сворачиваемых областей - `#Область`, `#Если`, процедуры и функции, блоки кода
 * Диагностики
+* "Быстрые исправления" (quick fixes) для ряда диагностик
 * Запуск движка диагностик из командной строки
 
 ## Запуск из командной строки

docs/diagnostics/DeletingCollectionItem.md

@@ -0,0 +1,26 @@
+# Удаление элемента при обходе коллекции посредством оператора Для каждого ... Из ... Цикл
+
+Не следует непосредственно удалять элементы коллекции при ее обходе через синтаксическую конструкцию
+**Для каждого ... Из ... Цикл**. Т.к. при удалении элемента сдвигается индекс следующего элемента.
+
+Например:
+
+```bsl
+Для Каждого Элемент Из Коллекция Цикл
+   Коллекция.Удалить(Элемент)
+КонецЦикла;
+```
+
+Как вариант удаляйте элементы с конца:
+
+```bsl
+ТекущийИндекс = Числа.ВГраница();
+Пока ТекущийИндекс >= 0 Цикл
+    Если Числа[ТекущийИндекс] < 10 Тогда
+        Числа.Удалить(ТекущийИндекс);
+    КонецЕсли;
+    ТекущийИндекс = ТекущийИндекс – 1;
+КонецЦикла;
+```
+
+Источник: [1С:Программирование для начинающих. Разработка в системе "1С:Предприятие 8.3"](https://its.1c.ru/db/pubprogforbeginners#content:88:hdoc)

docs/diagnostics/ExtraCommas.md

@@ -0,0 +1,19 @@
+# Запятые без указания параметра в конце вызова метода
+
+Не следует указывать запятую в конце вызова метода без указания параметра. Это затрудняет восприятие и не несет важной информации.
+
+Плохо:
+
+```bsl
+Результат = Действие(П1, П2,,);
+```
+
+Хорошо:
+
+```bsl
+Результат = Действие(П1, П2);
+```
+
+Основание:
+Обязательные параметры подпадают под рекомендацию ["Соглащения о написании кода. Параметры процедур и функций. Пункт 7"](https://its.1c.ru/db/v8std#content:640:hdoc).
+Необязательные параметры попадают под принцип Бритва Оккама "Не следует множить сущности без необходимости", так как "висящая" запятая мало информативна.

docs/diagnostics/MagicNumber.md

@@ -0,0 +1,25 @@
+# Использование магического числа
+
+Магические числа - это любое число в вашем коде, которое не сразу становится очевидным без погружения в контекст.
+
+Плохо
+```Bsl
+Функция ПопадаетВИнтервал(Длительность)
+
+    Возврат Длительность < 10 * 60 * 60; 
+
+КонецФункции
+``` 
+
+Хорошо
+```Bsl
+Функция ПопадаетВИнтервал(ДлительностьВСекундах)
+    
+    МинутВЧасе      = 60;
+    СекундВМинуте   = 60;
+    СекундВЧасе     = СекундВМинуте * МинутВЧасе;
+    ЧасовВИнтервале = 10;
+    Возврат Длительность < ЧасовВИнтервале * СекундВЧасе; 
+
+КонецФункции
+``` 

docs/diagnostics/UsingServiceTag.md

@@ -3,9 +3,18 @@
 Диагностика отлавливает использование служебных тегов в комментариях. Список тегов:
 * TODO
 * FIXME
+* !!
+* @
+* MRG
+* ОТЛАДКА
+* ДЛЯ ОТЛАДКИ
+* КОНСТРУКТОР_ЗАПРОСА_С_ОБРАБОТКОЙ_РЕЗУЛЬТАТА
+* КОНСТРУКТОР_ДВИЖЕНИЙ_РЕГИСТРОВ
+* КОНСТРУКТОР_ПЕЧАТИ
+* КОНСТРУКТОР_ВВОДА_НА_ОСНОВАНИИ
 
 Список тегов так же можно расширить через настройки.
 
 ## Параметры
 
-* `serviceTags` - `Строка` - ключевые слово для поиска. По-умолчанию: "todo|fixme".
+* `serviceTags` - `Строка` - ключевые слово для поиска. По-умолчанию: "todo|fixme|!!|mrg|@|отладка|debug|для\s*отладки|(\{\{|\}\})КОНСТРУКТОР_|(\{\{|\}\})MRG".

docs/en/diagnostics/DeletingCollectionItem.md

@@ -0,0 +1,26 @@
+# Deleting an item when iterating through collection using the operator "For each ... In ... Do"
+
+Don't delete elements of collection %s when iterating through collection using the 
+operator **For each ... In ... Do**. Because it change index of next element.
+
+Example:
+
+```bsl
+For each Element In Collection Do
+   Collection.Delete(Element)
+EndDo;
+```
+
+Alternatively remove elements from the end:
+
+```bsl
+IndexOf = Numbers.UBound();
+While IndexOf >= 0 Do
+    If Numbers[IndexOf] < 10 Then
+        Numbers.Delete(IndexOf);
+    EndIf;
+    IndexOf = IndexOf – 1;
+EndDo;
+```
+
+Refference: [1С:Программирование для начинающих. Разработка в системе "1С:Предприятие 8.3" (RU)](https://its.1c.ru/db/pubprogforbeginners#content:88:hdoc)

docs/en/diagnostics/ExtraCommas.md

@@ -0,0 +1,19 @@
+# Commas without a parameter at the end of a method call
+
+Do not include a comma at the end of a method call without specifying a parameter. It is difficult to understand and does not carry important information.
+
+Bad:
+
+```bsl
+Result = Action (P1, P2 ,,);
+```
+
+Good:
+
+```bsl
+Result = Action (P1, P2);
+```
+
+Base:
+Required parameters fall under the recommendation of ["Code-writing conventions. Parameters of procedures and functions. Item 7"](https://its.1c.ru/db/v8std#content:640:hdoc).
+Not required parameters under the principle of Occam's Razor "Do not multiply entities without need", since the "hanging" comma is not very informative.

docs/en/diagnostics/MagicNumber.md

@@ -0,0 +1,25 @@
+# Using magic number
+
+Magic numbers are any number in your code that does not immediately become apparent without being immersed in context.
+
+Bad
+```Bsl
+Function GetsTheInterval(Duration)
+
+     Return Duration < 10 * 60 * 60;
+
+End Function
+```
+
+Good
+```Bsl
+Function GetsTheInterval (Duration in Seconds)
+    
+     MinutesHour     = 60;
+     SecondsMinute   = 60;
+     SecondsHour     = SecondsMinute * MinutesHour;
+     HoursIninterval = 10;
+     Return Duration < HoursWininterval * SecondsHour;
+
+End Function
+```

docs/en/diagnostics/UsingServiceTag.md

@@ -4,9 +4,18 @@ The diagnostic finds use of service tags in comments. Tags list:
 
 - TODO
 - FIXME
+- !!
+- @
+- MRG
+- ОТЛАДКА
+- ДЛЯ ОТЛАДКИ
+- КОНСТРУКТОР_ЗАПРОСА_С_ОБРАБОТКОЙ_РЕЗУЛЬТАТА
+- КОНСТРУКТОР_ДВИЖЕНИЙ_РЕГИСТРОВ
+- КОНСТРУКТОР_ПЕЧАТИ
+- КОНСТРУКТОР_ВВОДА_НА_ОСНОВАНИИ
 
 Tags list can be extended via options.
 
 ## Parameters
 
-- `serviceTags` - `String` - keyword for search. Bu default : "todo|fixme".
+- `serviceTags` - `String` - keyword for search. Bu default : "todo|fixme|!!|mrg|@|отладка|debug|для\s*отладки|(\{\{|\}\})КОНСТРУКТОР_|(\{\{|\}\})MRG".

docs/en/index.md

@@ -9,7 +9,7 @@
 
 [Language Server Protocol](https://microsoft.github.io/language-server-protocol/) implementation for 1C (BSL) - 1C:Enterprise 8 and [OneScript](http://oscript.io) languages.
 
-[English version](en/index.md)
+[Russian version](../index.md)
 
 - <a href="#capabilities">Capabilities</a>
 - <a href="#cli">Run from command line</a>
@@ -24,7 +24,10 @@
 
 - File formatting
 - Selected region formatting
+- Symbol definition for current file (regions, procedures, functions, variables, defined via `Var` keyword)
+- Folding regions definition `#Region`, `#If`, procedures and functions, code blocks
 - Diagnostics
+- Quick fixes for several diagnostics
 - Run diagnostics engine from command line
 
 <a id="cli"></a>
@@ -130,11 +133,13 @@ Used for code analysis to meet coding standards and search for possible errors.
 - [DeprecatedMessage - Restriction on the use of deprecated "Message" method](diagnostics/DeprecatedMessage.md)
 - [EmptyCodeBlock - Empty code block](diagnostics/EmptyCodeBlock.md)
 - [EmptyStatement - Empty Ssatement](diagnostics/EmptyStatement.md)
+- [ExtraCommas - Commas without a parameter at the end of a method call](diagnostics/ExtraCommas.md)
 - [FunctionShouldHaveReturn - Function must have Return statement](diagnostics/FunctionShouldHaveReturn.md)
 - [IfElseDuplicatedCodeBlockDiagnostic - Duplicated code blocks in If...Then...ElsIf...](diagnostics/IfElseDuplicatedCodeBlock.md)
 - [IfElseDuplicatedConditionDiagnostic - Duplicated conditions in If...Then...ElsIf...](diagnostics/IfElseDuplicatedCondition.md)
 - [IfElseIfEndsWithElse - Using If...Then...ElsIf... statement](diagnostics/IfElseIfEndsWithElse.md)
 - [LineLength - Line length restriction](diagnostics/LineLength.md)
+- [MagicNumber - Using magic number](diagnostics/MagicNumber.md)
 - [MethodSize - Method size restriction](diagnostics/MethodSize.md)
 - [NestedConstructorsInStructureDeclaration - Nested constructors with parameters in structure declaration](diagnostics/NestedConstructorsInStructureDeclaration.md)
 - [NestedStatements - Control flow statements should not be nested too deep](diagnostics/NestedStatements.md)