Merge remote-tracking branch 'origin/develop' into ref-index-fix-2832

Author: artbear

.github/workflows/sentry.yml

@@ -0,0 +1,19 @@
+name: Sentry
+on:
+  push:
+    branches-ignore:
+      - "translations_*"
+jobs:
+  sentry:
+    name: Sentry
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v2
+      - name: Create Sentry release
+        uses: getsentry/action-release@v1
+        env:
+          SENTRY_AUTH_TOKEN: ${{ secrets.SENTRY_AUTH_TOKEN }}
+          SENTRY_ORG: ${{ secrets.SENTRY_ORG }}
+          SENTRY_PROJECT: ${{ secrets.SENTRY_PROJECT }}
+        with:
+          environment: production

build.gradle.kts

@@ -9,17 +9,18 @@ plugins {
     signing
     id("org.cadixdev.licenser") version "0.6.1"
     id("org.sonarqube") version "3.4.0.2513"
-    id("io.freefair.lombok") version "6.4.3.1"
-    id("io.freefair.javadoc-links") version "6.4.3.1"
-    id("io.freefair.javadoc-utf-8") version "6.4.3.1"
-    id("io.freefair.aspectj.post-compile-weaving") version "6.4.3.1"
-    id("io.freefair.maven-central.validate-poms") version "6.4.3.1"
-    id("me.qoomon.git-versioning") version "6.1.6"
+    id("io.freefair.lombok") version "6.5.0.3"
+    id("io.freefair.javadoc-links") version "6.5.0.3"
+    id("io.freefair.javadoc-utf-8") version "6.5.0.3"
+    id("io.freefair.aspectj.post-compile-weaving") version "6.5.0.3"
+    id("io.freefair.maven-central.validate-poms") version "6.5.0.3"
+    id("me.qoomon.git-versioning") version "6.2.0"
     id("com.github.ben-manes.versions") version "0.42.0"
     id("org.springframework.boot") version "2.6.7"
-    id("io.spring.dependency-management") version "1.0.11.RELEASE"
+    id("io.spring.dependency-management") version "1.0.12.RELEASE"
     id("io.github.1c-syntax.bslls-dev-tools") version "0.7.0"
     id("ru.vyarus.pom") version "2.2.1"
+    id("com.gorylenko.gradle-git-properties") version "2.4.1"
     id("io.codearte.nexus-staging") version "0.30.0"
 }
 
@@ -51,6 +52,12 @@ val isSnapshot = gitVersioning.gitVersionDetails.refType != GitRefType.TAG
 
 val languageToolVersion = "5.6"
 
+dependencyManagement {
+    imports {
+        mavenBom("io.sentry:sentry-bom:6.2.1")
+    }
+}
+
 dependencies {
 
     // RUNTIME
@@ -72,7 +79,9 @@ dependencies {
         exclude("org.glassfish", "javax.json")
     }
     api("com.github.1c-syntax", "utils", "0.4.0")
-    api("com.github.1c-syntax", "mdclasses", "0.9.5")
+    api("com.github.1c-syntax", "mdclasses", "0.10.0")
+    api("io.github.1c-syntax", "bsl-common-library", "0.3.0")
+    api("io.github.1c-syntax", "supportconf", "0.1.1")
 
     // JLanguageTool
     implementation("org.languagetool", "languagetool-core", languageToolVersion)
@@ -101,6 +110,10 @@ dependencies {
     // SARIF serialization
     implementation("com.contrastsecurity", "java-sarif", "2.0")
 
+    // Sentry
+    implementation("io.sentry:sentry-spring-boot-starter")
+    implementation("io.sentry:sentry-logback")
+
     // COMPILE
 
     // stat analysis

docs/diagnostics/IncorrectUseLikeInQuery.md

@@ -71,4 +71,4 @@
 * Полезная информация: [Отказ от использования модальных окон](https://its.1c.ru/db/metod8dev#content:5272:hdoc)
 * Источник: [Cognitive complexity, ver. 1.4](https://www.sonarsource.com/docs/CognitiveComplexity.pdf) -->
 
-- Источник: [Стандарт: Особенности использования в запросах оператора ПОДОБНО](https://its.1c.ru/db/v8std/content/726/hdoc?ysclid=l3g3fkmxsx)
+- Источник: [Стандарт: Особенности использования в запросах оператора ПОДОБНО](https://its.1c.ru/db/v8std/content/726/hdoc?ysclid=l3g3fkmxsx)

docs/diagnostics/RewriteMethodParameter.md

@@ -0,0 +1,44 @@
+# Перезапись параметров метода (RewriteMethodParameter)
+
+<!-- Блоки выше заполняются автоматически, не трогать -->
+## Описание диагностики
+<!-- Описание диагностики заполняется вручную. Необходимо понятным языком описать смысл и схему работу -->
+Иногда разработчик пишут функции таким образом, когда аргументы функции перезаписываются сразу на входе в функцию/процедуру. 
+
+Такое поведение вводит в заблуждение других разработчиков, которые вызывают подобные функции/процедуры. 
+Эти функции нужно исправить. Например, убрать параметры, преобразовав их в локальные переменные.
+
+## Примеры
+<!-- В данном разделе приводятся примеры, на которые диагностика срабатывает, а также можно привести пример, как можно исправить ситуацию -->
+Подозрительный код
+```bsl
+Процедура Конфигуратор(Знач СтрокаПодключения, Знач Пользователь = "", Знач Пароль = "") Экспорт
+  СтрокаПодключения = "/F""" + КаталогБазы + """"; // Здесь
+...
+КонецФункции
+```
+
+Исправленный код
+```bsl
+Процедура Конфигуратор(Знач КаталогБазы, Знач Пользователь = "", Знач Пароль = "") Экспорт
+  СтрокаПодключения = "/F""" + КаталогБазы + """"; // Здесь
+...
+КонецФункции
+```
+или
+```bsl
+Процедура Конфигуратор(Знач КаталогБазы, Знач Пользователь = "", Знач Пароль = "") Экспорт
+  Если Не ПустаяСтрока(КаталогБазы) Тогда
+   НоваяСтрокаПодключения = "/F""" + КаталогБазы + """";
+  Иначе
+   НоваяСтрокаПодключения = СтрокаПодключения;
+  КонецЕсли;
+...
+КонецФункции
+```
+
+## Источники
+<!-- Необходимо указывать ссылки на все источники, из которых почерпнута информация для создания диагностики -->
+<!-- Примеры источников
+
+* [PVS-Studio V763. Parameter is always rewritten in function body before being used](https://pvs-studio.com/ru/docs/warnings/v6023)

docs/en/diagnostics/CrazyMultilineString.md

@@ -1,7 +1,8 @@
 # Crazy multiline literals (CrazyMultilineString)
 
 <!-- Блоки выше заполняются автоматически, не трогать -->
-## Описание диагностики
+## Description
+
 <!-- Описание диагностики заполняется вручную. Необходимо понятным языком описать смысл и схему работу -->
 
 In source text, multi-line constants can be initialized in two ways:

docs/en/diagnostics/IncorrectLineBreak.md

@@ -1,7 +1,7 @@
 # Incorrect expression line break (IncorrectLineBreak)
 
 <!-- Блоки выше заполняются автоматически, не трогать -->
-## Описание диагностики
+## Description
 <!-- Описание диагностики заполняется вручную. Необходимо понятным языком описать смысл и схему работу -->
 
 Long arithmetic expressions are carried as follows: one entry can contain more than one operand; when wrapping, operation characters are written at the beginning of the line (and not at the end of the previous line); operands on a new line are preceded by standard indentation, or they are aligned to the beginning of the first operand, regardless of the operation signs.

docs/en/diagnostics/IncorrectUseLikeInQuery.md

@@ -1,7 +1,7 @@
 # Incorrect use of 'LIKE' (IncorrectUseLikeInQuery)
 
 <!-- Блоки выше заполняются автоматически, не трогать -->
-## Описание диагностики
+## Description
 <!-- Описание диагностики заполняется вручную. Необходимо понятным языком описать смысл и схему работу -->
 
 When using the operator `LIKE` in the query text, it is allowed to use only
@@ -70,4 +70,4 @@ Query.SetParameter("NameTemplate", "FU_");
 * Полезная информация: [Отказ от использования модальных окон](https://its.1c.ru/db/metod8dev#content:5272:hdoc)
 * Источник: [Cognitive complexity, ver. 1.4](https://www.sonarsource.com/docs/CognitiveComplexity.pdf) -->
 
-- [Standard. Features of use in operator requests LIKE (RU)](https://its.1c.ru/db/v8std/content/726/hdoc?ysclid=l3g3fkmxsx)
+- [Standard. Features of use in operator requests LIKE (RU)](https://its.1c.ru/db/v8std/content/726/hdoc?ysclid=l3g3fkmxsx)

docs/en/diagnostics/LogicalOrInTheWhereSectionOfQuery.md

@@ -1,7 +1,8 @@
 # Using a logical "OR" in the "WHERE" section of a query (LogicalOrInTheWhereSectionOfQuery)
 
 <!-- Блоки выше заполняются автоматически, не трогать -->
-## Описание диагностики
+## Description
+
 <!-- Описание диагностики заполняется вручную. Необходимо понятным языком описать смысл и схему работу -->
 Не следует использовать `ИЛИ` в секции `ГДЕ` запроса. Это может привести к тому, что СУБД не сможет использовать 
 индексы таблиц и будет выполнять сканирование, что увеличит время работы запроса и вероятность возникновения блокировок. 

docs/en/diagnostics/MultilineStringInQuery.md

@@ -1,7 +1,7 @@
 # Multi-line literal in query (MultilineStringInQuery)
 
 <!-- Блоки выше заполняются автоматически, не трогать -->
-## Описание диагностики
+## Description
 <!-- Описание диагностики заполняется вручную. Необходимо понятным языком описать смысл и схему работу -->
 
 Multi-line literals are rarely used in query texts, mostly these are error results due to an incorrect number of double quotes.

docs/en/diagnostics/RewriteMethodParameter.md

@@ -0,0 +1,16 @@
+# Rewrite method parameter (RewriteMethodParameter)
+
+<!-- Блоки выше заполняются автоматически, не трогать -->
+## Description
+<!-- Описание диагностики заполняется вручную. Необходимо понятным языком описать смысл и схему работу -->
+
+## Examples
+<!-- В данном разделе приводятся примеры, на которые диагностика срабатывает, а также можно привести пример, как можно исправить ситуацию -->
+
+## Sources
+<!-- Необходимо указывать ссылки на все источники, из которых почерпнута информация для создания диагностики -->
+<!-- Примеры источников
+
+* Источник: [Стандарт: Тексты модулей](https://its.1c.ru/db/v8std#content:456:hdoc)
+* Полезная информация: [Отказ от использования модальных окон](https://its.1c.ru/db/metod8dev#content:5272:hdoc)
+* Источник: [Cognitive complexity, ver. 1.4](https://www.sonarsource.com/docs/CognitiveComplexity.pdf) -->