diff options
| -rw-r--r-- | lib/pleroma/user.ex | 3 | ||||
| -rw-r--r-- | lib/pleroma/web/activity_pub/activity_pub.ex | 1 | ||||
| -rw-r--r-- | priv/repo/migrations/20230504173400_remove_user_ap_enabled.exs | 13 | ||||
| -rw-r--r-- | test/pleroma/user_test.exs | 11 | ||||
| -rw-r--r-- | test/pleroma/web/activity_pub/activity_pub_controller_test.exs | 1 | ||||
| -rw-r--r-- | test/pleroma/web/activity_pub/activity_pub_test.exs | 1 | ||||
| -rw-r--r-- | test/pleroma/web/activity_pub/publisher_test.exs | 15 | ||||
| -rw-r--r-- | test/pleroma/web/common_api_test.exs | 2 | ||||
| -rw-r--r-- | test/pleroma/web/federator_test.exs | 6 | ||||
| -rw-r--r-- | test/support/factory.ex | 1 | 
10 files changed, 24 insertions, 30 deletions
diff --git a/lib/pleroma/user.ex b/lib/pleroma/user.ex index eff82afd9..ce125d608 100644 --- a/lib/pleroma/user.ex +++ b/lib/pleroma/user.ex @@ -124,7 +124,6 @@ defmodule Pleroma.User do      field(:domain_blocks, {:array, :string}, default: [])      field(:is_active, :boolean, default: true)      field(:no_rich_text, :boolean, default: false) -    field(:ap_enabled, :boolean, default: false)      field(:is_moderator, :boolean, default: false)      field(:is_admin, :boolean, default: false)      field(:show_role, :boolean, default: true) @@ -488,7 +487,6 @@ defmodule Pleroma.User do          :nickname,          :public_key,          :avatar, -        :ap_enabled,          :banner,          :is_locked,          :last_refreshed_at, @@ -1894,7 +1892,6 @@ defmodule Pleroma.User do        confirmation_token: nil,        domain_blocks: [],        is_active: false, -      ap_enabled: false,        is_moderator: false,        is_admin: false,        mascot: nil, diff --git a/lib/pleroma/web/activity_pub/activity_pub.ex b/lib/pleroma/web/activity_pub/activity_pub.ex index 9175e9e13..1a10aef1d 100644 --- a/lib/pleroma/web/activity_pub/activity_pub.ex +++ b/lib/pleroma/web/activity_pub/activity_pub.ex @@ -1547,7 +1547,6 @@ defmodule Pleroma.Web.ActivityPub.ActivityPub do      %{        ap_id: data["id"],        uri: get_actor_url(data["url"]), -      ap_enabled: true,        banner: normalize_image(data["image"]),        fields: fields,        emoji: emojis, diff --git a/priv/repo/migrations/20230504173400_remove_user_ap_enabled.exs b/priv/repo/migrations/20230504173400_remove_user_ap_enabled.exs new file mode 100644 index 000000000..0aea41324 --- /dev/null +++ b/priv/repo/migrations/20230504173400_remove_user_ap_enabled.exs @@ -0,0 +1,13 @@ +# Pleroma: A lightweight social networking server +# Copyright © 2017-2023 Pleroma Authors <https://pleroma.social/> +# SPDX-License-Identifier: AGPL-3.0-only + +defmodule Pleroma.Repo.Migrations.RemoveUserApEnabled do +  use Ecto.Migration + +  def change do +    alter table(:users) do +      remove(:ap_enabled, :boolean, default: false, null: false) +    end +  end +end diff --git a/test/pleroma/user_test.exs b/test/pleroma/user_test.exs index c16312a65..7f60b959a 100644 --- a/test/pleroma/user_test.exs +++ b/test/pleroma/user_test.exs @@ -1844,7 +1844,6 @@ defmodule Pleroma.UserTest do          confirmation_token: "qqqq",          domain_blocks: ["lain.com"],          is_active: false, -        ap_enabled: true,          is_moderator: true,          is_admin: true,          mascot: %{"a" => "b"}, @@ -1885,7 +1884,6 @@ defmodule Pleroma.UserTest do               confirmation_token: nil,               domain_blocks: [],               is_active: false, -             ap_enabled: false,               is_moderator: false,               is_admin: false,               mascot: nil, @@ -2473,8 +2471,7 @@ defmodule Pleroma.UserTest do          insert(:user,            local: false,            follower_address: "http://localhost:4001/users/masto_closed/followers", -          following_address: "http://localhost:4001/users/masto_closed/following", -          ap_enabled: true +          following_address: "http://localhost:4001/users/masto_closed/following"          )        assert other_user.following_count == 0 @@ -2495,8 +2492,7 @@ defmodule Pleroma.UserTest do          insert(:user,            local: false,            follower_address: "http://localhost:4001/users/masto_closed/followers", -          following_address: "http://localhost:4001/users/masto_closed/following", -          ap_enabled: true +          following_address: "http://localhost:4001/users/masto_closed/following"          )        assert other_user.following_count == 0 @@ -2517,8 +2513,7 @@ defmodule Pleroma.UserTest do          insert(:user,            local: false,            follower_address: "http://localhost:4001/users/masto_closed/followers", -          following_address: "http://localhost:4001/users/masto_closed/following", -          ap_enabled: true +          following_address: "http://localhost:4001/users/masto_closed/following"          )        assert other_user.following_count == 0 diff --git a/test/pleroma/web/activity_pub/activity_pub_controller_test.exs b/test/pleroma/web/activity_pub/activity_pub_controller_test.exs index ef91066c1..62eb9b5a3 100644 --- a/test/pleroma/web/activity_pub/activity_pub_controller_test.exs +++ b/test/pleroma/web/activity_pub/activity_pub_controller_test.exs @@ -575,7 +575,6 @@ defmodule Pleroma.Web.ActivityPub.ActivityPubControllerTest do        user =          insert(:user,            ap_id: "https://mastodon.example.org/users/raymoo", -          ap_enabled: true,            local: false,            last_refreshed_at: nil          ) diff --git a/test/pleroma/web/activity_pub/activity_pub_test.exs b/test/pleroma/web/activity_pub/activity_pub_test.exs index a6f8b6152..54fc6ef0a 100644 --- a/test/pleroma/web/activity_pub/activity_pub_test.exs +++ b/test/pleroma/web/activity_pub/activity_pub_test.exs @@ -174,7 +174,6 @@ defmodule Pleroma.Web.ActivityPub.ActivityPubTest do        {:ok, user} = ActivityPub.make_user_from_ap_id(user_id)        assert user.ap_id == user_id        assert user.nickname == "admin@mastodon.example.org" -      assert user.ap_enabled        assert user.follower_address == "http://mastodon.example.org/users/admin/followers"      end diff --git a/test/pleroma/web/activity_pub/publisher_test.exs b/test/pleroma/web/activity_pub/publisher_test.exs index e2db3d575..c5137cbb7 100644 --- a/test/pleroma/web/activity_pub/publisher_test.exs +++ b/test/pleroma/web/activity_pub/publisher_test.exs @@ -276,8 +276,7 @@ defmodule Pleroma.Web.ActivityPub.PublisherTest do        follower =          insert(:user, %{            local: false, -          inbox: "https://domain.com/users/nick1/inbox", -          ap_enabled: true +          inbox: "https://domain.com/users/nick1/inbox"          })        actor = insert(:user, follower_address: follower.ap_id) @@ -313,8 +312,7 @@ defmodule Pleroma.Web.ActivityPub.PublisherTest do        follower =          insert(:user, %{            local: false, -          inbox: "https://domain.com/users/nick1/inbox", -          ap_enabled: true +          inbox: "https://domain.com/users/nick1/inbox"          })        actor = insert(:user, follower_address: follower.ap_id) @@ -348,8 +346,7 @@ defmodule Pleroma.Web.ActivityPub.PublisherTest do        follower =          insert(:user, %{            local: false, -          inbox: "https://domain.com/users/nick1/inbox", -          ap_enabled: true +          inbox: "https://domain.com/users/nick1/inbox"          })        actor = insert(:user, follower_address: follower.ap_id) @@ -382,15 +379,13 @@ defmodule Pleroma.Web.ActivityPub.PublisherTest do        fetcher =          insert(:user,            local: false, -          inbox: "https://domain.com/users/nick1/inbox", -          ap_enabled: true +          inbox: "https://domain.com/users/nick1/inbox"          )        another_fetcher =          insert(:user,            local: false, -          inbox: "https://domain2.com/users/nick1/inbox", -          ap_enabled: true +          inbox: "https://domain2.com/users/nick1/inbox"          )        actor = insert(:user) diff --git a/test/pleroma/web/common_api_test.exs b/test/pleroma/web/common_api_test.exs index 5c9103e9f..5bd0df0b2 100644 --- a/test/pleroma/web/common_api_test.exs +++ b/test/pleroma/web/common_api_test.exs @@ -1328,7 +1328,7 @@ defmodule Pleroma.Web.CommonAPITest do      test "cancels a pending follow for a remote user" do        follower = insert(:user) -      followed = insert(:user, is_locked: true, local: false, ap_enabled: true) +      followed = insert(:user, is_locked: true, local: false)        assert {:ok, follower, followed, %{id: activity_id, data: %{"state" => "pending"}}} =                 CommonAPI.follow(follower, followed) diff --git a/test/pleroma/web/federator_test.exs b/test/pleroma/web/federator_test.exs index 1ffe6aae1..6826e6c2f 100644 --- a/test/pleroma/web/federator_test.exs +++ b/test/pleroma/web/federator_test.exs @@ -78,16 +78,14 @@ defmodule Pleroma.Web.FederatorTest do          local: false,          nickname: "nick1@domain.com",          ap_id: "https://domain.com/users/nick1", -        inbox: inbox1, -        ap_enabled: true +        inbox: inbox1        })        insert(:user, %{          local: false,          nickname: "nick2@domain2.com",          ap_id: "https://domain2.com/users/nick2", -        inbox: inbox2, -        ap_enabled: true +        inbox: inbox2        })        dt = NaiveDateTime.utc_now() diff --git a/test/support/factory.ex b/test/support/factory.ex index 09f02458c..d94544717 100644 --- a/test/support/factory.ex +++ b/test/support/factory.ex @@ -50,7 +50,6 @@ defmodule Pleroma.Factory do        last_refreshed_at: NaiveDateTime.utc_now(),        notification_settings: %Pleroma.User.NotificationSetting{},        multi_factor_authentication_settings: %Pleroma.MFA.Settings{}, -      ap_enabled: true,        keys: pem      }  | 
