Fix lint issues, update deps

Signed-off-by: Thomas Citharel <tcit@tcit.fr>

lib/graphql/resolvers/user.ex

@@ -23,11 +23,19 @@ defmodule Mobilizon.GraphQL.Resolvers.User do
   @spec find_user(any(), map(), Absinthe.Resolution.t()) :: {:ok, User.t()} | {:error, String.t()}
   def find_user(_parent, %{id: id}, %{context: %{current_user: %User{role: role}}})
       when is_moderator(role) do
-    with {:ok, %User{} = user} <- Users.get_user_with_actors(id) do
-      {:ok, user}
+    case Users.get_user_with_actors(id) do
+      {:ok, %User{} = user} ->
+        {:ok, user}
+
+      _ ->
+        {:error, :user_not_found}
     end
   end
 
+  def find_user(_parent, _args, _resolution) do
+    {:error, :unauthorized}
+  end
+
   @doc """
   Return current logged-in user
   """

lib/web/templates/email/activity/_comment_activity_item.html.heex

@@ -2,9 +2,10 @@
   <% :event_comment_mention -> %>
     <%= dgettext("activity", "%{profile} mentionned you in a comment under event %{event}.", %{
       profile: "<b>#{Mobilizon.Actors.Actor.display_name_and_username(@activity.author)}</b>",
-      event: "<a href=\"#{Routes.page_url(Mobilizon.Web.Endpoint,
-      :event,
-      @activity.subject_params["event_uuid"]) |> URI.decode()}\">
+      event:
+        "<a href=\"#{Routes.page_url(Mobilizon.Web.Endpoint,
+        :event,
+        @activity.subject_params["event_uuid"]) |> URI.decode()}\">
                             #{@activity.subject_params["event_title"]}
                         </a>"
     })
@@ -12,9 +13,10 @@
   <% :participation_event_comment -> %>
     <%= dgettext("activity", "%{profile} has posted an announcement under event %{event}.", %{
       profile: "<b>#{Mobilizon.Actors.Actor.display_name_and_username(@activity.author)}</b>",
-      event: "<a href=\"#{Routes.page_url(Mobilizon.Web.Endpoint,
-      :event,
-      @activity.subject_params["event_uuid"]) |> URI.decode()}\">
+      event:
+        "<a href=\"#{Routes.page_url(Mobilizon.Web.Endpoint,
+        :event,
+        @activity.subject_params["event_uuid"]) |> URI.decode()}\">
                             #{@activity.subject_params["event_title"]}
                         </a>"
     })
@@ -23,9 +25,10 @@
     <%= if @activity.subject_params["comment_reply_to"] do %>
       <%= dgettext("activity", "%{profile} has posted a new reply under your event %{event}.", %{
         profile: "<b>#{Mobilizon.Actors.Actor.display_name_and_username(@activity.author)}</b>",
-        event: "<a href=\"#{Routes.page_url(Mobilizon.Web.Endpoint,
-        :event,
-        @activity.subject_params["event_uuid"]) |> URI.decode()}#comment-#{@activity.subject_params["comment_reply_to_uuid"]}-#{@activity.subject_params["comment_uuid"]}\">
+        event:
+          "<a href=\"#{Routes.page_url(Mobilizon.Web.Endpoint,
+          :event,
+          @activity.subject_params["event_uuid"]) |> URI.decode()}#comment-#{@activity.subject_params["comment_reply_to_uuid"]}-#{@activity.subject_params["comment_uuid"]}\">
                                 #{@activity.subject_params["event_title"]}
                             </a>"
       })
@@ -36,9 +39,10 @@
         "%{profile} has posted a new comment under your event %{event}.",
         %{
           profile: "<b>#{Mobilizon.Actors.Actor.display_name_and_username(@activity.author)}</b>",
-          event: "<a href=\"#{Routes.page_url(Mobilizon.Web.Endpoint,
-          :event,
-          @activity.subject_params["event_uuid"]) |> URI.decode()}#comment-#{@activity.subject_params["comment_uuid"]}\">
+          event:
+            "<a href=\"#{Routes.page_url(Mobilizon.Web.Endpoint,
+            :event,
+            @activity.subject_params["event_uuid"]) |> URI.decode()}#comment-#{@activity.subject_params["comment_uuid"]}\">
                                 #{@activity.subject_params["event_title"]}
                             </a>"
         }

lib/web/templates/email/activity/_event_activity_item.html.heex

@@ -2,9 +2,10 @@
   <% :event_created -> %>
     <%= dgettext("activity", "The event %{event} was created by %{profile}.", %{
       profile: "<b>#{Mobilizon.Actors.Actor.display_name_and_username(@activity.author)}</b>",
-      event: "<a href=\"#{Routes.page_url(Mobilizon.Web.Endpoint,
-      :event,
-      @activity.subject_params["event_uuid"]) |> URI.decode()}\">
+      event:
+        "<a href=\"#{Routes.page_url(Mobilizon.Web.Endpoint,
+        :event,
+        @activity.subject_params["event_uuid"]) |> URI.decode()}\">
                             #{@activity.subject_params["event_title"]}
                         </a>"
     })
@@ -12,9 +13,10 @@
   <% :event_updated -> %>
     <%= dgettext("activity", "The event %{event} was updated by %{profile}.", %{
       profile: "<b>#{Mobilizon.Actors.Actor.display_name_and_username(@activity.author)}</b>",
-      event: "<a href=\"#{Routes.page_url(Mobilizon.Web.Endpoint,
-      :event,
-      @activity.subject_params["event_uuid"]) |> URI.decode()}\">
+      event:
+        "<a href=\"#{Routes.page_url(Mobilizon.Web.Endpoint,
+        :event,
+        @activity.subject_params["event_uuid"]) |> URI.decode()}\">
                             #{@activity.subject_params["event_title"]}
                         </a>"
     })
@@ -29,9 +31,10 @@
     <%= if @activity.subject_params["comment_reply_to"] do %>
       <%= dgettext("activity", "%{profile} replied to a comment on the event %{event}.", %{
         profile: "<b>#{Mobilizon.Actors.Actor.display_name_and_username(@activity.author)}</b>",
-        event: "<a href=\"#{Routes.page_url(Mobilizon.Web.Endpoint,
-        :event,
-        @activity.subject_params["event_uuid"]) |> URI.decode()}\">
+        event:
+          "<a href=\"#{Routes.page_url(Mobilizon.Web.Endpoint,
+          :event,
+          @activity.subject_params["event_uuid"]) |> URI.decode()}\">
                             #{@activity.subject_params["event_title"]}
                         </a>"
       })
@@ -39,9 +42,10 @@
     <% else %>
       <%= dgettext("activity", "%{profile} posted a comment on the event %{event}.", %{
         profile: "<b>#{Mobilizon.Actors.Actor.display_name_and_username(@activity.author)}</b>",
-        event: "<a href=\"#{Routes.page_url(Mobilizon.Web.Endpoint,
-        :event,
-        @activity.subject_params["event_uuid"]) |> URI.decode()}\">
+        event:
+          "<a href=\"#{Routes.page_url(Mobilizon.Web.Endpoint,
+          :event,
+          @activity.subject_params["event_uuid"]) |> URI.decode()}\">
                             #{@activity.subject_params["event_title"]}
                         </a>"
       })

lib/web/templates/email/activity/_group_activity_item.html.heex

@@ -2,9 +2,10 @@
   <% :group_created -> %>
     <%= dgettext("activity", "%{profile} created the group %{group}.", %{
       profile: "<b>#{Mobilizon.Actors.Actor.display_name_and_username(@activity.author)}</b>",
-      group: "<a href=\"#{Routes.page_url(Mobilizon.Web.Endpoint,
-      :actor,
-      @activity.subject_params["group_federated_username"]) |> URI.decode()}\">
+      group:
+        "<a href=\"#{Routes.page_url(Mobilizon.Web.Endpoint,
+        :actor,
+        @activity.subject_params["group_federated_username"]) |> URI.decode()}\">
                             #{@activity.subject_params["group_name"]}
                         </a>"
     })
@@ -12,9 +13,10 @@
   <% :group_updated -> %>
     <%= dgettext("activity", "%{profile} updated the group %{group}.", %{
       profile: "<b>#{Mobilizon.Actors.Actor.display_name_and_username(@activity.author)}</b>",
-      group: "<a href=\"#{Routes.page_url(Mobilizon.Web.Endpoint,
-      :actor,
-      @activity.subject_params["group_federated_username"]) |> URI.decode()}\">
+      group:
+        "<a href=\"#{Routes.page_url(Mobilizon.Web.Endpoint,
+        :actor,
+        @activity.subject_params["group_federated_username"]) |> URI.decode()}\">
                             #{@activity.subject_params["group_name"]}
                         </a>"
     })

lib/web/templates/email/activity/_post_activity_item.html.heex

@@ -2,9 +2,10 @@
   <% :post_created -> %>
     <%= dgettext("activity", "The post %{post} was created by %{profile}.", %{
       profile: "<b>#{Mobilizon.Actors.Actor.display_name_and_username(@activity.author)}</b>",
-      post: "<a href=\"#{Routes.page_url(Mobilizon.Web.Endpoint,
-      :post,
-      @activity.subject_params["post_slug"]) |> URI.decode()}\">
+      post:
+        "<a href=\"#{Routes.page_url(Mobilizon.Web.Endpoint,
+        :post,
+        @activity.subject_params["post_slug"]) |> URI.decode()}\">
                             #{@activity.subject_params["post_title"]}
                         </a>"
     })
@@ -12,9 +13,10 @@
   <% :post_updated -> %>
     <%= dgettext("activity", "The post %{post} was updated by %{profile}.", %{
       profile: "<b>#{Mobilizon.Actors.Actor.display_name_and_username(@activity.author)}</b>",
-      post: "<a href=\"#{Routes.page_url(Mobilizon.Web.Endpoint,
-      :post,
-      @activity.subject_params["post_slug"]) |> URI.decode()}\">
+      post:
+        "<a href=\"#{Routes.page_url(Mobilizon.Web.Endpoint,
+        :post,
+        @activity.subject_params["post_slug"]) |> URI.decode()}\">
                             #{@activity.subject_params["post_title"]}
                         </a>"
     })

lib/web/templates/email/activity/_resource_activity_item.html.heex

@@ -3,9 +3,10 @@
     <%= if @activity.subject_params["is_folder"] do %>
       <%= dgettext("activity", "%{profile} created the folder %{resource}.", %{
         profile: "<b>#{Mobilizon.Actors.Actor.display_name_and_username(@activity.author)}</b>",
-        resource: "<a href=\"#{Routes.page_url(Mobilizon.Web.Endpoint,
-        :resource,
-        @activity.subject_params["resource_uuid"]) |> URI.decode()}\">
+        resource:
+          "<a href=\"#{Routes.page_url(Mobilizon.Web.Endpoint,
+          :resource,
+          @activity.subject_params["resource_uuid"]) |> URI.decode()}\">
                                 #{@activity.subject_params["resource_title"]}
                             </a>"
       })
@@ -13,9 +14,10 @@
     <% else %>
       <%= dgettext("activity", "%{profile} created the resource %{resource}.", %{
         profile: "<b>#{Mobilizon.Actors.Actor.display_name_and_username(@activity.author)}</b>",
-        resource: "<a href=\"#{Routes.page_url(Mobilizon.Web.Endpoint,
-        :resource,
-        @activity.subject_params["resource_uuid"]) |> URI.decode()}\">
+        resource:
+          "<a href=\"#{Routes.page_url(Mobilizon.Web.Endpoint,
+          :resource,
+          @activity.subject_params["resource_uuid"]) |> URI.decode()}\">
                                 #{@activity.subject_params["resource_title"]}
                             </a>"
       })
@@ -28,9 +30,10 @@
         "%{profile} renamed the folder from %{old_resource_title} to %{resource}.",
         %{
           profile: "<b>#{Mobilizon.Actors.Actor.display_name_and_username(@activity.author)}</b>",
-          resource: "<a href=\"#{Routes.page_url(Mobilizon.Web.Endpoint,
-          :resource,
-          @activity.subject_params["resource_uuid"]) |> URI.decode()}\">
+          resource:
+            "<a href=\"#{Routes.page_url(Mobilizon.Web.Endpoint,
+            :resource,
+            @activity.subject_params["resource_uuid"]) |> URI.decode()}\">
                                 #{@activity.subject_params["resource_title"]}
                             </a>",
           old_resource_title: "<b>#{@activity.subject_params["old_resource_title"]}</b>"
@@ -43,9 +46,10 @@
         "%{profile} renamed the resource from %{old_resource_title} to %{resource}.",
         %{
           profile: "<b>#{Mobilizon.Actors.Actor.display_name_and_username(@activity.author)}</b>",
-          resource: "<a href=\"#{Routes.page_url(Mobilizon.Web.Endpoint,
-          :resource,
-          @activity.subject_params["resource_uuid"]) |> URI.decode()}\">
+          resource:
+            "<a href=\"#{Routes.page_url(Mobilizon.Web.Endpoint,
+            :resource,
+            @activity.subject_params["resource_uuid"]) |> URI.decode()}\">
                                 #{@activity.subject_params["resource_title"]}
                             </a>",
           old_resource_title: "<b>#{@activity.subject_params["old_resource_title"]}</b>"
@@ -57,9 +61,10 @@
     <%= if @activity.subject_params["is_folder"] do %>
       <%= dgettext("activity", "%{profile} moved the folder %{resource}.", %{
         profile: "<b>#{Mobilizon.Actors.Actor.display_name_and_username(@activity.author)}</b>",
-        resource: "<a href=\"#{Routes.page_url(Mobilizon.Web.Endpoint,
-        :resource,
-        @activity.subject_params["resource_uuid"]) |> URI.decode()}\">
+        resource:
+          "<a href=\"#{Routes.page_url(Mobilizon.Web.Endpoint,
+          :resource,
+          @activity.subject_params["resource_uuid"]) |> URI.decode()}\">
                                 #{@activity.subject_params["resource_title"]}
                             </a>"
       })
@@ -67,9 +72,10 @@
     <% else %>
       <%= dgettext("activity", "%{profile} moved the resource %{resource}.", %{
         profile: "<b>#{Mobilizon.Actors.Actor.display_name_and_username(@activity.author)}</b>",
-        resource: "<a href=\"#{Routes.page_url(Mobilizon.Web.Endpoint,
-        :resource,
-        @activity.subject_params["resource_uuid"]) |> URI.decode()}\">
+        resource:
+          "<a href=\"#{Routes.page_url(Mobilizon.Web.Endpoint,
+          :resource,
+          @activity.subject_params["resource_uuid"]) |> URI.decode()}\">
                                 #{@activity.subject_params["resource_title"]}
                             </a>"
       })

lib/web/templates/email/email_anonymous_activity.html.heex

@@ -51,9 +51,10 @@
                   %{
                     profile:
                       "<b>#{Mobilizon.Actors.Actor.display_name_and_username(@activity.author)}</b>",
-                    event: "<a href=\"#{Routes.page_url(Mobilizon.Web.Endpoint,
-                    :event,
-                    @activity.subject_params["event_uuid"]) |> URI.decode()}\">
+                    event:
+                      "<a href=\"#{Routes.page_url(Mobilizon.Web.Endpoint,
+                      :event,
+                      @activity.subject_params["event_uuid"]) |> URI.decode()}\">
                                 #{@activity.subject_params["event_title"]}
                             </a>"
                   }