fixed backwards compat issue

lib/zdb.ex

@@ -280,9 +280,10 @@ defmodule Zdb do
  struct_s = Poison.encode!(item)
  attributes = [hk: hk, rk: rk,struct: struct_s]
  attributes = keys_to_strings(attributes)
- _put(table,attributes)
+ special_put(table,attributes)
  end
- defp _put(table_name,item,opts \\[return_values: :all_old]) do
+ # TODO: fix this terrible hack
+ defp special_put(table_name,item,opts \\[return_values: :all_old]) do
  table = "#{Mix.env}_#{table_name}"
  case :erlcloud_ddb2.put_item(table,item,opts,config()) do
  {:ok, ret} when ret == [] ->
@@ -294,6 +295,17 @@ defmodule Zdb do
  {:error,e} -> raise "Zdb.put error: #{inspect e} \n\t table: #{inspect table_name}\n\t item #{inspect item}\n\t opts: #{inspect opts}"
  end
  end
+ defp _put(table_name,item,opts \\[return_values: :all_old]) do
+ table = "#{Mix.env}_#{table_name}"
+ case :erlcloud_ddb2.put_item(table,item,opts,config()) do
+ {:ok, ret} when ret == [] ->
+ {:ok,[]}
+ {:ok, ret} ->
+ Logger.warn "WARNING: put overwrote an existing item #{inspect ret}"
+ {:ok,[]}
+ {:error,e} -> raise "Zdb.put error: #{inspect e} \n\t table: #{inspect table_name}\n\t item #{inspect item}\n\t opts: #{inspect opts}"
+ end
+ end
  def parse_key(item) do
  {hk,rk} = item_key_to_strings(item.key)
  key = infer_keys([hk: hk,rk: rk])

test/zdb_test.exs

@@ -35,8 +35,11 @@ defmodule ZdbTest do
  key = res.key
  assert key == {"bar","foo"}
  end
- test "put returns Zitem" do
+ test "put returns old Zitem" do
+ raise "this is wrong, need to return nil, or the old item, not the input item"
  item = %Zitem{key: {:bar,:foo},table: "test_table"}
+ {:ok,res} = Zdb.put(item) 
+ assert res == nil
  {:ok,res} = Zdb.put(item)
  assert match?(%Zitem{},res), "should get %Zitem{} \n\tgot: #{inspect res}"
  assert res == %Zitem{key: {:bar,:foo},table: "test_table"}