Merge pull request #1478 from 1c-syntax/fix/issue1452

Author: nixel2007

docs/diagnostics/CodeOutOfRegion.md

@@ -4,6 +4,12 @@
  :-: | :-: | :-: | :-: | :-: | :-: 
  `Дефект кода` | `BSL` | `Информационный` | `Да` | `1` | `standard` 
 
+## Параметры 
+
+ Имя | Тип | Описание | Значение по умолчанию 
+ :-: | :-: | :-- | :-: 
+ `checkUnknownModuleType` | `Строка` | ```Проверять неизвестные типы модулей``` | ```false``` 
+
 <!-- Блоки выше заполняются автоматически, не трогать -->
 ## Описание диагностики
 <!-- Описание диагностики заполняется вручную. Необходимо понятным языком описать смысл и схему работу -->
@@ -65,5 +71,7 @@
 ### Параметр конфигурационного файла
 
 ```json
-"CodeOutOfRegion": false
+"CodeOutOfRegion": {
+ "checkUnknownModuleType": "false"
+}
 ```

docs/en/diagnostics/CodeOutOfRegion.md

@@ -4,6 +4,12 @@
  :-: | :-: | :-: | :-: | :-: | :-: 
  `Code smell` | `BSL` | `Info` | `Yes` | `1` | `standard` 
 
+## Parameters 
+
+ Name | Type | Description | Default value 
+ :-: | :-: | :-- | :-: 
+ `checkUnknownModuleType` | `String` | ```Check unknown module type``` | ```false``` 
+
 <!-- Блоки выше заполняются автоматически, не трогать -->
 ## Description
 
@@ -69,5 +75,7 @@ RU | EN
 ### Parameter for config
 
 ```json
-"CodeOutOfRegion": false
+"CodeOutOfRegion": {
+ "checkUnknownModuleType": "false"
+}
 ```

src/main/java/com/github/_1c_syntax/bsl/languageserver/diagnostics/CodeOutOfRegionDiagnostic.java

@@ -25,6 +25,7 @@
 import com.github._1c_syntax.bsl.languageserver.context.symbol.RegionSymbol;
 import com.github._1c_syntax.bsl.languageserver.diagnostics.metadata.DiagnosticCompatibilityMode;
 import com.github._1c_syntax.bsl.languageserver.diagnostics.metadata.DiagnosticMetadata;
+import com.github._1c_syntax.bsl.languageserver.diagnostics.metadata.DiagnosticParameter;
 import com.github._1c_syntax.bsl.languageserver.diagnostics.metadata.DiagnosticScope;
 import com.github._1c_syntax.bsl.languageserver.diagnostics.metadata.DiagnosticSeverity;
 import com.github._1c_syntax.bsl.languageserver.diagnostics.metadata.DiagnosticTag;
@@ -56,13 +57,20 @@
  compatibilityMode = DiagnosticCompatibilityMode.COMPATIBILITY_MODE_8_3_1
 )
 public class CodeOutOfRegionDiagnostic extends AbstractVisitorDiagnostic {
+ private static final boolean CHECK_UNKNOWN_MODULE_TYPE = false;
  private final List<Range> regionsRanges = new ArrayList<>();
 
+ @DiagnosticParameter(
+ type = String.class,
+ defaultValue = "" + CHECK_UNKNOWN_MODULE_TYPE
+ )
+ private boolean checkUnknownModuleType = CHECK_UNKNOWN_MODULE_TYPE;
+
  @Override
  public ParseTree visitFile(BSLParser.FileContext ctx) {
 
  // Для неизвестных модулей не будем требовать нахождения кода в области
- if (documentContext.getModuleType() == ModuleType.UNKNOWN) {
+ if (documentContext.getModuleType() == ModuleType.UNKNOWN && !checkUnknownModuleType) {
  return ctx;
  }
 

src/main/resources/com/github/_1c_syntax/bsl/languageserver/configuration/parameters-schema.json

@@ -52,6 +52,14 @@
  "object"
  ],
  "title": "Code out of region",
+ "properties": {
+ "checkUnknownModuleType": {
+ "description": "Check unknown module type",
+ "default": "false",
+ "type": "string",
+ "title": "Check unknown module type"
+ }
+ },
  "$id": "#/definitions/CodeOutOfRegion"
  },
  "CognitiveComplexity": {

src/main/resources/com/github/_1c_syntax/bsl/languageserver/diagnostics/CodeOutOfRegionDiagnostic_en.properties

@@ -1,2 +1,3 @@
 diagnosticMessage=Move code to region
 diagnosticName=Code out of region
+checkUnknownModuleType=Check unknown module type

src/main/resources/com/github/_1c_syntax/bsl/languageserver/diagnostics/CodeOutOfRegionDiagnostic_ru.properties

@@ -1,2 +1,3 @@
 diagnosticMessage=Переместите код в область
 diagnosticName=Код расположен вне области
+checkUnknownModuleType=Проверять неизвестные типы модулей

src/test/java/com/github/_1c_syntax/bsl/languageserver/diagnostics/CodeOutOfRegionDiagnosticTest.java

@@ -78,6 +78,13 @@ void testNoRegions() {
  void testNoRegionsUnknown() {
  var diagnostics = getDiagnostics("CodeOutOfRegionDiagnosticNoRegions", ModuleType.UNKNOWN);
  assertThat(diagnostics).isEmpty();
+
+ var configuration = diagnosticInstance.getInfo().getDefaultConfiguration();
+ configuration.put("checkUnknownModuleType", true);
+ diagnosticInstance.configure(configuration);
+
+ diagnostics = getDiagnostics("CodeOutOfRegionDiagnosticNoRegions", ModuleType.UNKNOWN);
+ assertThat(diagnostics).isNotEmpty();
  }
 
  @Test