fix(emqx_rule_runtime): Dialyzer warnings
This commit is contained in:
parent
42ed274ec3
commit
ff55b0ff45
|
@ -49,7 +49,7 @@ apply_rules([], _Input) ->
|
||||||
apply_rules([#rule{enabled = false}|More], Input) ->
|
apply_rules([#rule{enabled = false}|More], Input) ->
|
||||||
apply_rules(More, Input);
|
apply_rules(More, Input);
|
||||||
apply_rules([Rule = #rule{id = RuleID}|More], Input) ->
|
apply_rules([Rule = #rule{id = RuleID}|More], Input) ->
|
||||||
try apply_rule(Rule, Input)
|
try apply_rule_discard_result(Rule, Input)
|
||||||
catch
|
catch
|
||||||
%% ignore the errors if select or match failed
|
%% ignore the errors if select or match failed
|
||||||
_:{select_and_transform_error, Error} ->
|
_:{select_and_transform_error, Error} ->
|
||||||
|
@ -70,6 +70,13 @@ apply_rules([Rule = #rule{id = RuleID}|More], Input) ->
|
||||||
end,
|
end,
|
||||||
apply_rules(More, Input).
|
apply_rules(More, Input).
|
||||||
|
|
||||||
|
apply_rule_discard_result(Rule, Input) ->
|
||||||
|
%% TODO check if below two clauses are ok to discard:
|
||||||
|
%% {'error','nomatch'}
|
||||||
|
%% {'ok',[any()]}
|
||||||
|
_ = apply_rule(Rule, Input),
|
||||||
|
ok.
|
||||||
|
|
||||||
apply_rule(Rule = #rule{id = RuleID}, Input) ->
|
apply_rule(Rule = #rule{id = RuleID}, Input) ->
|
||||||
clear_rule_payload(),
|
clear_rule_payload(),
|
||||||
do_apply_rule(Rule, add_metadata(Input, #{rule_id => RuleID})).
|
do_apply_rule(Rule, add_metadata(Input, #{rule_id => RuleID})).
|
||||||
|
@ -160,6 +167,7 @@ select_and_collect([Field|More], Input, {Output, LastKV}) ->
|
||||||
{nested_put(Key, Val, Output), LastKV}).
|
{nested_put(Key, Val, Output), LastKV}).
|
||||||
|
|
||||||
%% Filter each item got from FOREACH
|
%% Filter each item got from FOREACH
|
||||||
|
-dialyzer({nowarn_function, filter_collection/4}).
|
||||||
filter_collection(Input, InCase, DoEach, {CollKey, CollVal}) ->
|
filter_collection(Input, InCase, DoEach, {CollKey, CollVal}) ->
|
||||||
lists:filtermap(
|
lists:filtermap(
|
||||||
fun(Item) ->
|
fun(Item) ->
|
||||||
|
|
Loading…
Reference in New Issue