code review update - minor refactor of process_parts

Signed-off-by: Marc Paradise <[email protected]>

src/oc_erchef/apps/oc_chef_authz/src/oc_chef_authz_acl.erl

@@ -214,7 +214,7 @@ fetch(Type, AuthzId, Granular) ->
     Result = oc_chef_authz_http:request(Path, get, ?DEFAULT_HEADERS, [], SuperuserId),
     case Result of
         {ok, Record} ->
-            ids_to_names(Record, Granular);
+            convert_all_ids_to_names(Record, Granular);
         {error, forbidden} ->
             forbidden;
         Other ->
@@ -334,15 +334,15 @@ convert_actor_ids_to_names(AuthzIds) ->
         oc_chef_group:find_users_names(RemainingAuthzIds, fun chef_sql:select_rows/1),
     {ClientNames, UserNames, DefunctActorAuthzIds}.
 
-ids_to_names(Record, Granular) ->
-    Record1 = process_part(<<"create">>, Record, Granular),
-    Record2 = process_part(<<"read">>, Record1, Granular),
-    Record3 = process_part(<<"update">>, Record2, Granular),
-    Record4 = process_part(<<"delete">>, Record3, Granular),
-    process_part(<<"grant">>, Record4, Granular).
 
--spec process_part(binary(), ejson_term(), granular|undefined) -> ejson_term().
-process_part(Part, Record, Granular) ->
+convert_all_ids_to_names(Record, Granular) ->
+    convert_ids_to_names_in_part([<<"create">>, <<"read">>, <<"update">>, <<"delete">>],
+                                 Record, Granular).
+
+-spec convert_ids_to_names_in_part(list(binary()), ejson_term(), granular|undefined) -> ejson_term().
+convert_ids_to_names_in_part([], Record, _Granular) ->
+    Record;
+convert_ids_to_names_in_part([Part | Rest], Record, Granular) ->
     Members = ej:get({Part}, Record),
     ActorIds = ej:get({<<"actors">>}, Members),
     GroupIds = ej:get({<<"groups">>}, Members),
@@ -351,7 +351,9 @@ process_part(Part, Record, Granular) ->
     oc_chef_authz_cleanup:add_authz_ids(DefunctActorAuthzIds, DefunctGroupAuthzIds),
     Members1 = part_with_actors(Members, ClientNames, UserNames, Granular),
     Members2 = ej:set({<<"groups">>}, Members1, GroupNames),
-    ej:set({Part}, Record, Members2).
+    NewRecord = ej:set({Part}, Record, Members2),
+    convert_ids_to_names_in_part(Rest, NewRecord, Granular).
+
 
 part_with_actors(PartRecord, Clients, Users, granular) ->
     PartRecord0 = ej:set({<<"users">>}, PartRecord, Users),