adgroup.rb in sklik-api-0.0.15

- old
+ new

@@ -8,11 +8,11 @@
 =begin
 Example of input hash
 {
   :adgroup_id => 1234, #(OPTIONAL) -> when setted it will on save do update of existing adgroup
   :name => "my adgroup name",
-  :ads => [ 
+  :ads => [
     {
       :headline => "Super headline",
       :description1 => "Trying to do ",
       :description2 => "best description ever",
       :display_url => "my_test_url.cz",
@@ -27,11 +27,11 @@
   ]
 }
 
 
 =end
-  
+
     def initialize campaign, args
       @adgroup_data = nil
       #set adgroup owner campaign
       @campaign = campaign
 
@@ -47,37 +47,37 @@
       if args[:keywords] && args[:keywords].size > 0
         args[:keywords].each do |keyword|
           @keywords << SklikApi::Keyword.new(self, :keyword => keyword)
         end
       end
-      
+
       super args
     end
-    
+
     def self.find campaign, args = {}
       out = []
       super(NAME, campaign.args[:campaign_id]).each do |adgroup|
         if args[:adgroup_id].nil? || (args[:adgroup_id] && args[:adgroup_id].to_i == adgroup[:id].to_i)
-          out << SklikApi::Adgroup.new( campaign, 
+          out << SklikApi::Adgroup.new( campaign,
             :adgroup_id => adgroup[:id],
-            :cpc => adgroup[:cpc].to_f/100.0, 
-            :name => adgroup[:name], 
+            :cpc => adgroup[:cpc].to_f/100.0,
+            :name => adgroup[:name],
             :status => fix_status(adgroup)
           )
         end
       end
       out
     end
-    
+
     def self.fix_status adgroup
       if adgroup[:removed] == true
         return :stopped
       else
         return :running
       end
     end
-    
+
     def keywords_stats from, to
       output = []
       keywords = Keyword.find(self)
       keywords.in_groups_of(100, false).each do |keywords_group|
         out = connection.call("keywords.stats", keywords_group.collect{|k| k.args[:keyword_id]}, from, to ) { |param|
@@ -103,72 +103,157 @@
         @adgroup_data[:ads] = Adtext.find(self).collect{|a| a.to_hash}
         @adgroup_data[:keywords] = Keyword.find(self).collect{|k| k.to_hash}
         @adgroup_data
       end
     end
-    
-    
+
+
     def create_args
       raise ArgumentError, "Adgroup need's to know campaign_id" unless @campaign.args[:campaign_id]
       raise ArgumentError, "Adgroup need's to know campaigns CPC" unless @campaign.args[:cpc]
 
       out = []
       #add campaign id to know where to create adgroup
-      out << @campaign.args[:campaign_id] 
-      
+      out << @campaign.args[:campaign_id]
+
       #add adgroup struct
       args = {}
       args[:name] = @args[:name]
       args[:cpc] = (@campaign.args[:cpc] * 100).to_i if @campaign.args[:cpc]
       out << args
-      
+
       #return output
       out
     end
-    
-    def save 
+
+    def adtexts
+      Adtext.find(self)
+    end
+
+    def keywords
+      Keyword.find(self)
+    end
+
+    def update args = {}
+      if args.is_a?(SklikApi::Adgroup)
+        #get data from another adgroup
+        @adtexts = args.instance_variable_get("@adtexts")
+        @keywords = args.instance_variable_get("@keywords")
+
+        #set parent to this adgroup
+        @adtexts.each{|a| a.instance_variable_set("@adgroup", self)}
+        @keywords.each{|a| a.instance_variable_set("@adgroup", self)}
+      else
+        #initialize new adtexts
+        @adtexts = []
+        if args[:ads] && args[:ads].size > 0
+          args[:ads].each do |adtext|
+            @adtexts << SklikApi::Adtext.new(self, adtext)
+          end
+        end
+
+        #initialize new keywords
+        @keywords = []
+        if args[:keywords] && args[:keywords].size > 0
+          args[:keywords].each do |keyword|
+            @keywords << SklikApi::Keyword.new(self, :keyword => keyword)
+          end
+        end
+      end
+
+      save
+    end
+
+    def save
       if @args[:adgroup_id]  #do update
-        
+
+        ############
+        ## KEYWORDS
+        ############
+        #create new keyowrds and delete old
+        @saved_keywords = keywords.inject({}){|o,k| o[k.args[:keyword]] = k ; o}
+        @new_keywords = @keywords.inject({}){|o,k| o[k.args[:keyword]] = k ; o}
+
+        #keywords to be deleted
+        (@saved_keywords.keys - @new_keywords.keys).each do |k|
+          #don't try to remove already removed keyword
+          @saved_keywords[k].remove unless @saved_keywords[k].args[:status] == :stopped
+        end
+
+        #keywords to be created
+        (@new_keywords.keys - @saved_keywords.keys).each do |k|
+          @new_keywords[k].save
+        end
+
+        #check status to be running
+        (@new_keywords.keys & @saved_keywords.keys).each do |k|
+          @saved_keywords[k].restore if @saved_keywords[k].args[:status] == :stopped
+        end
+
+        ############
+        ## ADTEXTS
+        ############
+
+        #create new adtexts and delete old
+        @saved_adtexts = adtexts.inject({}){|o,a| o[a.uniq_identifier] = a ; o}
+        @new_adtexts = @adtexts.inject({}){|o,a| o[a.uniq_identifier] = a ; o}
+
+        #adtexts to be deleted
+        (@saved_adtexts.keys - @new_adtexts.keys).each do |k|
+          #don't try to remove already removed adtext
+          @saved_adtexts[k].remove  unless @saved_adtexts[k].args[:status] == :stopped
+        end
+
+        #adtexts to be created
+        (@new_adtexts.keys - @saved_adtexts.keys).each do |k|
+          @new_adtexts[k].save
+        end
+
+        #check status to be running
+        (@new_adtexts.keys & @saved_adtexts.keys).each do |k|
+          @saved_adtexts[k].restore if @saved_adtexts[k].args[:status] == :stopped
+        end
+
+
       else                    #do save
         #create adgroup
         create
-        
+
         #create adtexts
-        @adtexts.each do |adtext| 
-          begin 
+        @adtexts.each do |adtext|
+          begin
             adtext.save
           rescue Exception => e
             #take care about error message -> do it nicer
             if /There is error from sklik ad.create: Invalid parameters/ =~ e.message
               @campaign.errors << "Problem with creating #{adtext.args} in adgroup #{@args[:name]}"
             else
               @campaign.errors << e.message
             end
           end
         end
-          
-        
+
+
         #create keywords
         keywords_error = []
-        @keywords.each do |keyword| 
-          begin 
+        @keywords.each do |keyword|
+          begin
             keyword.save
           rescue Exception => e
             #take care about error message -> do it nicer
-            if /Sklik returned: keyword.create: Invalid data in request/ =~ e.message 
+            if /Sklik returned: keyword.create: Invalid data in request/ =~ e.message
               keywords_error << e.message.split("{:name=>\"").last.split("\", :matchType").first
             else
               @campaign.errors << e.message
             end
           end
         end
         if keywords_error.size > 0
           @campaign.errors << "Problem with creating keywords: #{keywords_error.join(", ")} in adgroup #{@args[:name]}"
         end
-        
+
       end
     end
 
   end
 end
-      
-      
\ No newline at end of file
+