item: move subitems to the same case, when main item is moved #9

app/controllers/items_controller.rb

@@ -78,6 +78,9 @@ def edit
 
   def update
     if @item.update_attributes(item_params)
+      # move subitems to same case
+      @item.move_sub_items
+
       redirect_to case_path(@item.case), notice: "Updated '#{@item.name}'"
     else
       render action: 'edit'

app/models/item.rb

@@ -47,4 +47,16 @@ def destroy
   def md5_sum
     Digest::MD5.hexdigest(self.to_json)
   end
+
+  # Move all suitems to the same case.
+  #
+  # TODO: show message in view before save
+  def move_sub_items
+    self.items.each do |sub_item|
+      if self.case != sub_item.case
+        sub_item.case = self.case
+        sub_item.save!
+      end
+    end
+  end
 end

spec/models/item_spec.rb

@@ -3,7 +3,9 @@
 RSpec.describe Item, type: :model do
 
   before do
-    @item = Item.new(name: 'my item', model: 'foobar')
+    @case = Case.new(name: "möp", acronym: "bla")
+
+    @item = Item.new(name: 'my item', model: 'foobar', case: @case)
     @item.item_type = ItemType.new(name: 'Fach')
   end
 
@@ -46,6 +48,30 @@
     end
   end
 
+  context "#move_sub_items" do
+
+    before do
+      @new_case = Case.new(name: "new_case", acronym: "nc")
+      @sub_item = Item.new(name: "sub_item", case: @case)
+
+      @item.items << @sub_item
+      @item.save
+    end
+
+    it "should move all subitems to the same case" do
+      expect(@item.case).to eq @sub_item.case
+      expect(@item.items.count).to eq 1
+
+      @item.case = @new_case
+      @item.save
+
+      expect(@item.case).not_to eq @sub_item.case
+
+      @item.move_sub_items
+      expect(@item.case).to eq @sub_item.case
+    end
+  end
+
   context "#delete" do
     it "should not delete item" do
       item = Item.new(name: 'foobar')