Resolve "Контроль уникальности названия" #213

There is no content yet.

changed milestone to %5

added 1 commit

b57142e2 - #70 added unique name control

added 1 commit <ul><li>b57142e2 - #70 added unique name control</li></ul> [Compare with previous version](https://gitlab.com/romanov73/ng-tracker/merge_requests/86/diffs?diff_id=41443502&start_sha=2dff183662769376db3efc91cc60a2ac93235a20)

assigned to @Nightblade73

assigned to @romanov73 and unassigned @Nightblade73

unmarked as a Work In Progress

unmarked as a **Work In Progress**

Смысл в создании всей иерархии классов (NameContainer, NameService, NameRepository) было повысить переиспользование кода.
Т.е. для того чтобы в другом модуле начать контролировать уникальность названия придется всего лишь отнаследоваться от нужных классов/интерфейсов и вставить вызов метода. В вашем же случае в каждом сервисе придется писать однотипный код.
Нужно этот метод засунуть в NameService. И переименуйте его хотя бы в CoreService, т.к. по логике там будет общее поведение для всех сервисов, не только контроль уникальности.

Смысл в создании всей иерархии классов (NameContainer, NameService, NameRepository) было повысить переиспользование кода. Т.е. для того чтобы в другом модуле начать контролировать уникальность названия придется всего лишь отнаследоваться от нужных классов/интерфейсов и вставить вызов метода. В вашем же случае в каждом сервисе придется писать однотипный код. Нужно этот метод засунуть в NameService. И переименуйте его хотя бы в CoreService, т.к. по логике там будет общее поведение для всех сервисов, не только контроль уникальности.

а так все норм реализовано

changed this line in version 2 of the diff

changed this line in [version 2 of the diff](https://gitlab.com/romanov73/ng-tracker/merge_requests/86/diffs?diff_id=41773892&start_sha=b57142e28fc423b5779372e28107e536f03db963#4507aa768a06337c46539c4bab2e3f780d6dc7b5_289_290)

added 1 commit

d8461920 - #111 fixes

Compare with previous version

added 1 commit <ul><li>d8461920 - #111 fixes</li></ul> [Compare with previous version](https://gitlab.com/romanov73/ng-tracker/merge_requests/86/diffs?diff_id=41773892&start_sha=b57142e28fc423b5779372e28107e536f03db963)

поправил, внёс небольшие изменения
теперь в метод передаётся больше параметров. репозиторий переехал из контейнера в базовый сервис
была проблема с тем, чтобы сохранить отредактированную сущность, не меняя её имени, пришлось расширить запрос к бд

поправил, внёс небольшие изменения теперь в метод передаётся больше параметров. репозиторий переехал из контейнера в базовый сервис была проблема с тем, чтобы сохранить отредактированную сущность, не меняя её имени, пришлось расширить запрос к бд

Нужно порефакторить, слишком много методов в контроллере вызывается. Контроль уникальность нужно добавить в метод сохранения, именно там эта логика нужна (нельзя сохранить если имя уже присутствует), т.к. при отдельном вызове метода сохранения контроля не будет

changed this line in version 3 of the diff

changed this line in [version 3 of the diff](https://gitlab.com/romanov73/ng-tracker/merge_requests/86/diffs?diff_id=41918416&start_sha=d846192046aae130e1882f0e889845a664c4180c#ffd2d92a659641c319b215743e9f95cb233117c3_77_77)

added 1 commit

6044d6f8 - #111 save method refactor

Compare with previous version

added 1 commit <ul><li>6044d6f8 - #111 save method refactor</li></ul> [Compare with previous version](https://gitlab.com/romanov73/ng-tracker/merge_requests/86/diffs?diff_id=41918416&start_sha=d846192046aae130e1882f0e889845a664c4180c)

исправлено

resolved all discussions

changed the description

merged

mentioned in commit 10a9bd7a28

mentioned in commit 10a9bd7a28226d6f4b3da43573a9afe993c61447

Labels Milestones

Resolve "Контроль уникальности названия" #213

Step 1:

Step 2: