Commit dbb3a35

@@ -23,37 +23,39 @@ namespace MVPtoMVVM.mvp.presenters
             InitializeView();
         }
 
-        public void AddNewItem()
+        private void InitializeView()
         {
-            var items = new List<ITodoItemPresenter>(view.GetTodoItems());
-            var newItem = new TodoItemPresenter(itemRepository) {Description = string.Empty, DueDate = DateTime.Today} ;
-            items.Add(newItem);
-            view.SetTodoItems(items);
+            RefreshItems();
         }
 
-        public void CancelAllChanges()
+        private void RefreshItems()
         {
-            RefreshItems();
+            var todoItemPresenters = itemRepository.GetAll().Select(MapFrom);
+            view.ShowTodoItems(todoItemPresenters);
         }
 
-        public void Remove(int itemId)
+        private ITodoItemPresenter MapFrom(TodoItem todoItem)
         {
-            view.SetTodoItems(view.GetTodoItems().Where(x => x.Id != itemId));
+            return new TodoItemPresenter(itemRepository) { Description = todoItem.Description, DueDate = todoItem.DueDate, Id = todoItem.Id };
         }
 
-        private void InitializeView()
+        public void AddNewItem()
         {
-            RefreshItems();
+            var items = new List<ITodoItemPresenter>(view.GetTodoItems());
+            var newItem = new TodoItemPresenter(itemRepository) {Description = string.Empty, DueDate = DateTime.Today} ;
+            items.Add(newItem);
+            view.ShowTodoItems(items);
         }
 
-        private void RefreshItems()
+        public void CancelAllChanges()
         {
-            view.SetTodoItems(itemRepository.GetAll().Select(MapFrom));
+            RefreshItems();
         }
 
-        private ITodoItemPresenter MapFrom(TodoItem todoItem)
+        public void Remove(int itemId)
         {
-            return new TodoItemPresenter(itemRepository) { Description = todoItem.Description, DueDate = todoItem.DueDate, Id = todoItem.Id};
+            view.ShowTodoItems(view.GetTodoItems().Where(x => x.Id != itemId));
         }
+
     }
 }
\ No newline at end of file

@@ -22,14 +22,6 @@ namespace MVPtoMVVM.mvp.presenters
             InitializeView();
         }
 
-        public void SetItem(TodoItem item)
-        {
-            Id = item.Id;
-            Description = item.Description;
-            DueDate = item.DueDate;
-            IsDirty = false;
-        }
-
         private void InitializeView()
         {
             view.Id = Id;
@@ -38,6 +30,24 @@ namespace MVPtoMVVM.mvp.presenters
             view.SaveButtonEnabled = false;
         }
 
+        private void UpdateControlState()
+        {
+            view.SaveButtonEnabled = IsDirty && IsDescriptionValid() && IsDueDateValid();
+            view.DescriptionHasValidationErrors = !IsDescriptionValid();
+            view.DescriptionValidationMessage = GetDescriptionValidationMessage();
+            view.DueDateHasValidationErrors = !IsDueDateValid();
+            view.DueDateValidationMessage = GetDueDateValidationMessage();
+            view.IsDueSoon = IsDueSoon();
+        }
+
+        public void SetItem(TodoItem item)
+        {
+            Id = item.Id;
+            Description = item.Description;
+            DueDate = item.DueDate;
+            IsDirty = false;
+        }
+
         public void SaveItem()
         {
             var item = GetTodoItem();
@@ -88,16 +98,6 @@ namespace MVPtoMVVM.mvp.presenters
             }
         }
 
-        private void UpdateControlState()
-        {
-            view.SaveButtonEnabled = IsDirty && IsDescriptionValid() && IsDueDateValid();
-            view.DescriptionHasValidationErrors = !IsDescriptionValid();
-            view.DescriptionValidationMessage = GetDescriptionValidationMessage();
-            view.DueDateHasValidationErrors = !IsDueDateValid();
-            view.DueDateValidationMessage = GetDueDateValidationMessage();
-            view.IsDueSoon = IsDueSoon();
-        }
-
         private bool IsDescriptionValid()
         {
             return description.Length > 0;

@@ -6,9 +6,6 @@ using MVPtoMVVM.mvp.views;
 
 namespace MVPtoMVVM.mvp
 {
-    /// <summary>
-    /// Interaction logic for TodoItemView.xaml
-    /// </summary>
     public partial class TodoItemView : ITodoItemView
     {
         private readonly IMvpPresenter parent;