Clenup event handler

Author: ziopio

src/grisp_connect_app.erl

@@ -20,13 +20,18 @@ start(_StartType, _StartArgs) ->
     logger:add_handlers(grisp_connect),
     case grisp_connect_utils:using_grisp_netman() of
         true ->
-            grisp_connect_netman:add_handler();
+            grisp_netman:add_event_handler(grisp_connect_netman_handler, []);
         false ->
             ok
     end,
     grisp_connect_sup:start_link().
 
 stop(_State) ->
-    ok.
+    case grisp_connect_utils:using_grisp_netman() of
+        true ->
+            grisp_netman:delete_event_handler(grisp_connect_netman_handler, []);
+        false ->
+            ok
+    end.
 
 %--- Internal Functions --------------------------------------------------------

src/grisp_connect_netman.erl renamed to src/grisp_connect_netman_handler.erl

@@ -1,21 +1,17 @@
--module(grisp_connect_netman).
+-module(grisp_connect_netman_handler).
+-moduledoc """
+Handler for grisp_netman events.
+""".
 
 -behaviour(gen_event).
 
--export([add_handler/0, remove_handler/0]).
 -export([init/1, handle_event/2, handle_call/2]).
 
 -include_lib("kernel/include/logger.hrl").
 
-add_handler() ->
-    gen_event:add_handler(grisp_netman_event, ?MODULE, []).
-
-remove_handler() ->
-    gen_event:delete_handler(grisp_netman_event, ?MODULE, []).
-
 % Behaviour gen_event callbacks ------------------------------------------------
 
-init([]) ->
+init(_) ->
     {ok, #{}}.
 
 handle_event({connection_status, _IfName, Status}, State) ->
@@ -36,5 +32,5 @@ handle_event({connection_status, _IfName, Status}, State) ->
     {ok, State}.
 
 handle_call(Request, State) ->
-    ?LOG_WARNING("Unexpected grisp_netman_event call: ~p", [Request]),
+    ?LOG_WARNING("Unexpected ~p call: ~p", [?MODULE, Request]),
     {reply, unexpected_call, State}.