air18n 0.1.42 → 0.1.43

data/lib/air18n.rb

@@ -10,7 +10,7 @@ require 'air18n/default_text_change_observer'
 require 'air18n/reflection'
 
 module Air18n
-  AIR18N_RESERVED_KEYS = [:routes_context, :suppress_ct, :default_is_low_priority, :disable_xss_check, :disable_bookkeeping]
+  AIR18N_RESERVED_KEYS = [:routes_context, :suppress_ct, :default_is_low_priority, :disable_xss_check, :disable_bookkeeping, :context]
 
   RTL_LOCALES = [ :he, :ar ]
 

data/lib/air18n/backend.rb

@@ -146,11 +146,12 @@ module Air18n
       @prim_and_proper.guess(text, orig_locale, other_locale)
     end
 
-    # We will only take 10 screenshots per Rails instance per key.
+    # We will only take 2 screenshots per Rails instance per key.
     # This is to avoid having to moderate 200 screenshots for every new site-wide
     # phrase.
-    # Because this is a per-Rails-thread cap, we call it approximate.
-    APPROX_MAX_SCREENSHOTS_PER_KEY = 10
+    # Because this is a per-Rails-thread cap, it is a lower bound for how many
+    # screenshot we will end up with per widely-used phrase.
+    APPROX_MAX_SCREENSHOTS_PER_KEY = 3
 
     # This method does the meat of Air18n functionality.
     # 1) Finds the translations for specified key and locale, falling back to

data/lib/air18n/phrase_screenshot.rb

@@ -13,7 +13,7 @@ module Air18n
     def self.all_phrase_urls
       Hash.new { |h, k| h[k] = [] }.tap do |all_phrase_urls|
         # Uses raw SQL for speed.
-        connection.select_all("SELECT `phrase_key`, `action`, `controller` FROM phrase_screenshots").each do |record|
+        connection.select_all("SELECT `phrase_key`, `action`, `controller` FROM phrase_screenshots where screenshot_url != ''").each do |record|
           all_phrase_urls[record['phrase_key']] << make_routes_context(record['controller'], record['action'])
         end
       end
@@ -72,5 +72,22 @@ module Air18n
       controller, action = page_id.split("#")
       Struct.new(:controller, :action).new(controller, action)
     end
+
+    # Associates a phrase with a controller, action pair and a screenshot.
+    # Set screenshot_url to the empty string if you don't have a screenshot.
+    # Returns the PhraseScreenshot.
+    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)
+
+      # 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
+      end
+
+      ps
+    end
   end
 end

data/lib/air18n/version.rb

@@ -1,3 +1,3 @@
 module Air18n
-  VERSION = "0.1.42"
+  VERSION = "0.1.43"
 end

metadata

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