I’ve got this function, translated from an old codebase (imperative language) into Elixir, by our contractor.

  def pub0(payload) when is_map(payload) do
    if ["country", "commodity", "share", "mon", "cog", "esa1"] |> Enum.all?(fn x -> Map.has_key?(payload, x) end) &&
        Map.get(payload, "esa1") == "hs10" &&
        is_intger(Map.get(payload, "cog") do
      payload = Map.put(payload, "itime", utimestamp(2))
      emporio_ts(Jason.encode!(payload))
    end
  end

The function works fine, it performs its job, but I don’t like the code structure:

There is an if with tree conditions that have to evaluate all to true (logically: if (true & true & true) then …). If satisfices this, then adds new property to the object and finally calls another function.

It does not seem functional. Is there a way to refactory the function in a (more) elegant and functional manner? And also more comprehensible, consistent, shorter ?

Thanks!