feat(activitypub): allow simple text for address field

Closes #1387

Signed-off-by: Thomas Citharel <tcit@tcit.fr>
This commit is contained in:
Thomas Citharel 2023-12-19 10:52:31 +01:00
parent 8544278c9a
commit 64237cfc26
No known key found for this signature in database
GPG key ID: A061B9DDE0CA0773
3 changed files with 93 additions and 12 deletions

View file

@ -24,16 +24,23 @@ defmodule Mobilizon.Federation.ActivityStream.Converter.Address do
}
res =
if is_nil(object["address"]) or not is_map(object["address"]) do
res
else
Map.merge(res, %{
"country" => object["address"]["addressCountry"],
"postal_code" => object["address"]["postalCode"],
"region" => object["address"]["addressRegion"],
"street" => object["address"]["streetAddress"],
"locality" => object["address"]["addressLocality"]
})
cond do
is_binary(object["address"]) ->
Map.merge(res, %{
"street" => object["address"]
})
is_map(object["address"]) ->
Map.merge(res, %{
"country" => object["address"]["addressCountry"],
"postal_code" => object["address"]["postalCode"],
"region" => object["address"]["addressRegion"],
"street" => object["address"]["streetAddress"],
"locality" => object["address"]["addressLocality"]
})
is_nil(object["address"]) ->
res
end
latitude = Map.get(object, "latitude")

View file

@ -278,8 +278,8 @@ defmodule Mobilizon.Federation.ActivityStream.Converter.Utils do
end
@spec get_address(map | binary | nil) :: Address.t() | nil
def get_address(address_url) when is_binary(address_url) do
get_address(%{"id" => address_url})
def get_address(text_address) when is_binary(text_address) do
get_address(%{"type" => "Place", "name" => text_address})
end
def get_address(%{"id" => url} = map) when is_map(map) and is_binary(url) do

View file

@ -0,0 +1,74 @@
defmodule Mobilizon.Federation.ActivityStream.Converter.AddressTest do
use Mobilizon.DataCase
alias Mobilizon.Addresses.Address
alias Mobilizon.Federation.ActivityStream.Converter.Address, as: AddressConverter
describe "address to AS" do
test "valid address to as" do
data =
AddressConverter.model_to_as(%Address{
country: "France",
locality: "Lyon",
description: "Locaux'Motiv",
postal_code: "69007",
street: "10Bis Rue Jangot"
})
assert is_map(data)
assert data["type"] == "Place"
assert data["name"] == "Locaux'Motiv"
assert data["address"]["type"] == "PostalAddress"
assert data["address"]["addressLocality"] == "Lyon"
assert data["address"]["postalCode"] == "69007"
assert data["address"]["addressCountry"] == "France"
assert data["address"]["streetAddress"] == "10Bis Rue Jangot"
end
end
describe "AS to Address" do
test "basic AS data to model" do
address =
AddressConverter.as_to_model_data(%{
"type" => "Place",
"name" => "Federazione Anarchica Torinese",
"address" => "corso Palermo 46",
"latitude" => nil,
"longitude" => nil
})
assert address["description"] == "Federazione Anarchica Torinese"
assert address["street"] == "corso Palermo 46"
assert address["locality"] == nil
assert address["geom"] == nil
end
test "AS data with PostalAddress to model" do
address =
AddressConverter.as_to_model_data(%{
"type" => "Place",
"name" => "Federazione Anarchica Torinese",
"address" => %{
"streetAddress" => "Corso Palermo, 46",
"addressCountry" => "Italia",
"postalCode" => "10152",
"addressLocality" => "Torino"
},
"latitude" => 45.08281,
"longitude" => 7.69311
})
assert address["description"] == "Federazione Anarchica Torinese"
assert address["street"] == "Corso Palermo, 46"
assert address["locality"] == "Torino"
assert address["postal_code"] == "10152"
assert address["country"] == "Italia"
assert address["geom"] == %Geo.Point{
coordinates: {7.69311, 45.08281},
srid: 4326
}
end
end
end