Use elixir format

Author: Adrián Quintás

Diff for: .formatter.exs

@@ -0,0 +1,7 @@
+[
+  inputs: [
+    "{lib,config,test}/**/*.{ex,exs}",
+    "mix.exs"
+  ]
+]
+

Diff for: config/config.exs

@@ -31,4 +31,4 @@ config :exsolr,
 # Configuration from the imported file will override the ones defined
 # here (which is why it is important to import them last).
 #
-import_config "#{Mix.env}.exs"
+import_config "#{Mix.env()}.exs"

Diff for: lib/exsolr.ex

@@ -16,7 +16,7 @@ defmodule Exsolr do
       %{hostname: "localhost", port: 8983, core: "elixir_test"}
   """
   def info do
-    Config.info
+    Config.info()
   end
 
   @doc """
@@ -48,7 +48,7 @@ defmodule Exsolr do
   Commits the pending changes to Solr
   """
   def commit do
-    Indexer.commit
+    Indexer.commit()
   end
 
   @doc """
@@ -64,6 +64,6 @@ defmodule Exsolr do
   https://wiki.apache.org/solr/FAQ#How_can_I_delete_all_documents_from_my_index.3F
   """
   def delete_all do
-    Indexer.delete_all
+    Indexer.delete_all()
   end
 end

Diff for: lib/exsolr/config.ex

@@ -46,4 +46,3 @@ defmodule Exsolr.Config do
 
   defp base_url, do: "http://#{hostname}:#{port}/solr/#{core}"
 end
-

Diff for: lib/exsolr/http_response.ex

@@ -8,13 +8,15 @@ defmodule Exsolr.HttpResponse do
   def body({status, response}) do
     case {status, response} do
       {:ok, %HTTPoison.Response{status_code: 200, body: response_body}} ->
-        _ = Logger.debug response_body
+        _ = Logger.debug(response_body)
         response_body
+
       {:ok, %HTTPoison.Response{status_code: status_code, body: response_body}} ->
-        _ = Logger.warn "status_code: #{status_code} - #{response_body}"
+        _ = Logger.warn("status_code: #{status_code} - #{response_body}")
         nil
+
       {_, %HTTPoison.Error{id: _, reason: error_reason}} ->
-        _ = Logger.error error_reason
+        _ = Logger.error(error_reason)
         nil
     end
   end

Diff for: lib/exsolr/indexer.ex

@@ -9,7 +9,7 @@ defmodule Exsolr.Indexer do
   def add(document) do
     json_docs_update_url
     |> HTTPoison.post(encode(document), json_headers)
-    |> HttpResponse.body
+    |> HttpResponse.body()
   end
 
   @doc """
@@ -47,9 +47,9 @@ defmodule Exsolr.Indexer do
   end
 
   defp update_request(headers, body) do
-    Config.update_url
+    Config.update_url()
     |> HTTPoison.post(body, headers)
-    |> HttpResponse.body
+    |> HttpResponse.body()
   end
 
   defp json_headers, do: [{"Content-Type", "application/json"}]
@@ -67,25 +67,27 @@ defmodule Exsolr.Indexer do
       ~s({"delete":{"id":"42"}})
 
   """
-  def delete_by_id_json_body(id) when is_integer(id)  do
+  def delete_by_id_json_body(id) when is_integer(id) do
     id
     |> Integer.to_string()
     |> delete_by_id_json_body
   end
+
   def delete_by_id_json_body(id) do
-    {:ok, body} = %{delete: %{id: id}}
-                  |> Poison.encode
+    {:ok, body} =
+      %{delete: %{id: id}}
+      |> Poison.encode()
 
     body
   end
 
   defp delete_all_xml_body, do: "<delete><query>*:*</query></delete>"
   defp commit_xml_body, do: "<commit/>"
 
-  defp json_docs_update_url, do: "#{Config.update_url}/json/docs"
+  defp json_docs_update_url, do: "#{Config.update_url()}/json/docs"
 
   defp encode(document) do
     {:ok, body} = Poison.encode(document)
     body
   end
-end
+end

Diff for: lib/exsolr/query.ex

@@ -0,0 +1 @@
+

Diff for: lib/exsolr/searcher.ex

@@ -51,7 +51,7 @@ defmodule Exsolr.Searcher do
   defp build_solr_query_params(params) do
     params
     |> add_default_params
-    |> Enum.map(fn({key, value}) -> build_solr_query_parameter(key, value) end)
+    |> Enum.map(fn {key, value} -> build_solr_query_parameter(key, value) end)
     |> Enum.join("&")
   end
 
@@ -65,14 +65,17 @@ defmodule Exsolr.Searcher do
   end
 
   defp build_solr_query_parameter(_, []), do: nil
-  defp build_solr_query_parameter(key, [head|tail]) do
+
+  defp build_solr_query_parameter(key, [head | tail]) do
     [build_solr_query_parameter(key, head), build_solr_query_parameter(key, tail)]
-    |> Enum.reject(fn(x) -> x == nil end)
+    |> Enum.reject(fn x -> x == nil end)
     |> Enum.join("&")
   end
+
   defp build_solr_query_parameter(:q, value) do
     "q=#{URI.encode_www_form(value)}"
   end
+
   defp build_solr_query_parameter(key, value) do
     [Atom.to_string(key), value]
     |> Enum.join("=")
@@ -81,26 +84,28 @@ defmodule Exsolr.Searcher do
   def do_search(solr_query) do
     solr_query
     |> build_solr_url
-    |> HTTPoison.get
-    |> HttpResponse.body
+    |> HTTPoison.get()
+    |> HttpResponse.body()
   end
 
   defp build_solr_url(solr_query) do
-    url = Config.select_url <> solr_query
-    _ = Logger.debug url
+    url = Config.select_url() <> solr_query
+    _ = Logger.debug(url)
     url
   end
 
   defp extract_response(solr_response) do
-    case solr_response |> Poison.decode do
-      {:ok, %{"response" => response, "moreLikeThis" => moreLikeThis}} -> Map.put(response, "mlt", extract_mlt_result(moreLikeThis))
-      {:ok, %{"response" => response}} -> response
+    case solr_response |> Poison.decode() do
+      {:ok, %{"response" => response, "moreLikeThis" => moreLikeThis}} ->
+        Map.put(response, "mlt", extract_mlt_result(moreLikeThis))
+
+      {:ok, %{"response" => response}} ->
+        response
     end
   end
 
-  defp extract_mlt_result(mlt) do 
-    result =
-    for k <- Map.keys(mlt), do: get_in(mlt, [k, "docs"])
-    result |> List.flatten
+  defp extract_mlt_result(mlt) do
+    result = for k <- Map.keys(mlt), do: get_in(mlt, [k, "docs"])
+    result |> List.flatten()
   end
 end

Diff for: mix.exs

@@ -6,11 +6,11 @@ defmodule Exsolr.Mixfile do
       app: :exsolr,
       version: "0.0.1",
       elixir: "~> 1.2",
-      build_embedded: Mix.env == :prod,
-      start_permanent: Mix.env == :prod,
+      build_embedded: Mix.env() == :prod,
+      start_permanent: Mix.env() == :prod,
       description: description,
       package: package,
-      deps: deps,
+      deps: deps
     ]
   end
 
@@ -25,8 +25,10 @@ defmodule Exsolr.Mixfile do
       files: ["lib", "mix.exs", "README.md"],
       maintainers: ["Daniel Carneiro"],
       licenses: ["MIT License (MIT)"],
-      links: %{"GitHub" => "https://github.com/dcarneiro/exsolr",
-              "Docs" => "http://hexdocs.pm/exsolr/"}
+      links: %{
+        "GitHub" => "https://github.com/dcarneiro/exsolr",
+        "Docs" => "http://hexdocs.pm/exsolr/"
+      }
     ]
   end
 
@@ -53,7 +55,7 @@ defmodule Exsolr.Mixfile do
       {:credo, "~> 0.3", only: [:dev, :test]},
       {:earmark, "~> 0.1", only: :dev},
       {:ex_doc, "~> 0.11", only: :dev},
-      {:dialyxir, "~> 0.3", only: :dev},
+      {:dialyxir, "~> 0.3", only: :dev}
     ]
   end
 end

Diff for: test/exsolr_test.exs

@@ -7,7 +7,7 @@ defmodule ExsolrTest do
   end
 
   test "add a document" do
-    document = %{id: "3", title: "Doc 3" }
+    document = %{id: "3", title: "Doc 3"}
     Exsolr.add(document)
   end
 
@@ -16,6 +16,6 @@ defmodule ExsolrTest do
   end
 
   test "commit" do
-    assert Exsolr.commit
+    assert Exsolr.commit()
   end
 end