From 07a11d792c3f4c120b771b080e83703d5c4d10c3 Mon Sep 17 00:00:00 2001 From: Thomas Citharel Date: Sat, 20 Nov 2021 18:44:55 +0100 Subject: [PATCH] Fix updating a group's avatar and banner picture Signed-off-by: Thomas Citharel --- lib/graphql/resolvers/group.ex | 13 ++++++++++--- 1 file changed, 10 insertions(+), 3 deletions(-) diff --git a/lib/graphql/resolvers/group.ex b/lib/graphql/resolvers/group.ex index d0532067e..fed9d320f 100644 --- a/lib/graphql/resolvers/group.ex +++ b/lib/graphql/resolvers/group.ex @@ -120,9 +120,11 @@ defmodule Mobilizon.GraphQL.Resolvers.Group do with {:ok, %{name: name, url: url, content_type: content_type, size: _size}} <- Upload.store(pic.file, type: key, description: pic.alt) do - Map.put(args, key, %{"name" => name, "url" => url, "mediaType" => content_type}) + Logger.debug("Uploaded #{name} to #{url}") + Map.put(args, key, %{name: name, url: url, content_type: content_type}) end else + Logger.debug("No picture upload") args end end) @@ -200,12 +202,17 @@ defmodule Mobilizon.GraphQL.Resolvers.Group do {:error, :file_too_large} -> {:error, dgettext("errors", "The provided picture is too heavy")} - map when is_map(map) -> + args when is_map(args) -> case API.Groups.update_group(args) do {:ok, _activity, %Actor{type: :Group} = group} -> {:ok, group} - {:error, _err} -> + {:error, %Ecto.Changeset{} = changeset} -> + {:error, changeset} + + {:error, err} -> + Logger.info("Failed to update group #{inspect(group_id)}") + Logger.debug(inspect(err)) {:error, dgettext("errors", "Failed to update the group")} end end