MastoAPI: Make attachment ids strings.
authorRoger Braun <roger@rogerbraun.net>
Wed, 15 Nov 2017 17:58:13 +0000 (18:58 +0100)
committerRoger Braun <roger@rogerbraun.net>
Wed, 15 Nov 2017 17:58:39 +0000 (18:58 +0100)
lib/pleroma/web/mastodon_api/views/status_view.ex
test/web/mastodon_api/status_view_test.exs

index d97b2acb4c502d712c475416893b9bc17e8c95ac..38abdb35f4889ae7fbbf904fbed360201a61f37e 100644 (file)
@@ -120,7 +120,7 @@ defmodule Pleroma.Web.MastodonAPI.StatusView do
     << hash_id::signed-32, _rest::binary >> = :crypto.hash(:md5, href)
 
     %{
-      id: attachment["id"] || hash_id,
+      id: to_string(attachment["id"] || hash_id),
       url: href,
       remote_url: href,
       preview_url: href,
index 601e551a9596218de1e33688b2dd2099ac0716e2..93c0b7236ec7cefd3742a09cfc78283bef7fe369 100644 (file)
@@ -78,7 +78,7 @@ defmodule Pleroma.Web.MastodonAPI.StatusViewTest do
     }
 
     expected = %{
-      id: 1638338801,
+      id: "1638338801",
       type: "image",
       url: "someurl",
       remote_url: "someurl",
@@ -90,7 +90,7 @@ defmodule Pleroma.Web.MastodonAPI.StatusViewTest do
 
     # If theres a "id", use that instead of the generated one
     object = Map.put(object, "id", 2)
-    assert %{id: 2} = StatusView.render("attachment.json", %{attachment: object})
+    assert %{id: "2"} = StatusView.render("attachment.json", %{attachment: object})
   end
 
   test "a reblog" do