acts_as_icontact 0.2.3 → 0.2.4

data/VERSION

@@ -1 +1 @@
-0.2.3
+0.2.4

data/acts_as_icontact.gemspec

@@ -2,11 +2,11 @@
 
 Gem::Specification.new do |s|
   s.name = %q{acts_as_icontact}
-  s.version = "0.2.3"
+  s.version = "0.2.4"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Stephen Eley"]
-  s.date = %q{2009-07-27}
+  s.date = %q{2009-07-29}
   s.default_executable = %q{icontact}
   s.description = %q{ActsAsIcontact connects Ruby applications with the iContact e-mail marketing service using the iContact API v2.0.  Building on the RestClient gem, it offers two significant feature sets:
 

data/lib/acts_as_icontact/rails/callbacks.rb

@@ -22,8 +22,36 @@ module ActsAsIcontact
             end
           end
           self.save
+          @icontact_in_progress = false  # Very primitive loop prevention
         end
-        
+      end
+
+      def icontact_after_update
+        unless @icontact_in_progress # Avoid callback loops
+          c = find_contact_by_identity
+          self.class.icontact_mappings.each do |rails, iContact|
+            if (value = self.send(rails))
+              ic = (iContact.to_s + '=').to_sym   # Blah. This feels like it should be easier.
+              c.send(ic, value)
+            end
+          end
+          c.save
+          # No need to update the record this time; iContact field changes don't have side effects
+        end
+      end
+      
+      private
+      def find_contact_by_identity
+        im = self.class.icontact_identity_map
+        if (im[1] == :contactId)
+          ActsAsIcontact::Contact.find(self.send(im[0]))
+        elsif (im[0] == :id)
+          ActsAsIcontact::Contact.find(im[1] => id)
+        else
+          ActsAsIcontact::Contact.find(:email => self.send(im[0]))
+        end
+      rescue ActsAsIcontact::QueryError
+        nil
       end
       
     end

data/lib/acts_as_icontact/rails/macro.rb

@@ -20,6 +20,7 @@ module ActsAsIcontact
           # If we haven't flaked out so far, we must be doing okay.  Make magic happen.
           include ActsAsIcontact::Rails::Callbacks
           after_create :icontact_after_create
+          after_update :icontact_after_update
         end
 
       end

data/lib/acts_as_icontact/resources/contact.rb

@@ -21,5 +21,12 @@ module ActsAsIcontact
       @lists ||= ActsAsIcontact::Subscription.lists(:contactId => id)
     end
     
+    # Creates a new subscription for the contact to the specified list
+    def subscribe(list)
+      l = ActsAsIcontact::List.find(list)
+      s = ActsAsIcontact::Subscription.new(:contactId => id, :listId => l.id)
+      s.save
+    end
+    
   end
 end

data/lib/acts_as_icontact/resources/list.rb

@@ -30,5 +30,12 @@ module ActsAsIcontact
       @subscribers ||= ActsAsIcontact::Subscription.contacts(:listId => id)
     end
     
+    # Creates a new subscription for the specified list by the contact
+    def subscribe(contact)
+      c = ActsAsIcontact::Contact.find(contact)
+      s = ActsAsIcontact::Subscription.new(:contactId => c.id, :listId => id)
+      s.save
+    end
+    
   end
 end

data/spec/rails_spec/callbacks_spec.rb

@@ -5,15 +5,32 @@ share_as :Callbacks do
       @person = @class.new(:firstName => "John", :surname => "Smith", :email => "john@example.org")
     end
     
-    it "will create a new contact after record creation" do
-#      ActsAsIcontact::Contact.any_instance.expects(:save).returns(true)
-      @person.save
+    context "for creation" do
+      it "creates a new contact" do
+        conn = mock('Class Connection')
+        conn.expects(:post).with(regexp_matches(/Smith/)).returns('{"contacts":{}}')
+        ActsAsIcontact::Contact.expects(:connection).returns(conn)
+        @person.save
+      end
+      
+      it "updates the Person with the results of the contact creation" do
+        @person.save
+        @person.icontact_id.should == 333444
+      end
     end
     
-    it "updates the Person with the results of the contact save" do
-      @person.save
-      @person.icontact_id.should == 333444
+    context "for update" do
+      before(:each) do
+        @person.save
+        @person.surname = "Nielsen Hayden"
+      end
+      
+      it "updates the contact with the new fields" do
+        conn = mock('Instance Connection')
+        conn.expects(:post).with(regexp_matches(/Nielsen Hayden/)).returns('{"contact":{}}')
+        ActsAsIcontact::Contact.any_instance.expects(:connection).returns(conn)
+        @person.save
+      end
     end
-    
   end
 end

data/spec/resources/contact_spec.rb

@@ -22,6 +22,13 @@ describe ActsAsIcontact::Contact do
       @john.lists.first.should == ActsAsIcontact::List.find(444444)
     end
     
+    it "can subscribe oneself to a list" do
+      conn = mock('Class Connection')
+      conn.expects(:post).with(regexp_matches(/444444/) && regexp_matches(/333333/)).returns('{"subscriptions":{}}')
+      ActsAsIcontact::Subscription.expects(:connection).returns(conn)
+      @john.subscribe(444444)
+    end
+    
     it "knows its history"
   end
     

data/spec/resources/list_spec.rb

@@ -40,5 +40,13 @@ describe ActsAsIcontact::List do
     it "knows its welcome message" do
       @list.welcomeMessage.should == ActsAsIcontact::Message.find(555555)
     end
+    
+    it "can be subscribed to by a subscriber" do
+      conn = mock('Class Connection')
+      conn.expects(:post).with(regexp_matches(/444444/) && regexp_matches(/333333/)).returns('{"subscriptions":{}}')
+      ActsAsIcontact::Subscription.expects(:connection).returns(conn)
+      @list.subscribe(333333)
+    end
+      
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: acts_as_icontact
 version: !ruby/object:Gem::Version 
-  version: 0.2.3
+  version: 0.2.4
 platform: ruby
 authors: 
 - Stephen Eley
@@ -9,7 +9,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2009-07-27 00:00:00 -04:00
+date: 2009-07-29 00:00:00 -04:00
 default_executable: icontact
 dependencies: []