Do not consider a plural node if it only has "other" with no count

Fix glebm#310
Gargron · Oct 26, 2018 · ab6e108 · ab6e108
1 parent c17c9d2
commit ab6e108
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/lib/i18n/tasks/plural_keys.rb b/lib/i18n/tasks/plural_keys.rb
@@ -46,9 +46,14 @@ def plural_nodes(tree)
   end
 
   def plural_forms?(s)
+    return false if non_plural_other?(s)
     s.present? && s.all? { |node| node.leaf? && plural_suffix?(node.key) }
   end
 
+  def non_plural_other?(s)
+    s.size == 1 && s.first.leaf? && s.first.key == 'other' && !s.first.value.include?('%{count}')
+  end
+
   def plural_suffix?(key)
     PLURAL_KEY_SUFFIXES.include?(key)
   end