Skip to content

Commit

Permalink
Фикс замечаний
Browse files Browse the repository at this point in the history
  • Loading branch information
alkoleft committed Dec 11, 2023
1 parent 280b083 commit 95dd87c
Show file tree
Hide file tree
Showing 12 changed files with 30 additions and 52 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -69,9 +69,9 @@

Функция НовыйПустойОбработчик(КоличествоПараметров = 1) Экспорт

МаксимальноеДопустивоеЧислоПараметров = 3;
МаксимальноеДопустимоеЧислоПараметров = 3;

Если КоличествоПараметров >= 1 И КоличествоПараметров <= МаксимальноеДопустивоеЧислоПараметров Тогда
Если КоличествоПараметров >= 1 И КоличествоПараметров <= МаксимальноеДопустимоеЧислоПараметров Тогда
Возврат Новый ОписаниеОповещения("ПустойОбработчик" + КоличествоПараметров, ЭтотОбъект);
Иначе
ВызватьИсключение СтрШаблон("Пустой обработчик с %1 параметрами не реализован", КоличествоПараметров);
Expand Down
4 changes: 2 additions & 2 deletions exts/yaxunit/src/CommonModules/ЮТЗапросы/Module.bsl
Original file line number Diff line number Diff line change
Expand Up @@ -179,8 +179,8 @@
//
// Возвращаемое значение:
// Структура - Описание запроса:
// * ИмяТаблицы - Строка -
// * ВыбираемыеПоля - Массив из Строка
// * ИмяТаблицы - Строка - Имя таблицы, из которой нужно получить данные
// * ВыбираемыеПоля - Массив из Строка - Выражения выбираемых полей
// * КоличествоЗаписей - Число, Неопределено - Ограничение количества выбираемых записей
// * Условия - Массив из Строка - Коллекция выражений условий, которые будут объединены через `И`
// * ЗначенияПараметров - Структура - Набор параметров запроса
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@
ТипПараметра = ТипЗнч(ВыбираемыеПоля);

Если ТипПараметра = Тип("Строка") Тогда
ВыбираемыеПоля = ЮТОбщий.РазложитьСтрокуВМассивПодстрок(ВыбираемыеПоля, ",", Истина);
ВыбираемыеПоля = ЮТОбщий.РазложитьСтрокуВМассивПодстрок(ВыбираемыеПоля, ",");
ТипПараметра = Тип("Массив");
КонецЕсли;

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -277,13 +277,14 @@

КонтекстИсполнения = ЮТФабрика.КонтекстИсполнения(Набор.Режим);

ОшибкаКонтекста = Неопределено;
Если КонтекстыПриложения.Найти(Набор.Режим) = Неопределено Тогда
ОшибкаКонтекста = "Неподдерживаемый режим запуска";
ИначеЕсли КонтекстыМодуля.Найти(Набор.Режим) = Неопределено Тогда
ОшибкаКонтекста = "Модуль не доступен в этом контексте";
ИначеЕсли КонтекстИсполнения <> КонтекстыИсполнения.Сервер И КонтекстИсполнения <> КонтекстыИсполнения.Клиент Тогда
ОшибкаКонтекста = "Неизвестный контекст/режим исполнения";
Иначе
ОшибкаКонтекста = Неопределено;
КонецЕсли;

Если ОшибкаКонтекста <> Неопределено Тогда
Expand Down Expand Up @@ -405,20 +406,4 @@

КонецПроцедуры

Функция НаборыТестовРежима(ТестовыйМодуль, ОжидаемыйРежимИсполнения)

Результат = Новый Массив();

Для Каждого Набор Из ТестовыйМодуль.НаборыТестов Цикл

Если Набор.Выполнять И ЮТФабрика.КонтекстИсполнения(Набор.Режим) = ОжидаемыйРежимИсполнения Тогда
Результат.Добавить(Набор);
КонецЕсли;

КонецЦикла;

Возврат Результат;

КонецФункции

#КонецОбласти
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@

Процедура ИнициализироватьКонтекст() Экспорт

ЮТДанныеКонтекста = ЮТКонтекст.ОписаниеКонтекста();
ЮТДанныеКонтекста = ЮТКонтекст.ОписаниеКонтекста(); // BSLLS:UnusedLocalVariable-off

КонецПроцедуры

Expand All @@ -32,7 +32,7 @@

Процедура УдалитьКонтекст() Экспорт

ЮТДанныеКонтекста = Неопределено;
ЮТДанныеКонтекста = Неопределено; // BSLLS:UnusedLocalVariable-off

КонецПроцедуры

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -410,11 +410,13 @@
ВызватьИсключение "Метод записи лога не доступен в web-клиенте";
#Иначе

//@skip-check empty-except-statement
Попытка
Для Каждого Сообщение Из Сообщения Цикл
ЮТОбщий.ВывестиВКонсоль(Сообщение);
КонецЦикла;
Исключение
// Игнорируем ошибку
КонецПопытки;
#КонецЕсли

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -55,19 +55,20 @@
Возврат Значение;
КонецЕсли;

ТипТип = Тип("Тип");
#Если Сервер Тогда
Если ТипЗначения = Тип("ОбъектМетаданных") Тогда
Значение = Значение.ПолноеИмя();
ТипЗначения = Тип("Строка");
КонецЕсли;
#КонецЕсли

Если ТипЗначения <> Тип("Тип") И ТипЗначения <> Тип("Строка") Тогда
Если ТипЗначения <> ТипТип И ТипЗначения <> Тип("Строка") Тогда
Значение = ТипЗнч(Значение);
ТипЗначения = Тип("Тип");
ТипЗначения = ТипТип;
КонецЕсли;

Если ТипЗначения = Тип("Тип") Тогда
Если ТипЗначения = ТипТип Тогда
ИдентификаторТипа = ЮТТипыДанныхСлужебный.ИдентификаторТипа(Значение); // Для работы кэширования
Возврат ЮТМетаданныеПовтИсп.ОписаниеОбъектаМетаданныхПоИдентификаторуТипа(ИдентификаторТипа);
Иначе
Expand Down
14 changes: 4 additions & 10 deletions exts/yaxunit/src/CommonModules/ЮТОбщий/Module.bsl
Original file line number Diff line number Diff line change
Expand Up @@ -94,13 +94,11 @@
// Параметры:
// Значение - Строка - преобразуемая строка
// Разделитель - Строка - строка-разделитель
// УдалятьКонцевыеПробелы - Булево - удалять или нет концевые пробелы между элементами в строке
// Фиксированный - Булево - возвращать фиксированный или обычный массив
//
// Возвращаемое значение:
// Массив Из Строка - массив строк
//
Функция РазложитьСтрокуВМассивПодстрок(Знач Значение, Разделитель = ";", УдалятьКонцевыеПробелы = Ложь) Экспорт
Функция РазложитьСтрокуВМассивПодстрок(Знач Значение, Разделитель = ";") Экспорт

КодУниверсальногоРазделителя = 5855;
УниверсальныйРазделитель = Символ(КодУниверсальногоРазделителя);
Expand All @@ -109,15 +107,11 @@
МассивСтрок = ?(МодифицированнаяСтрока = "", Новый Массив, СтрРазделить(МодифицированнаяСтрока,
УниверсальныйРазделитель));

Если УдалятьКонцевыеПробелы Тогда
Для Индекс = 0 По МассивСтрок.ВГраница() Цикл

Для Индекс = 0 По МассивСтрок.ВГраница() Цикл

МассивСтрок[Индекс] = СокрЛП(МассивСтрок[Индекс]);

КонецЦикла;
МассивСтрок[Индекс] = СокрЛП(МассивСтрок[Индекс]);

КонецЕсли;
КонецЦикла;

Возврат МассивСтрок;

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -73,13 +73,13 @@
Возврат Параметры;
КонецЕсли;

ПарыКлючЗначение = ЮТОбщий.РазложитьСтрокуВМассивПодстрок(ПараметрыЗапускаСтрокой, ";", Истина);
ПарыКлючЗначение = ЮТОбщий.РазложитьСтрокуВМассивПодстрок(ПараметрыЗапускаСтрокой, ";");

ПараметрыЗапуска = Новый Структура;

Для Каждого Пара Из ПарыКлючЗначение Цикл

КлючЗначение = ЮТОбщий.РазложитьСтрокуВМассивПодстрок(Пара, "=", Истина);
КлючЗначение = ЮТОбщий.РазложитьСтрокуВМассивПодстрок(Пара, "=");

Если КлючЗначение.Количество() = 1 Тогда
ПараметрыЗапуска.Вставить(КлючЗначение[0], Истина);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,7 @@
Продолжить;
КонецЕсли;

КолонкиМакета = ЮТОбщий.РазложитьСтрокуВМассивПодстрок(Строка, Разделитель, Истина);
КолонкиМакета = ЮТОбщий.РазложитьСтрокуВМассивПодстрок(Строка, Разделитель);

СледующаяСтрока(ПараметрыИтератора);
Прервать;
Expand All @@ -112,7 +112,7 @@
КонецЕсли;

СтрокаДанных = ПараметрыЗагрузки.Таблица.Добавить();
Блоки = ЮТОбщий.РазложитьСтрокуВМассивПодстрок(Строка, Разделитель, Истина);
Блоки = ЮТОбщий.РазложитьСтрокуВМассивПодстрок(Строка, Разделитель);

Для Каждого ОписаниеКолонки Из ПараметрыЗагрузки.ОписаниеКолонок Цикл

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,9 @@

ИначеЕсли Тип = Тип("Дата") Тогда

Значение = ЮТТестовыеДанные.СлучайнаяДата(ТекущаяДата() - 315360000, ТекущаяДата() + 315360000); // +/- 10 лет
//@skip-check use-non-recommended-method
Значение = ЮТТестовыеДанные.СлучайнаяДата(ТекущаяДата() - 315360000, ТекущаяДата() + 315360000); // BSLLS:DeprecatedCurrentDate-off
// +/- 10 лет

ИначеЕсли Тип = Тип("Булево") Тогда

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -584,7 +584,7 @@
//
// Параметры:
// ОжидаемоеЗначение - Произвольный - Ожидается, что значение Контекста (или его свойство) содержит указанное значение
// - см. ЮТест.Предикат - Модуль настройки предикатов, см. ЮТест.Предикат
// - ОбщийМодуль - Модуль настройки предикатов, см. ЮТест.Предикат
// ОписаниеПроверки - Строка - Описание конкретной проверки
//
// Возвращаемое значение:
Expand Down Expand Up @@ -619,7 +619,7 @@
//
// Параметры:
// ОжидаемоеЗначение - Произвольный - Ожидается, что значение Контекста (или его свойство) НЕ содержит указанное
// - см. ЮТест.Предикат - Модуль настройки предикатов, см. ЮТест.Предикат
// - ОбщийМодуль - Модуль настройки предикатов, см. ЮТест.Предикат
// ОписаниеПроверки - Строка - Описание конкретной проверки
//
// Возвращаемое значение:
Expand Down Expand Up @@ -1564,9 +1564,9 @@
ВызватьИсключение "Использование транзакции внутри транзакции при проверке метода недопустимо";
КонецЕсли;

НачатьТранзакцию();
НачатьТранзакцию(); // BSLLS:PairingBrokenTransaction-off BSLLS:BeginTransactionBeforeTryCatch-off
Ошибка = ЮТОбщий.ВыполнитьМетод(ПолноеИмяМетода, ПараметрыМетода, Модуль);
ОтменитьТранзакцию();
ОтменитьТранзакцию(); // BSLLS:WrongUseOfRollbackTransactionMethod-off
Иначе
Ошибка = ЮТОбщий.ВыполнитьМетод(ПолноеИмяМетода, ПараметрыМетода, Модуль);
КонецЕсли;
Expand Down Expand Up @@ -1610,12 +1610,6 @@

КонецПроцедуры

Функция ЭтоТипМассива(Тип)

Возврат Тип = Тип("Массив") Или Тип = Тип("ФиксированныйМассив");

КонецФункции

#КонецОбласти

Функция СравнитьПоЗначению(Значение1, Значение2)
Expand Down

0 comments on commit 95dd87c

Please sign in to comment.