{:ok, activities}
end
- def make_share(_entry, doc, retweeted_activity) do
+ def make_share(entry, doc, retweeted_activity) do
with {:ok, actor} <- find_make_or_update_user(doc),
- %Object{} = object <- Object.get_cached_by_ap_id(retweeted_activity.data["object"]["id"]),
- {:ok, activity, _object} = ActivityPub.announce(actor, object, false) do
+ %Object{} = object <- Object.get_by_ap_id(retweeted_activity.data["object"]["id"]),
+ id when not is_nil(id) <- string_from_xpath("/entry/id", entry),
+ {:ok, activity, _object} = ActivityPub.announce(actor, object, id, false) do
{:ok, activity}
end
end
end
end
- def make_favorite(_entry, doc, favorited_activity) do
+ def make_favorite(entry, doc, favorited_activity) do
with {:ok, actor} <- find_make_or_update_user(doc),
- %Object{} = object <- Object.get_cached_by_ap_id(favorited_activity.data["object"]["id"]),
- {:ok, activity, _object} = ActivityPub.like(actor, object, false) do
+ %Object{} = object <- Object.get_by_ap_id(favorited_activity.data["object"]["id"]),
+ id when not is_nil(id) <- string_from_xpath("/entry/id", entry),
+ {:ok, activity, _object} = ActivityPub.like(actor, object, id, false) do
{:ok, activity}
end
end
# TODO: Bail out sooner and use transaction.
if Object.get_by_ap_id(id) do
- {:error, "duplicate activity"}
+ {:ok, Activity.get_create_activity_by_object_ap_id(id)}
else
ActivityPub.create(to, actor, context, object, %{}, date, false)
end
uri = string_from_xpath("//author/uri[1]", doc)
with {:ok, user} <- find_or_make_user(uri) do
avatar = make_avatar_object(doc)
- if user.avatar != avatar do
+ if !user.local && user.avatar != avatar do
change = Ecto.Changeset.change(user, %{avatar: avatar})
Repo.update(change)
else