Merge pull request #195 from raffacabofrio/master

Corrige o bug no job 3 - remove vitrine.
SharebookBR · Jan 14, 2019 · c68c560 · c68c560
2 parents 561e2d4 + bc5ae35
commit c68c560
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 7 deletions.
diff --git a/ShareBook/ShareBook.Api/ViewModels/BooksVM.cs b/ShareBook/ShareBook.Api/ViewModels/BooksVM.cs
@@ -16,6 +16,6 @@ public class BooksVM
         public int TotalInterested { get; set; }
         public string Status { get; set; }
         public DateTime CreationDate { get; set; }
-        public DateTime ChooseDate { get; set; } 
+        public DateTime? ChooseDate { get; set; }
     }
 }
diff --git a/ShareBook/ShareBook.Service/Book/BookService.cs b/ShareBook/ShareBook.Service/Book/BookService.cs
@@ -137,7 +137,7 @@ public override Result<Book> Update(Book entity)
 
 
             //imagem eh opcional no update
-            if (entity.ImageName != "" && entity.ImageBytes.Length > 0)
+            if (!string.IsNullOrEmpty(entity.ImageName) && entity.ImageBytes.Length > 0)
             {
                 entity.ImageSlug = ImageHelper.FormatImageName(entity.ImageName, entity.Slug);
                 _uploadService.UploadImage(entity.ImageBytes, entity.ImageSlug, "Books");

diff --git a/ShareBook/Sharebook.Jobs/Executor/JobExecutor.cs b/ShareBook/Sharebook.Jobs/Executor/JobExecutor.cs
@@ -12,7 +12,7 @@ namespace Sharebook.Jobs
 
     public class JobExecutor : IJobExecutor
     {
-        private readonly List<IJob> _jobs = new List<IJob>();
+        private readonly IList<IJob> _jobs;
         private readonly IJobHistoryRepository _jobHistoryRepo;
         private Stopwatch _stopwatch;
 
@@ -23,9 +23,12 @@ public JobExecutor(IJobHistoryRepository jobHistoryRepo,
         {
             _jobHistoryRepo = jobHistoryRepo;
 
-            _jobs.Add(job1);
-            _jobs.Add(job2);
-            _jobs.Add(job3);
+            _jobs = new List<IJob>
+            {
+                job1,
+                job2,
+                job3
+            };
 
         }
 

diff --git a/ShareBook/Sharebook.Jobs/Jobs/3 - RemoveBookFromShowcase.cs b/ShareBook/Sharebook.Jobs/Jobs/3 - RemoveBookFromShowcase.cs
@@ -57,7 +57,7 @@ public override JobHistory Work()
                 else
                 {
                     book.ChooseDate = DateTime.Today.AddDays(10);
-                    messages.Add(string.Format("Livro '{0}' vai ficar +10 dias na vitrine porque ainda não te interessados. :/", book.Title));
+                    messages.Add(string.Format("Livro '{0}' vai ficar +10 dias na vitrine porque ainda não tem interessados. :/", book.Title));
                     SendEmail(book);
                 }