Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

изменена логика получения имени метода, изменено имя метода в переопределяемых общих модулях #43

Merged
merged 1 commit into from
Apr 16, 2024
Merged

изменена логика получения имени метода, изменено имя метода в переопределяемых общих модулях #43

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 10 additions & 2 deletions src/CommonModules/__МодификацияКонфигурацииПереопределяемый/Ext/Module.bsl
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -540,14 +540,22 @@
ИмяФормы = Форма.ИмяФормы;
МассивПодимен = __СтроковыеФункцииСлужебныйКлиентСервер.РазложитьСтрокуВМассивПодстрок(ИмяФормы, ".");

Если МассивПодимен.Количество() = 0 Тогда
КолЭлементов = МассивПодимен.Количество();
Если КолЭлементов = 0 Тогда
Возврат Неопределено;
КонецЕсли;

ПостфиксМодуля = МассивПодимен.Получить(0);

//Удалим класс

Check notice on line 550 in src/CommonModules/__МодификацияКонфигурацииПереопределяемый/Ext/Module.bsl

View check run for this annotation

sonar.openbsl.ru qa-bot / 1C: Project SubSystems Library Sonarqube Results

src/CommonModules/__МодификацияКонфигурацииПереопределяемый/Ext/Module.bsl#L550 

Между символами комментария '//' и самим текстом комментария должен быть пробел.
МассивПодимен.Удалить(0);
Если КолЭлементов > 2 Тогда

Check warning on line 552 in src/CommonModules/__МодификацияКонфигурацииПереопределяемый/Ext/Module.bsl

View check run for this annotation

sonar.openbsl.ru qa-bot / 1C: Project SubSystems Library Sonarqube Results

src/CommonModules/__МодификацияКонфигурацииПереопределяемый/Ext/Module.bsl#L552 

Создайте константу с понятным названием, присвойте ей значение "2" и используйте эту константу вместо магического числа.
// Удалим слово "Форма"
МассивПодимен.Удалить(1);
КонецЕсли;

ИмяМодуля = "__МодификацияКонфигурацииПереопределяемый" + ПостфиксМодуля;
ИмяПроцедуры = СтрЗаменить(ИмяФормы, ".", "") + ИмяПроцедурыВход;
ИмяПроцедуры = СтрСоединить(МассивПодимен, "") + ИмяПроцедурыВход;
ИмяМетода = ИмяМодуля + "." + ИмяПроцедуры;

Возврат ИмяМетода;
Expand Down
2 changes: 1 addition & 1 deletion src/CommonModules/__МодификацияКонфигурацииПереопределяемыйДокумент/Ext/Module.bsl
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@

// Процедура носит ознакомительный характер по работе с подсистемой "__РаботаСФормами", не переносить.
//
Процедура ДокументДемоРаботаСФормамиФормаФормаДокументаПриСозданииНаСервере(Форма, Отказ, СтандартнаяОбработка, ДополнительныеПараметры = Неопределено) Экспорт
Процедура ДемоРаботаСФормамиФормаДокументаПриСозданииНаСервере(Форма, Отказ, СтандартнаяОбработка, ДополнительныеПараметры = Неопределено) Экспорт

Check warning on line 31 in src/CommonModules/__МодификацияКонфигурацииПереопределяемыйДокумент/Ext/Module.bsl

View check run for this annotation

sonar.openbsl.ru qa-bot / 1C: Project SubSystems Library Sonarqube Results

src/CommonModules/__МодификацияКонфигурацииПереопределяемыйДокумент/Ext/Module.bsl#L31 

Необходимо добавить описание всех параметров метода

Check warning on line 31 in src/CommonModules/__МодификацияКонфигурацииПереопределяемыйДокумент/Ext/Module.bsl

View check run for this annotation

sonar.openbsl.ru qa-bot / 1C: Project SubSystems Library Sonarqube Results

src/CommonModules/__МодификацияКонфигурацииПереопределяемыйДокумент/Ext/Module.bsl#L31 

Длина строки 146 превышает максимально допустимую 120

// Демо-пример можно запустить на конфигурации БСП, в каркасной конфигурации отсутствуют необходимые _Демо объекты
Возврат;
Expand Down
Loading