devise_invitable 2.0.12 → 2.0.14

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 73e4a8fc4cdc87b6f5397619a737fe13c6376b9534bf9ba810d9e2355bc2b588
-  data.tar.gz: 8dac212d256560b1cafff5e9193ffe72ddb988e52cb232a630668ac883d1b4ff
+  metadata.gz: e29dc7f305f5cb11941463a2115645c6de83ac0fde470243f8c4ee9c23535c7f
+  data.tar.gz: 1351e523f14e6dcb0243df3a6b0eca0156e0185e25fd488335c8acf4b260c7b7
 SHA512:
-  metadata.gz: 359b97a7be863db6e484282887e075723011b5d06fd38a02f7e5a761872a6a002838776bd46c082c3f94360f53e2248a8432a5a7cabdae6ba7d457db42ef7168
-  data.tar.gz: 848f10d801857bdb53d64348c3492a7d2e227611deab7d62d54af567eb03310787a4f2533b2366c236e0e54b701aa32642ee9f084aaa26ea44145ea8bd44ad2e
+  metadata.gz: 667c2b93fe46151947c3845fd19c4b370d5c9435331cc84ef52e2d375ba06e9748aa57539333b7951dd8b6a09ae6ebdc8ad8e9bb177b5e5d7741bec1691f9b6a
+  data.tar.gz: a7403ae4e341a1a1eaf8431addfc9336534ef1fbaacee60614880f0fa94d772bface4433929088c7025766c264c8820b5608914eabb1f8a76624b2ea6116a62c

data/config/locales/zh-HK.yml

@@ -3,7 +3,7 @@ zh-HK:
     failure:
       invited: "你有一個待處理的邀請, 請接受它以完成創建您的帳戶。"
     invitations:
-      send_instructions: "邀請電子郵件已發送到%{email)。"
+      send_instructions: "邀請電子郵件已發送到%{email}。"
       invitation_token_invalid: "提供嘅邀請令牌無效!"
       updated: "你嘅密碼設置成功。 你而家已經登錄。"
       updated_not_active: "你嘅密碼設置成功。"

data/config/locales/zh-TW.yml

@@ -3,7 +3,7 @@ zh-TW:
     failure:
       invited: "你有一封設立帳號的邀請，請接受邀請來創建你的帳號。"
     invitations:
-      send_instructions: "邀請已發送至%{email)"
+      send_instructions: "邀請已發送至%{email}"
       invitation_token_invalid: "所提供的邀請驗證無效！"
       updated: "你的密碼已設置成功，現在你已經登入了。"
       updated_not_active: "密碼設置成功"

data/lib/devise_invitable/rails.rb

@@ -5,14 +5,11 @@ module DeviseInvitable
       include DeviseInvitable::Controllers::Helpers
     end
 
-    # We use to_prepare instead of after_initialize here because Devise is a Rails engine; its
-    # mailer is reloaded like the rest of the user's app.  Got to make sure that our mailer methods
-    # are included each time Devise.mailer is (re)loaded.
-    config.to_prepare do
-      Devise.mailer.send :include, DeviseInvitable::Mailer
-      unless Devise.mailer.ancestors.include?(Devise::Mailers::Helpers)
-        Devise.mailer.send :include, Devise::Mailers::Helpers 
-      end
+    # Devise::Mailer includes Devise::Mailers::Helpers, so adding our mailer methods there reaches every
+    # (re)loaded Devise.mailer without resolving it at boot. Resolving it from an on_load(:action_mailer)
+    # hook re-enters the app's parent_mailer while it is still loading (#929).
+    initializer "devise_invitable.mailer" do
+      Devise::Mailers::Helpers.include DeviseInvitable::Mailer
     end
     # extend mapping with after_initialize because it's not reloaded
     config.after_initialize do

data/lib/devise_invitable/version.rb

@@ -1,3 +1,3 @@
 module DeviseInvitable
-  VERSION = '2.0.12'.freeze
+  VERSION = '2.0.14'.freeze
 end

data/test/mailers/parent_mailer_test.rb

@@ -0,0 +1,31 @@
+require 'test_helper'
+require 'open3'
+
+class ParentMailerTest < ActiveSupport::TestCase
+  APPLICATION = File.expand_path('../rails_app/config/application', __dir__)
+
+  # Each boot runs in a fresh process: the failure only shows up when nothing has loaded
+  # ActionMailer::Base yet, which is never the case inside this test process.
+  def boot_with_app_parent_mailer(first_reference)
+    script = <<~RUBY
+      require #{APPLICATION.inspect}
+      Devise.parent_mailer = 'ApplicationMailer'
+      RailsApp::Application.initialize!
+      #{first_reference}
+      print "invitation_instructions=\#{Devise.mailer.method_defined?(:invitation_instructions)}"
+    RUBY
+    Open3.capture3({ 'RAILS_ENV' => 'test' }, RbConfig.ruby, '-e', script)
+  end
+
+  test 'Devise.mailer resolves when the parent mailer is an app mailer' do
+    stdout, stderr, status = boot_with_app_parent_mailer('Devise.mailer')
+    assert status.success?, stderr
+    assert_match(/invitation_instructions=true\z/, stdout)
+  end
+
+  test 'the parent mailer can be the first mailer loaded, as with eager loading' do
+    stdout, stderr, status = boot_with_app_parent_mailer('ApplicationMailer')
+    assert status.success?, stderr
+    assert_match(/invitation_instructions=true\z/, stdout)
+  end
+end

data/test/rails_app/app/mailers/application_mailer.rb

@@ -0,0 +1,2 @@
+class ApplicationMailer < ActionMailer::Base
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: devise_invitable
 version: !ruby/object:Gem::Version
-  version: 2.0.12
+  version: 2.0.14
 platform: ruby
 authors:
 - Sergio Cambra
@@ -43,14 +43,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.0'
+        version: '4.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.0'
+        version: '4.0'
 description: It adds support for send invitations by email (it requires to be authenticated)
   and accept the invitation by setting a password.
 email:
@@ -120,6 +120,7 @@ files:
 - test/integration/invitation_test.rb
 - test/integration_tests_helper.rb
 - test/mailers/invitation_mail_test.rb
+- test/mailers/parent_mailer_test.rb
 - test/model_tests_helper.rb
 - test/models/invitable_test.rb
 - test/models_test.rb
@@ -132,6 +133,7 @@ files:
 - test/rails_app/app/controllers/home_controller.rb
 - test/rails_app/app/controllers/users_controller.rb
 - test/rails_app/app/helpers/application_helper.rb
+- test/rails_app/app/mailers/application_mailer.rb
 - test/rails_app/app/models/admin.rb
 - test/rails_app/app/models/octopussy.rb
 - test/rails_app/app/models/user.rb
@@ -199,6 +201,7 @@ test_files:
 - test/integration/invitation_test.rb
 - test/integration_tests_helper.rb
 - test/mailers/invitation_mail_test.rb
+- test/mailers/parent_mailer_test.rb
 - test/model_tests_helper.rb
 - test/models/invitable_test.rb
 - test/models_test.rb
@@ -211,6 +214,7 @@ test_files:
 - test/rails_app/app/controllers/home_controller.rb
 - test/rails_app/app/controllers/users_controller.rb
 - test/rails_app/app/helpers/application_helper.rb
+- test/rails_app/app/mailers/application_mailer.rb
 - test/rails_app/app/models/admin.rb
 - test/rails_app/app/models/octopussy.rb
 - test/rails_app/app/models/user.rb