From 0cddbe1667d7ae4d52e38aa1302423d99194276c Mon Sep 17 00:00:00 2001 From: Volker Mische Date: Sat, 21 May 2011 14:32:33 +0200 Subject: [PATCH] Rename sendStats config setting to send_stats. For a more consistent code base, I renamed the settings->stats->sendStats config key to send_stats. I think we should follow the rule: Within the Erlang world, use underscores as separator, for the outside world (JSON and POST parameters) use CamelCase. Change-Id: Ifecb0351ef20af51a12e2bd1e88c87aeed586cfd Reviewed-on: http://review.membase.org/6435 Reviewed-by: Aliaksey Kandratsenka Tested-by: Aliaksey Kandratsenka --- src/menelaus_web.erl | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/src/menelaus_web.erl b/src/menelaus_web.erl index 0dda168b67..25da3ba1ac 100644 --- a/src/menelaus_web.erl +++ b/src/menelaus_web.erl @@ -902,10 +902,12 @@ handle_settings_stats(Req) -> build_settings_stats() -> Defaults = default_settings_stats_config(), - ns_config:search_prop(ns_config:get(), settings, stats, Defaults). + [{send_stats, SendStats}] = ns_config:search_prop( + ns_config:get(), settings, stats, Defaults), + [{sendStats, SendStats}]. default_settings_stats_config() -> - [{sendStats, false}]. + [{send_stats, false}]. handle_settings_stats_post(Req) -> PostArgs = Req:parse_post(), @@ -915,7 +917,7 @@ handle_settings_stats_post(Req) -> Req:respond({400, add_header(), "The value of \"sendStats\" must be true or false."}); SendStats2 -> - ns_config:set(settings, [{stats, [{sendStats, SendStats2}]}]), + ns_config:set(settings, [{stats, [{send_stats, SendStats2}]}]), Req:respond({200, add_header(), []}) end.