romanov73/ng-tracker
1
0
Fork 0
Code Issues 18 Pull Requests 4 Projects Releases Wiki Activity

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

Merged
VladimirZarayskiy merged 1 commits from 111-unique-name-control-conf into dev 2019-05-13 23:20:45 +04:00
Conversation 18 Commits 1 Files Changed 3 +20 -14
VladimirZarayskiy commented 2019-05-07 19:05:06 +04:00 (Migrated from gitlab.com)
Copy Link
No description provided.
VladimirZarayskiy commented 2019-05-07 19:07:20 +04:00 (Migrated from gitlab.com)
Copy Link

changed milestone to %5

changed milestone to %5
VladimirZarayskiy commented 2019-05-08 17:08:52 +04:00 (Migrated from gitlab.com)
Copy Link

added 1 commit

Compare with previous version

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)
VladimirZarayskiy commented 2019-05-08 17:09:45 +04:00 (Migrated from gitlab.com)
Copy Link

assigned to @Nightblade73

assigned to @Nightblade73
VladimirZarayskiy commented 2019-05-08 17:09:51 +04:00 (Migrated from gitlab.com)
Copy Link

assigned to @romanov73 and unassigned @Nightblade73

assigned to @romanov73 and unassigned @Nightblade73
VladimirZarayskiy commented 2019-05-08 17:13:20 +04:00 (Migrated from gitlab.com)
Copy Link

unmarked as a Work In Progress

unmarked as a **Work In Progress**
romanov73 commented 2019-05-11 22:59:41 +04:00 (Migrated from gitlab.com)
Copy Link

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

Смысл в создании всей иерархии классов (NameContainer, NameService, NameRepository) было повысить переиспользование кода. Т.е. для того чтобы в другом модуле начать контролировать уникальность названия придется всего лишь отнаследоваться от нужных классов/интерфейсов и вставить вызов метода. В вашем же случае в каждом сервисе придется писать однотипный код. Нужно этот метод засунуть в NameService. И переименуйте его хотя бы в CoreService, т.к. по логике там будет общее поведение для всех сервисов, не только контроль уникальности.
romanov73 commented 2019-05-11 23:00:09 +04:00 (Migrated from gitlab.com)
Copy Link

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

а так все норм реализовано
VladimirZarayskiy commented 2019-05-12 00:28:19 +04:00 (Migrated from gitlab.com)
Copy Link

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)
VladimirZarayskiy commented 2019-05-12 00:28:19 +04:00 (Migrated from gitlab.com)
Copy Link

added 1 commit

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)
VladimirZarayskiy commented 2019-05-12 00:29:48 +04:00 (Migrated from gitlab.com)
Copy Link

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

поправил, внёс небольшие изменения теперь в метод передаётся больше параметров. репозиторий переехал из контейнера в базовый сервис была проблема с тем, чтобы сохранить отредактированную сущность, не меняя её имени, пришлось расширить запрос к бд
romanov73 commented 2019-05-13 10:53:46 +04:00 (Migrated from gitlab.com)
Copy Link

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

Нужно порефакторить, слишком много методов в контроллере вызывается. Контроль уникальность нужно добавить в метод сохранения, именно там эта логика нужна (нельзя сохранить если имя уже присутствует), т.к. при отдельном вызове метода сохранения контроля не будет
VladimirZarayskiy commented 2019-05-13 19:08:43 +04:00 (Migrated from gitlab.com)
Copy Link

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)
VladimirZarayskiy commented 2019-05-13 19:08:44 +04:00 (Migrated from gitlab.com)
Copy Link

added 1 commit

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)
VladimirZarayskiy commented 2019-05-13 19:42:15 +04:00 (Migrated from gitlab.com)
Copy Link

исправлено

исправлено
romanov73 commented 2019-05-13 23:19:45 +04:00 (Migrated from gitlab.com)
Copy Link

resolved all discussions

resolved all discussions
romanov73 commented 2019-05-13 23:20:11 +04:00 (Migrated from gitlab.com)
Copy Link

changed the description

changed the description
romanov73 commented 2019-05-13 23:20:45 +04:00 (Migrated from gitlab.com)
Copy Link

merged

merged
romanov73 commented 2019-05-13 23:20:45 +04:00 (Migrated from gitlab.com)
Copy Link

mentioned in commit 10a9bd7a28

mentioned in commit 10a9bd7a28226d6f4b3da43573a9afe993c61447
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
backend

   
bug

   
frontend

   
выполнить первой

   
выполняется

   
конференции

   
события

   
статьи
No Label
backend
bug
frontend
выполнить первой
выполняется
конференции
события
статьи
Milestone
Clear milestone
No items
No Milestone
Конференции
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: romanov73/ng-tracker#213
No description provided.
Delete Branch "111-unique-name-control-conf"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?