Refactor/nullability

[ronimizy]

Closes #24

[ronimizy]

В AddTeacher.Handler создаётся объект GroupSubjectTeacher, требующий в себя объект Teacher на данный момент DbSet для учителей не существует и получить его оттуда невозможно, так что я закомментил неработающий код.

Iwentys.EntityManager/Source/Iwentys.EntityManager.Web/Iwentys.EntityManager.Web.Api/Controllers/Teaching/Commands/AddTeacher.cs

Lines 34 to 40 in 1cdc89d

	// TODO: Add Teachers DbSet and invoke proper constructor
	// groupSubject.Teachers.Add(new GroupSubjectTeacher()
	// {
	// TeacherType = request.Args.TeacherType,
	// GroupSubjectId = groupSubject.Id,
	// TeacherId = request.Args.TeacherId
	// });

После закрытия этого пр добавлю пункт на фикс этого в соответствующую ишую

• Добавить DbSet<Teacher> в контекст #30

[ronimizy]

В StudyGroupFaker создаётся объект StudyGroup требующий в себя объект StudyCourse фейкера для него нет, так что пока вставил null, после того как вмержим создам ишую на рефактор и на то чтобы это правило нулабилити сделать ошибкой

[FrediKats]

Мне кажется, что с Response? мы немного не туда свернули. Респонс то мы всегда получаем. Просто он содержит null. Мне кажется, что стоит перейти именно к такой модели.

[ronimizy]

Мне кажется, что с Response? мы немного не туда свернули. Респонс то мы всегда получаем. Просто он содержит null. Мне кажется, что стоит перейти именно к такой модели.

Ля действительно, я чёт на том пр'е с токенами запутался из-за комментов, и за нулабилити доебался не в ту сторону.

Переделаю.

[lipa44]

Есть огромные вопросы к папке Teachers - наичная от того, какие типы мы там возвращаем, продолдая тем, как оно вообще реботает и работает ли, и заканчивая наличием команд помимо запросов - разве у нас этот сервис не для ридолни запросов прендначен?)

[ronimizy]

@lipa44 это то тут при чём? Заведи ишую.

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
8 Code Smells

0.0% Coverage
0.0% Duplication

[FrediKats]

lgtm
А то, что вкинул @lipa44 нужно обсудить.
Спасибо!

[lipa44]

С пивом пойдёт