air18n 0.1.45 → 0.1.46

data/lib/air18n/phrase_screenshot.rb

@@ -33,8 +33,11 @@ module Air18n
     end
 
     def create_row_in_screenshots
-      if !Screenshot.find_by_screenshot_url(self.screenshot_url)
-        screenshot = Screenshot.create(:screenshot_url => self.screenshot_url, :status => 0)
+      if !Screenshot.where(:screenshot_url => self.screenshot_url).exists?
+        Screenshot.create do |s|
+          s.screenshot_url = self.screenshot_url
+          s.status = 0
+        end
       end
     end
 
@@ -79,12 +82,18 @@ module Air18n
     def self.associate_phrase_with_screenshot(
       phrase_key, controller_name, action_name, screenshot_url)
       ps = PhraseScreenshot.find_or_create_by_controller_and_action_and_phrase_key(
-        controller_name, action_name, phrase_key, :screenshot_url => screenshot_url)
+        controller_name, action_name, phrase_key) do |new_phrase_screenshot|
+        new_phrase_screenshot.screenshot_url = screenshot_url
+      end
 
       # If the PhraseScreenshot already existed, update the screenshot URL.
       if ps.screenshot_url.blank? && ps.screenshot_url != screenshot_url
         ps.screenshot_url = screenshot_url
         ps.save
+
+        # Make sure that this new screenshot URL has a row in the screenshots
+        # table.
+        ps.create_row_in_screenshots
       end
 
       ps

data/lib/air18n/version.rb

@@ -1,3 +1,3 @@
 module Air18n
-  VERSION = "0.1.45"
+  VERSION = "0.1.46"
 end

data/spec/lib/air18n/backend_spec.rb

@@ -116,7 +116,8 @@ describe Air18n::Backend do
       @tAf = FactoryGirl.create(:phrase_translation, :phrase => @phraseA, :value => 'French value A', :locale => :fr)
 
       @backend.lookup(:es, @phraseA.key).should == 'Spanish value A'
-      @tAs.update_attributes(:value => 'Updated Spanish value A')
+      @tAs.value = 'Updated Spanish value A'
+      @tAs.save
       @backend.lookup(:es, @phraseA.key, [], :default => @phraseA.value).should == 'Spanish value A'
       @backend.lookup(:fr, @phraseA.key).should == 'French value A'
       @backend.lookup(:it, @phraseA.key, [], :default => @phraseA.value).should == 'value A'
@@ -129,8 +130,10 @@ describe Air18n::Backend do
 
       @backend.lookup(:es, @phraseC.key, [], :default => @phraseC.value).should == 'Spanish value C'
       @backend.lookup(:fr, @phraseC.key).should == 'French value C'
-      @tCs.update_attributes(:value => 'Newer Spanish value C')
-      @tCf.update_attributes(:value => 'Newer French value C')
+      @tCs.value = 'Newer Spanish value C'
+      @tCs.save
+      @tCf.value = 'Newer French value C'
+      @tCf.save
       I18n.mark_translations_as_changed(:es)
       I18n.mark_translations_as_changed(:fr)
       @backend.check_for_new_translations(:es)

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: air18n
 version: !ruby/object:Gem::Version
-  version: 0.1.45
+  version: 0.1.46
   prerelease: 
 platform: ruby
 authors:
@@ -13,7 +13,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-12-11 00:00:00.000000000 Z
+date: 2012-12-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: i18n