todo-void 0.1 → 0.1.1.pre

data/interactors/status_changes_interactor.rb

@@ -21,8 +21,12 @@ class StatusChangesInteractor
 
   private
   def change_todo_status(hash, status)
-    todos = @list.find(hash).to_array
-    todos = TodoFilter.new(todos).with_status([:finished, :started, :pending]).recent.execute
+    todos = TodoFilter.new(@list)
+                      .with_hash(hash)
+                      .with_status([:finished, :started, :pending])
+                      .recent
+                      .execute
+    todos = todos.to_a
 
     case todos.length
     when 0

data/interactors/todo_interactor.rb

@@ -11,7 +11,7 @@ class TodoInteractor
   end
 
   def list_all
-    todos = @store.todos.to_array
+    todos = @store.todos
     filter_todos(todos)
   end
 

data/lib/todo_filter.rb

@@ -1,9 +1,16 @@
+require_relative './todo_list'
+
 class TodoFilter
   def initialize(todos)
     @todos = todos.dup
     @todos.reject! { |todo| todo.status == :deleted }
   end
 
+  def with_hash(search)
+    @todos.reject! { |todo| not /^#{search}/.match(todo.id) }
+    self
+  end
+
   def with_status(statuses)
     result = []
     statuses.each do |status|
@@ -24,7 +31,7 @@ class TodoFilter
   end
 
   def execute
-    @todos
+    TodoList.new(@todos)
   end
 
   private

data/lib/todo_list.rb

@@ -1,38 +1,8 @@
-class TodoList
-  attr_reader :todo
-  
-  def initialize
-    @todo = {}
-  end
-
-  def add(todo)
-    @todo.store(todo.id, todo)
-  end
-
-  def find(search)
-    list = TodoList.new
-    @todo.each do |id, todo|
-      list.add todo if /^#{search}/.match id
-    end
-    list
-  end
-
-  def length
-    @todo.length
-  end
-
-  def empty?
-    @todo.length == 0
-  end
+require 'set'
 
+class TodoList < Set
   def update(todo)
-    @todo.delete(todo.id)
-    @todo[todo.id] = todo
-  end
-
-  def to_array
-    todos = []
-    @todo.each do |id, todo| todos << todo end
-    todos
+    delete(todo)
+    add(todo)
   end
 end

data/lib/todo_store.rb

@@ -24,24 +24,27 @@ class TodoStore
 
   private
   def read
-    @list = TodoList.new 
-    
     FileUtils.touch(todo_file)
     data = CSV.read(todo_file)
-    data.each do |raw_todo|
-      todo = Todo.new raw_todo[0]
-      unless raw_todo[2].nil?
-        todo.finished_at =  DateTime.parse(raw_todo[2]).to_time
-      end
-      todo.status = raw_todo[1].to_sym
-      @list.add todo
+    todos = data.map do |raw_todo|
+      convert_raw_todo(raw_todo)
+    end
+    @list = TodoList.new(todos)
+  end
+ 
+  def convert_raw_todo(raw_todo)
+    todo = Todo.new raw_todo[0]
+    unless raw_todo[2].nil?
+      todo.finished_at =  DateTime.parse(raw_todo[2]).to_time
     end
+    todo.status = raw_todo[1].to_sym
+    todo
   end
-  
+
   def write
     CSV.open(todo_file, "w") do |csv|
-      @list.to_array.each do |task|
-        csv << [task.description, task.status, task.finished_at]
+      @list.each do |todo|
+        csv << [todo.description, todo.status, todo.finished_at]
       end
     end
   end

metadata

@@ -1,8 +1,8 @@
 --- !ruby/object:Gem::Specification
 name: todo-void
 version: !ruby/object:Gem::Version
-  version: '0.1'
-  prerelease: 
+  version: 0.1.1.pre
+  prerelease: 6
 platform: ruby
 authors:
 - Robert Curth
@@ -95,7 +95,7 @@ files:
 homepage: http://rubygems.org/gems/todo-void
 licenses:
 - MIT
-post_install_message: 
+post_install_message: Enter t 'Mowing the lawn' to create your first task!
 rdoc_options: []
 require_paths:
 - lib
@@ -108,9 +108,9 @@ required_ruby_version: !ruby/object:Gem::Requirement
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
-  - - ! '>='
+  - - ! '>'
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 1.3.1
 requirements: []
 rubyforge_project: 
 rubygems_version: 1.8.24