commit 033768ddffb84e6e849230e292321ad3531294ce
author Klaus Trainer <klaus_trainer@posteo.de> Wed Aug 19 12:23:20 2015 +0200
committer Klaus Trainer <klaus_trainer@posteo.de> Wed Aug 19 12:23:20 2015 +0200
tree 78daac6af837153e8acaa233ec1f96e57136c3aa
parent a7e74f02f49c597ce06a55aabb0080965b0e46c9

Match return values

src/couchdb_peruser.erl

diff --git a/src/couchdb_peruser.erl b/src/couchdb_peruser.erl
index 992ac47..a4c8a2e 100644
--- a/src/couchdb_peruser.erl
+++ b/src/couchdb_peruser.erl
@@ -67,7 +67,7 @@
         case couch_util:get_value(<<"deleted">>, Doc, false) of
         false ->
             UserDb = ensure_user_db(User),
-            ensure_security(User, UserDb, fun add_user/3),
+            ok = ensure_security(User, UserDb, fun add_user/3),
             State;
         true ->
             case State#state.delete_dbs of
@@ -75,7 +75,8 @@
                 _UserDb = delete_user_db(User),
                 State;
             false ->
-                ensure_security(User, user_db_name(User), fun remove_user/3),
+                UserDb = user_db_name(User),
+                ok = ensure_security(User, UserDb, fun remove_user/3),
                 State
             end
         end;
@@ -88,7 +89,10 @@
 delete_user_db(User) ->
     UserDb = user_db_name(User),
     try
-        fabric:delete_db(UserDb, [?ADMIN_CTX])
+        case fabric:delete_db(UserDb, [?ADMIN_CTX]) of
+        ok -> ok;
+        accepted -> ok
+        end
     catch error:database_does_not_exist ->
         ok
     end,
@@ -97,9 +101,12 @@
 ensure_user_db(User) ->
     UserDb = user_db_name(User),
     try
-        fabric:get_db_info(UserDb)
+        {ok, _DbInfo} = fabric:get_db_info(UserDb)
     catch error:database_does_not_exist ->
-        fabric:create_db(UserDb, [?ADMIN_CTX])
+        case fabric:create_db(UserDb, [?ADMIN_CTX]) of
+        ok -> ok;
+        accepted -> ok
+        end
     end,
     UserDb.
 
@@ -149,7 +156,7 @@
     {false, _} ->
         ok;
     {true, SecProps1} ->
-        fabric:set_security(UserDb, {SecProps1}, [?ADMIN_CTX])
+        ok = fabric:set_security(UserDb, {SecProps1}, [?ADMIN_CTX])
     end.
 
 user_db_name(User) ->
@@ -196,10 +203,10 @@
 %% config_listener callbacks
 
 handle_config_change("couch_httpd_auth", "authentication_db", _Value, _Persist, Server) ->
-    gen_server:cast(Server, update_config),
+    ok = gen_server:cast(Server, update_config),
     {ok, Server};
 handle_config_change("couchdb_peruser", _Key, _Value, _Persist, Server) ->
-    gen_server:cast(Server, update_config),
+    ok = gen_server:cast(Server, update_config),
     {ok, Server};
 handle_config_change(_Section, _Key, _Value, _Persist, Server) ->
     {ok, Server}.