yuanbiao
/
emqx
1
0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fix: elvis warning

This commit is contained in:
Zhongwen Deng 2022-03-08 09:05:23 +08:00
parent 9b00f2756b
commit a7ba678463
1 changed files with 6 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
apps/emqx/src/emqx_config_handler.erl
View File

@ -76,7 +76,8 @@ update_config(SchemaModule, ConfKeyPath, UpdateArgs) ->
AtomKeyPath = [atom(Key) || Key <- ConfKeyPath],
gen_server:call(?MODULE, {change_config, SchemaModule, AtomKeyPath, UpdateArgs}, infinity).
-spec add_handler(emqx_config:config_key_path(), handler_name()) -> ok | {error, {conflict, list()}}.
-spec add_handler(emqx_config:config_key_path(), handler_name()) ->
ok | {error, {conflict, list()}}.
add_handler(ConfKeyPath, HandlerName) ->
assert_callback_function(HandlerName),
gen_server:call(?MODULE, {add_handler, ConfKeyPath, HandlerName}).
@ -149,6 +150,10 @@ deep_put_handler([Key | KeyPath], Handlers, Mod) ->
end.
%% Make sure that Specify Key and ?WKEY cannot be on the same level.
%%
%% [k1, ?, ?], [k1, ?], [k1] is allow.
%% [K1, ?, k2], [k1, ?, k3] is allow.
%% [k1, ?, ?], [k1, ?, k2] is not allow.
check_handler_conflict(Handlers) ->
Keys = filter_top_level_handlers(Handlers),
case lists:member(?WKEY, Keys) of