Skip to content
New issue

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

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

Already on GitHub? # to your account

225 / возможность устанавливать управляемые блокировки #374

Merged
merged 6 commits into from
Jul 29, 2024

Conversation

alkoleft
Copy link
Member

@alkoleft alkoleft commented Jun 6, 2024

Описание изменений

Добавлен метод для установки управляемой блокировки в фоновом сеансе


  • Добавлены тесты
  • Изменения отражены в документации (при необходимости)

Close #225

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

SonarQube Code Analytics

🔗Quality Gate failed

failed

Additional information

The following metrics might not affect the Quality Gate status but improving them will improve your project code quality.

Issues

bug 0 Bugs

vulnerability 0 Vulnerabilities

code_smell 1 Code Smells

Coverage and Duplications

No data Coverage

duplication_lt_3 0.00% Duplication

// Параметры:
// Блокировка - БлокировкаДанных
// Длительность - Число - Длительность блокировки в секундах
Процедура УстановитьФоновуюБлокировку(Блокировка, Длительность = 30) Экспорт
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔗Создайте константу с понятным названием, присвойте ей значение "30" и используйте эту константу вместо магического числа.

CODE_SMELL Codesmell  MINOR Minor   ⌛ 1min effort

badpractice parameters  Why is this an issue?

@alkoleft alkoleft merged commit b247584 into develop Jul 29, 2024
1 of 4 checks passed
@alkoleft alkoleft deleted the feature/225 branch July 29, 2024 20:46
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
1 participant