UI voor het toevoegen van Genre's

[tdehaes]

No description provided.

[tdehaes]

Nico, ik denk dat ik iets fout gedaan heb bij het aanpassen van de code. De bedoeling was om de bestaande pull request aan te passen, maar blijkbaar is er een 2e pull request gemaakt. Hoe moet ik nu verder om het juist te doen? Thorsten Op di 19 nov. 2019 om 16:20 schreef NicoVermeir <notifications@github.com>:

…

***@***.**** requested changes on this pull request. ------------------------------ In Fletnix.Repositories/GenreRepository.cs <#9 (comment)>: > + public bool Create(Genre item) + { + try + { + var context = new FletnixContext(); + context.Genre.Add(item); + context.SaveChanges(); + + return true; + } + catch (Exception) + { + return false; + + } + Overbodige whitespace verwijderen ------------------------------ In Fletnix.Repositories/GenreRepository.cs <#9 (comment)>: > + class GenreRepository : IRepository<Genre> + { + public bool Create(Genre item) + { + try + { + var context = new FletnixContext(); + context.Genre.Add(item); + context.SaveChanges(); + + return true; + } + catch (Exception) + { + return false; + overbodige whitespace ------------------------------ In Fletnix.Services/GenreService.cs <#9 (comment)>: > @@ -0,0 +1,28 @@ +using System; +using System.Collections.Generic; +using System.Text; +using Fletnix.Model; +using Fletnix.Repositories; + +namespace Fletnix.Services +{ + class GenreService : IGenreService + { + private readonly IRepository<Genre> _genreRepository; + teveel whitespace ------------------------------ In Fletnix.Services/IGenreService.cs <#9 (comment)>: > @@ -0,0 +1,15 @@ +using Fletnix.Model; +using System; +using System.Collections.Generic; +using System.Text; + +namespace Fletnix.Services +{ + public interface IGenreService + { + IList<Genre> GetGenres(); + + void SaveGenre(Genre item); + whitespace ------------------------------ In Fletnix/Messages/GenreCreatedMessage.cs <#9 (comment)>: > @@ -0,0 +1,16 @@ +using Fletnix.Model; +using System; +using System.Collections.Generic; +using System.Text; + +namespace Fletnix.Messages +{ + public class GenreCreatedMessage + { + public Genre NewGenre { get; set; } + public GenreCreatedMessage(Genre newGenre) hier mag een whitespace voor komen ------------------------------ In Fletnix/ViewModel/CreateGenreViewModel.cs <#9 (comment)>: > + public RelayCommand AddGenreCommand => _addGenreCommand ??= new RelayCommand(AddGenre); + + public string Name + { + get => _name; + set + { + _name = value; + } + } + + public CreateGenreViewModel(IGenreService genreService) + { + _genreService = genreService; + NewGenre = new Genre(); + whitespace ------------------------------ In Fletnix/ViewModel/MainViewModel.cs <#9 (comment)>: > @@ -56,18 +57,28 @@ public MainViewModel(ICatalogService catalogService) { _catalogService = catalogService; + teveel whitespace ------------------------------ In Fletnix/ViewModel/MainViewModel.cs <#9 (comment)>: > //Catalog = _catalogService.GetMovies().ToList(); Catalog = new ObservableCollection<CatalogItem>(catalog); + teveel whitespace ------------------------------ In Fletnix/ViewModel/MainViewModel.cs <#9 (comment)>: > private void OnCatalogItemCreated(CatalogItemCreatedMessage message) { Catalog.Add(message.NewCatalogItem); } + + teveel whitespace — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#9?email_source=notifications&email_token=AKMMAHUPJR2ZVLKRWZDYKNLQUP74BA5CNFSM4JO2CG6KYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCMCVH3Y#pullrequestreview-319116271>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AKMMAHVC4BU2LWDOWRMQ67TQUP74BANCNFSM4JO2CG6A> .

[NicoVermeir]

laat maar zo staan, mooi voorbeeld om maandag in de les te bekijken