Only update other avatars in one place

Prevent future mistakes

app/services/user_manager/import_remote_avatar.rb

@@ -7,8 +7,7 @@ module UserManager
 
     def call
       if import_remote_avatar
-        LdapManager::UpdateAvatar.call(user: @user)
-        XmppSetAvatarJob.perform_later(user: @user) if Setting.ejabberd_enabled?
+        UserManager::UpdateAvatar.call(user: @user)
       end
     end
 

app/services/user_manager/update_avatar.rb

@@ -0,0 +1,15 @@
+module UserManager
+  class UpdateAvatar < UserManagerService
+    def initialize(user:)
+      @user = user
+    end
+
+    def call
+      LdapManager::UpdateAvatar.call(user: @user)
+
+      if Setting.ejabberd_enabled?
+        XmppSetAvatarJob.perform_later(user: @user)
+      end
+    end
+  end
+end