Merge pull request #2005 from eriko/plugin_store_delete

clean up plugin store when removing data instead of setting the value to...

app/models/plugin_store.rb

@@ -22,6 +22,11 @@ class PluginStore
     row.save
   end
 
+  def self.remove(plugin_name, key)
+    PluginStoreRow.where(plugin_name: plugin_name, key: key).destroy_all
+  end
+
+
   protected
 
 

spec/models/plugin_store_spec.rb

@@ -10,6 +10,10 @@ describe PluginStore do
     PluginStore.get("my_plugin", k)
   end
 
+  def remove_row(k)
+    PluginStore.remove("my_plugin", k)
+  end
+
   it "sets strings correctly" do
     set("hello", "world")
     expect(get("hello")).to eq("world")
@@ -46,4 +50,11 @@ describe PluginStore do
     # ensure indiff access holds
     expect(result[:hi]).to eq("there")
   end
+
+  it "removes correctly" do
+    set("hello", true)
+    remove_row("hello")
+    expect(get("hello")).to eq(nil)
+  end
+
 end