diff --git a/app/models/user.rb b/app/models/user.rb index c1851a4..835e174 100644 --- a/app/models/user.rb +++ b/app/models/user.rb @@ -57,7 +57,7 @@ class User < ApplicationRecord end def devise_after_confirmation - enable_service %w[ discourse ejabberd gitea mediawiki ] + enable_service %w[ discourse gitea mediawiki xmpp ] #TODO enable in development when we have easy setup of ejabberd etc. return if Rails.env.development? @@ -137,12 +137,12 @@ class User < ApplicationRecord def pref_enabled?(key) value = preferences.dig(*key.split(":")) - [true, "true", 1, "enabled"].include?(value) + [true, "true", "enabled", 1].include?(value) end def exchange_xmpp_contact_with_inviter - return unless inviter.services_enabled.include?("ejabberd") && - services_enabled.include?("ejabberd") + return unless inviter.services_enabled.include?("xmpp") && + services_enabled.include?("xmpp") XmppExchangeContactsJob.perform_later(inviter, self.cn, self.ou) end diff --git a/app/views/admin/users/show.html.erb b/app/views/admin/users/show.html.erb index 5d89827..42c7963 100644 --- a/app/views/admin/users/show.html.erb +++ b/app/views/admin/users/show.html.erb @@ -135,7 +135,7 @@ XMPP (ejabberd) <%= render FormElements::ToggleComponent.new( - enabled: @services_enabled.include?("ejabberd"), + enabled: @services_enabled.include?("xmpp"), input_enabled: false ) %> diff --git a/spec/models/user_spec.rb b/spec/models/user_spec.rb index 37db8ae..d72c2e5 100644 --- a/spec/models/user_spec.rb +++ b/spec/models/user_spec.rb @@ -109,7 +109,7 @@ RSpec.describe User, type: :model do before do Invitation.create! user: user, invited_user_id: guest.id, used_at: DateTime.now - allow_any_instance_of(User).to receive(:services_enabled).and_return(%w[ ejabberd ]) + allow_any_instance_of(User).to receive(:services_enabled).and_return(%w[ xmpp ]) end it "enqueues a job to exchange XMPP contacts between inviter and invitee" do @@ -131,11 +131,11 @@ RSpec.describe User, type: :model do let(:user) { create :user, cn: "willherschel", ou: "kosmos.org" } it "enables default services" do - expect(user).to receive(:enable_service).with(%w[ discourse ejabberd gitea mediawiki ]) + expect(user).to receive(:enable_service).with(%w[ discourse gitea mediawiki xmpp ]) user.send(:devise_after_confirmation) end - context "for invited user with ejabberd enabled" do + context "for invited user with xmpp enabled" do let(:guest) { create :user, id: 2, cn: "isaacnewton", ou: "kosmos.org", email: "newt@example.com" } before do