Don't try to store an Alegre package for an item of "blank" type. (#2214

)

This commit fixes an error reported by Sentry.

Fixes: CV2-6150.

app/models/concerns/alegre_v2.rb

@@ -251,8 +251,10 @@ def delete_package_audio(project_media, _field, params)
     end
 
     def store_package(project_media, field, params={})
+      type = get_type(project_media)
+      return if type.nil?
       generic_package(project_media, field).merge(
-        self.send("store_package_#{get_type(project_media)}", project_media, field, params)
+        self.send("store_package_#{type}", project_media, field, params)
       )
     end
 

test/models/bot/alegre_4_test.rb

@@ -82,4 +82,11 @@ def teardown
     end
     Bot::Alegre.unstub.stubs(:request)
   end
+
+  test "should not try to store package for blank item" do
+    pm = create_project_media media: Blank.create!
+    assert_nothing_raised do
+      Bot::Alegre.store_package(pm, 'title')
+    end
+  end
 end