devise_invitable 1.3.3 → 1.3.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: b829731496401b7e957dc0da6fb5c314104be6b1
-  data.tar.gz: 39abb9f0a58cc51c5a6f45e50189f4f2275d0743
+  metadata.gz: ec084ea3f59520fb7b2cfb978cd79d53ec5c1488
+  data.tar.gz: 257a48340805f6bfe02bc4fd1279a8694ab24c5c
 SHA512:
-  metadata.gz: ce39f924778f62589d7eb7ad004074fe2d32cf7c5ac3efd5d0b1a9e8d39e36921dc79954932787e461b9dad121d9528520df869b34433f1f772cd1864f3250a3
-  data.tar.gz: 5c4f8c5eddc26ea113001257d6b4c33c4283807404a4d7aeb82cab7ed57cf21754b464debd404c549c41d84f299edaed7dad6aaf7ddcf6346b57d698a577f1e0
+  metadata.gz: 51c1385e5d2235ef7d3004208e489ecd241e99ade5afd8d7f5a1289e48cc7e9af6ba599b058862b82c5eb9c64ee02dc63d1be1526d692ccd86c33099bda8c8a8
+  data.tar.gz: 86ff8fa5e4f3990ca8c5dee25acc0e663204b671337e479fc19cfc0c5d2145d79f85f5a68c1ac53790b8a4c967050e13b0a070cf8943c6c4d639684ad68898b7

data/README.rdoc

@@ -1,7 +1,7 @@
 = DeviseInvitable
 {<img src="https://travis-ci.org/scambra/devise_invitable.png"/>}[http://travis-ci.org/scambra/devise_invitable]
 
-It adds support to devise[http://github.com/plataformatec/devise] for send invitations by email (it requires to be authenticated) and accept the invitation setting the password.
+It adds support to devise[http://github.com/plataformatec/devise] for sending invitations by email (it requires to be authenticated) and accept the invitation setting the password.
 
 DeviseInvitable currently supports Rails 3 and 4, if you want to use it with Rails 2.3 you must install version {0.2.3}[http://rubygems.org/gems/devise_invitable/versions/0.2.3]
 
@@ -172,6 +172,30 @@ be sure that you generate the views and put them into the controller that you ge
 
   rails generate devise_invitable:views users/invitations
 
+To change behaviour of inviting or accepting users, you can simply override two methods:
+
+  class Users::InvitationsController < Devise::InvitationsController
+    private
+
+    # this is called when creating invitation
+    # should return an instance of resource class
+    def invite_resource
+      ## skip sending emails on invite
+      resource_class.invite!(invite_params, current_inviter) do |u|
+        u.skip_invitation = true
+      end
+    end
+
+    # this is called when accepting invitation
+    # should return an instance of resource class
+    def accept_resource
+      resource = resource_class.accept_invitation!(update_resource_params)
+      ## Report accepting invitation to analytics
+      Analytics.report('invite.accept', resource.id)
+      resource
+    end
+  end
+
 == Strong Parameters
 
 When you customize your own views, you may end up adding new attributes to forms. Rails 4 moved the parameter sanitization from the model to the controller, causing DeviseInvitable to handle this concern at the controller as well. Read about it in {devise README}[http://github.com/plataformatec/devise#strong-parameters]

data/app/controllers/devise/invitations_controller.rb

@@ -17,6 +17,7 @@ class Devise::InvitationsController < DeviseController
     self.resource = invite_resource
 
     if resource.errors.empty?
+      yield resource if block_given?
       set_flash_message :notice, :send_instructions, :email => self.resource.email if self.resource.invitation_sent_at
       respond_with resource, :location => after_invite_path_for(resource)
     else
@@ -35,6 +36,7 @@ class Devise::InvitationsController < DeviseController
     self.resource = accept_resource
 
     if resource.errors.empty?
+      yield resource if block_given?
       flash_message = resource.active_for_authentication? ? :updated : :updated_not_active                                                                                        
       set_flash_message :notice, flash_message
       sign_in(resource_name, resource)

data/lib/devise_invitable.rb

@@ -61,6 +61,11 @@ module Devise
   # the #invited_by association is declared to be polymorphic. (default: nil)
   mattr_accessor :invited_by_class_name
   @@invited_by_class_name = nil
+
+  # Public: The column name used for counter_cache column. If this is nil,
+  # the #invited_by association is declared without counter_cache. (default: nil)
+  mattr_accessor :invited_by_counter_cache
+  @@invited_by_counter_cache = nil
 end
 
 Devise.add_module :invitable, :controller => :invitations, :model => 'devise_invitable/model', :route => {:invitation => [nil, :new, :accept]}

data/lib/devise_invitable/model.rb

@@ -35,7 +35,12 @@ module Devise
           {:polymorphic => true}
         end
         if defined?(ActiveRecord) && self < ActiveRecord::Base
-          belongs_to_options.merge! :counter_cache => :invitations_count
+          counter_cache = Devise.invited_by_counter_cache
+          if !counter_cache && Devise.invited_by_class_name
+            klass = Devise.invited_by_class_name.constantize
+            counter_cache = klass.table_exists? && klass.columns_hash['invitations_count'].try('name')
+          end
+          belongs_to_options.merge! :counter_cache => counter_cache if counter_cache
         end
         belongs_to :invited_by, belongs_to_options
 
@@ -61,9 +66,8 @@ module Devise
       def self.required_fields(klass)
         fields = [:invitation_token, :invitation_created_at, :invitation_sent_at, :invitation_accepted_at,
          :invitation_limit, :invited_by_id, :invited_by_type]
-        if Devise.invited_by_class_name
-          fields -= [:invited_by_type]
-        end
+        fields << :invitations_count if defined?(ActiveRecord) && self < ActiveRecord::Base
+        fields -= [:invited_by_type] if Devise.invited_by_class_name
         fields
       end
 
@@ -282,11 +286,6 @@ module Devise
           invitable unless only_valid && invitable.errors.present?
         end
 
-        # Generate a token checking if one does not already exist in the database.
-        def invitation_token
-          generate_token(:invitation_token)
-        end
-
         # Callback convenience methods
         def before_invitation_accepted(*args, &blk)
           set_callback(:invitation_accepted, :before, *args, &blk)

data/lib/devise_invitable/version.rb

@@ -1,3 +1,3 @@
 module DeviseInvitable
-  VERSION = '1.3.3'
+  VERSION = '1.3.4'
 end

data/lib/generators/active_record/templates/migration.rb

@@ -23,7 +23,7 @@ class DeviseInvitableAddTo<%= table_name.camelize %> < ActiveRecord::Migration
   def down
     change_table :<%= table_name %> do |t|
       t.remove_references :invited_by, :polymorphic => true
-      t.remove :invitation_limit, :invitation_sent_at, :invitation_accepted_at, :invitation_token, :invitation_created_at
+      t.remove :invitations_count, :invitation_limit, :invitation_sent_at, :invitation_accepted_at, :invitation_token, :invitation_created_at
     end
   end
 end

data/lib/generators/devise_invitable/install_generator.rb

@@ -43,6 +43,20 @@ module DeviseInvitable
   # Default: false
   # config.validate_on_invite = true
 
+  # Resend invitation if user with invited status is invited again
+  # Default: true
+  # config.resend_invitation = false
+
+  # The class name of the inviting model. If this is nil,
+  # the #invited_by association is declared to be polymorphic.
+  # Default: nil
+  # config.invited_by_class_name = 'User'
+
+  # The column name used for counter_cache column. If this is nil,
+  # the #invited_by association is declared without counter_cache.
+  # Default: nil
+  # config.invited_by_counter_cache = :invitations_count
+
 CONTENT
             end
           end

data/test/models/invitable_test.rb

@@ -548,13 +548,13 @@ class InvitableTest < ActiveSupport::TestCase
 
   test 'should pass validation before accept if field is required in post-invited instance' do
     user = User.invite!(:email => "valid@email.com")
-    user.testing_accepting_or_not_invited = true
+    user.testing_accepted_or_not_invited = true
     assert_equal true, user.valid?
   end
 
   test 'should fail validation after accept if field is required in post-invited instance' do
     user = User.invite!(:email => "valid@email.com")
-    user.testing_accepting_or_not_invited = true
+    user.testing_accepted_or_not_invited = true
     user.accept_invitation!
     assert_equal false, user.valid?
   end
@@ -562,7 +562,7 @@ class InvitableTest < ActiveSupport::TestCase
   test 'should pass validation after accept if field is required in post-invited instance' do
     user = User.invite!(:email => "valid@email.com")
     user.username = 'test'
-    user.testing_accepting_or_not_invited = true
+    user.testing_accepted_or_not_invited = true
     user.bio = "Test"
     user.accept_invitation!
     assert_equal true, user.valid?

data/test/rails_app/app/models/user.rb

@@ -39,12 +39,12 @@ class User < PARENT_MODEL_CLASS
   attr_accessor :callback_works, :bio, :token
   validates :username, :length => { :maximum => 20 }
   
-  attr_accessor :testing_accepting_or_not_invited
-  validates :username, :presence => true, :if => :testing_accepting_or_not_invited_validator?
+  attr_accessor :testing_accepted_or_not_invited
+  validates :username, :presence => true, :if => :testing_accepted_or_not_invited_validator?
   validates :bio, :presence => true, :if => :invitation_accepted?
 
-  def testing_accepting_or_not_invited_validator?
-    testing_accepting_or_not_invited && accepting_or_not_invited?
+  def testing_accepted_or_not_invited_validator?
+    testing_accepted_or_not_invited && accepted_or_not_invited?
   end
 
   after_invitation_accepted do |object|

data/test/rails_app/config/initializers/devise.rb

@@ -90,6 +90,45 @@ Devise.setup do |config|
   # Setup a pepper to generate the encrypted password.
   # config.pepper = "e31589192aeea8807cb7d8686b0f8484d6cbfaaa65443d45144519ed1d4ffbc6ccb73b21a69ece276d94f2cac95d83990d824f36f301d6f585ededd1bf90d67d"
 
+  # ==> Configuration for :invitable
+  # The period the generated invitation token is valid, after
+  # this period, the invited resource won't be able to accept the invitation.
+  # When invite_for is 0 (the default), the invitation won't expire.
+  # config.invite_for = 2.weeks
+
+  # Number of invitations users can send.
+  # - If invitation_limit is nil, there is no limit for invitations, users can
+  # send unlimited invitations, invitation_limit column is not used.
+  # - If invitation_limit is 0, users can't send invitations by default.
+  # - If invitation_limit n > 0, users can send n invitations.
+  # You can change invitation_limit column for some users so they can send more
+  # or less invitations, even with global invitation_limit = 0
+  # Default: nil
+  # config.invitation_limit = 5
+
+  # The key to be used to check existing users when sending an invitation
+  # and the regexp used to test it when validate_on_invite is not set.
+  # config.invite_key = {:email => /\\A[^@]+@[^@]+\\z/}
+  # config.invite_key = {:email => /\\A[^@]+@[^@]+\\z/, :username => nil}
+
+  # Flag that force a record to be valid before being actually invited
+  # Default: false
+  # config.validate_on_invite = true
+
+  # Resend invitation if user with invited status is invited again
+  # Default: true
+  # config.resend_invitation = false
+
+  # The class name of the inviting model. If this is nil,
+  # the #invited_by association is declared to be polymorphic.
+  # Default: nil
+  # config.invited_by_class_name = 'User'
+
+  # The column name used for counter_cache column. If this is nil,
+  # the #invited_by association is declared without counter_cache.
+  # Default: nil
+  config.invited_by_counter_cache = :invitations_count
+
   # ==> Configuration for :confirmable
   # A period that the user is allowed to access the website even without
   # confirming his account. For instance, if set to 2.days, the user will be

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: devise_invitable
 version: !ruby/object:Gem::Version
-  version: 1.3.3
+  version: 1.3.4
 platform: ruby
 authors:
 - Sergio Cambra
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-01-30 00:00:00.000000000 Z
+date: 2014-02-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler