my_todo 3.1.0 → 3.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 70b1f135ecffdf6707b18aa7a916493221bce327
-  data.tar.gz: e8fcb736c11ac62879afdc96b33f2f98a23f3875
+  metadata.gz: b4c70de321e879cf5515fbbd10b78e71f8a69017
+  data.tar.gz: 53acd0ce0b42fe4cd6baf3bfbadd7c272350c739
 SHA512:
-  metadata.gz: 01ec4a1b26546905b02194a39d4ccd84f9b74defdb49046183ea9ef06cc129f9a7100148ab105d72744f05bc79b9256ae1a8a8fe1c320a9f030ce3cb7504ec68
-  data.tar.gz: a6887b4de5c89b24e7e56ce2910941f9824658f7f27939081f3f03412a72271510f2f712cd636087daee721c8fdd4332c073e1495607a27d719f14fdc2fd0055
+  metadata.gz: 811c02060a4a567ba1528c55152f40d4075387fee2c3d09f810ea685a9f6f577082fabb4f9af87869ad90338d6d880edafb74eac937ba57e58bb409c8f4a21d6
+  data.tar.gz: cd090f60fd4879456d9ce52e3939d8dd344c5e8acaff04e21a41cee1a8f94b01044d02241f7adc54d43388ebeadb809934001f58ed7146d4ef8176cf092f5ea9

data/lib/my_todo/models/item.rb

@@ -21,6 +21,8 @@ class Item < ActiveRecord::Base
   # ActiveModel validation to ensure body is present
   validates :body, presence: true
 
+  # Hook to set done boolean based on detailed status
+  #(see #update_done)
   before_save :update_done
 
   #set done attribute based on detailed_status set

data/lib/my_todo/modules/my_todo_actions.rb

@@ -1,18 +1,33 @@
 module MyTodoActions
   def ask_status
     list_statuses
-    @status = ask("Choose a status for item", default: 1)
+    @status = ask("Choose a status for item", default: set_default_status)
   end
 
   def create_item(options)
-    ask_status
-    @item = Item.create!(options.merge({detailed_status: detailed_statuses[@status.to_i]}).except(:tags))
-    options[:tags].split(' ').each{|tag| item.tags.create(name: tag) } if options[:tags]
+    @item = Item.new(options.except(:tags))
+    assign_detailed_status
+    @item.save!
+    set_tags
   end
 
   def update_item(options)
+    item.assign_attributes(options)
+    @item = item #Find a better way!!!!
+    assign_detailed_status
+    item.save!
+  end
+
+  def assign_detailed_status
     ask_status
-    new_status = detailed_statuses[@status.to_i]
-    item.detailed_status != new_status ? item.update!(options.merge({detailed_status: new_status})) : item.update!(options)
+    @item.write_attribute(:detailed_status, detailed_statuses[@status.to_i])
+  end
+
+  def set_tags
+    options[:tags].split(' ').each{|tag| @item.tags.create(name: tag) } if options[:tags]
+  end
+
+  def set_default_status
+    @item.detailed_status.nil? ? 1 : detailed_statuses.index(@item.detailed_status)
   end
 end

data/lib/my_todo/version.rb

@@ -1,3 +1,3 @@
 module MyTodo
-  VERSION = "3.1.0"
+  VERSION = "3.1.1"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: my_todo
 version: !ruby/object:Gem::Version
-  version: 3.1.0
+  version: 3.1.1
 platform: ruby
 authors:
 - Vell
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-12-20 00:00:00.000000000 Z
+date: 2016-12-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler