Merge pull request #172 from raffacabofrio/master

Bugfix pra issue 139. Vai precisar de uma PR em front também que eu …
SharebookBR · Dec 13, 2018 · 89e481a · 89e481a
2 parents f938266 + ba1cb4e
commit 89e481a
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/ShareBook/ShareBook.Api/ViewModels/UpdateBookVM.cs b/ShareBook/ShareBook.Api/ViewModels/UpdateBookVM.cs
@@ -11,6 +11,8 @@ public class UpdateBookVM : BaseViewModel
 
         public Guid CategoryId { get; set; }
 
+        public Guid UserId { get; set; }
+
         public bool Approved { get; set; }
 
         public string ImageName { get; set; }

diff --git a/ShareBook/ShareBook.Service/Book/BookService.cs b/ShareBook/ShareBook.Service/Book/BookService.cs
@@ -43,9 +43,6 @@ public Result<Book> Approve(Guid bookId, DateTime? chooseDate = null)
             book.ChooseDate = chooseDate?.Date ?? DateTime.Today.AddDays(5);
             _repository.Update(book);
 
-            // TODO: obter o usuário doador. Não o usuário logado.
-            // Github issue: https://github.com/SharebookBR/sharebook-backend/issues/139
-            book.UserId = new Guid(Thread.CurrentPrincipal?.Identity?.Name);
             _booksEmailService.SendEmailBookApproved(book).Wait();
 
             return new Result<Book>(book);
@@ -155,7 +152,6 @@ public override Result<Book> Update(Book entity)
                 this.Approve(entity.Id);
             }
 
-
             return result;
         }