cck_forms 3.1.1 → 3.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 48d8cdb6dd4a8d35ee9b6af68d1739840506667f
-  data.tar.gz: f20ca16724e0f505b669ad67fb70334be39d8e79
+  metadata.gz: 8dceaa1bbafb95e971aa01a98f84e506330928e7
+  data.tar.gz: 99ae8648bafac7968f576b8f3e2c822c987691e6
 SHA512:
-  metadata.gz: da19c4c957d0e67c812be74d1d6281ddb0dcd134221768f2338f5db4ea919afc23aeb59aea6bfbcdd9d604ecb71c04e8f9f541fa85849663fdf45386dab5ca9b
-  data.tar.gz: 1fc9d3f7a8344ec24e3296cbe8db58acc0f2a9843edd22390a3ed4142193b4384eed2df2467dbfc1e0661333b4794ba82302fd12db29fffb6b351e71867f6f70
+  metadata.gz: 035dc59ed36c8546d3c78ef4a0c9a1d50300943ba7b0c6c71e8afff7d10c1d99430b6580fdc26ebcee6224d241d293871d3b6d0c6bf8bc6c7e5b50a7c44c0de8
+  data.tar.gz: 377da4b610b5d1b961a835786cd6eb4b95aef62326ca5129527fe43e2ee7610415d5dba720d79e36103b53c9b3bc662eeeceb9ee823467063a05276df39bd603

data/README.md

@@ -210,6 +210,8 @@ Available field types
 
 ***WorkHours***: complex input to construct work schedule on a weekly basis.
 
+***WatermarklessAlbum***, ***WatermarklessImage***: same as Album & Image, but do not place watermarks. For banners or
+important sliders, for example.
 
 Configuration
 -------------

data/lib/cck_forms/date_time.rb

@@ -49,7 +49,7 @@ module CckForms::DateTime
       [{default: date_in_time_zone, include_blank: false, with_css_classes: true}, {class: 'form-control'}]
     end
 
-    def demongoize_value(value, parameter_type_class=nil)
+    def demongoize_value(value, _parameter_type_class=nil)
       date_object_from_what_stored_in_database value
     end
   end

data/lib/cck_forms/parameter_type_class/album.rb

@@ -4,10 +4,6 @@ class CckForms::ParameterTypeClass::Album
   include CckForms::ParameterTypeClass::Base
   include CckForms::NeofilesDenormalize
 
-  def self.name
-    'Альбом'
-  end
-
   # Converts input array of Neofiles::Image or IDs to array of hashes (denormalized image data) or IDs
   def mongoize
     the_value = value.is_a?(Hash) ? value['value'] : value

data/lib/cck_forms/parameter_type_class/base.rb

@@ -119,7 +119,7 @@ module CckForms::ParameterTypeClass::Base
 
     # A type name, e.g. "A text string"
     def name
-      nil
+      I18n.t("cck_forms.parameter_type_name.#{code}")
     end
   end
 

data/lib/cck_forms/parameter_type_class/boolean.rb

@@ -3,10 +3,6 @@
 class CckForms::ParameterTypeClass::Boolean
   include CckForms::ParameterTypeClass::Base
 
-  def self.name
-    'Галочка (да/нет)'
-  end
-
   # Is it true?
   def value?
     value.present? && value != '0'
@@ -18,8 +14,8 @@ class CckForms::ParameterTypeClass::Boolean
   end
 
   # 'yes/no' string
-  def to_s(options = nil)
-    value? ? 'да' : 'нет'
+  def to_s(_options = nil)
+    value? ? I18n.t('cck_forms.boolean.yes') : I18n.t('cck_forms.boolean.no')
   end
 
   # Checkbox HTML

data/lib/cck_forms/parameter_type_class/checkboxes.rb

@@ -3,10 +3,6 @@
 class CckForms::ParameterTypeClass::Checkboxes
   include CckForms::ParameterTypeClass::Base
 
-  def self.name
-    'Галочки'
-  end
-
   # {kazakh: '1', russian: '0', english: '1'} -> ['kazakh', 'english']
   def mongoize
     resulting_set = []
@@ -86,9 +82,9 @@ class CckForms::ParameterTypeClass::Checkboxes
       param_title = cck_parameter ? " «#{cck_parameter.title}»" : ''
 
       if unchecked_num == 0
-        return "Все значения#{param_title}"
+        return I18n.t('cck_forms.checkboxes.all_values', param_title: param_title)
       elsif unchecked_num < checked_keys.count./(2).round
-        return "Все значения#{param_title}, кроме " + valid_values.values_at(*unchecked_keys).map { |x| sprintf(template, x) }.join(glue)
+        return I18n.t('cck_forms.checkboxes.all_values_except', param_title: param_title) + ' ' + valid_values.values_at(*unchecked_keys).map { |x| sprintf(template, x) }.join(glue)
       end
     end
 
@@ -173,7 +169,7 @@ class CckForms::ParameterTypeClass::Checkboxes
 
           v = options[:map][v] || v
 
-          # required не встраиваю, иначе пока я все чекбоксы не отмечу, не могу отправить форму
+          # skip `required` since form will not be submitted unless a user checks all the checkboxes
           data = options[:data] ? extract_data_for_key(k, options[:data]) : nil
           sprintf(options[:block], ff.check_box(k.to_sym, {checked: checked}, '1', options[:for] == :search ? nil : '0'), ff.label(k.to_sym, v, data: data)).html_safe
         end

data/lib/cck_forms/parameter_type_class/date.rb

@@ -4,10 +4,6 @@ class CckForms::ParameterTypeClass::Date
   include CckForms::ParameterTypeClass::Base
   include CckForms::DateTime
 
-  def self.name
-    'Дата'
-  end
-
   # Date SELECT
   def build_form(form_builder, options)
     set_value_in_hash options
@@ -24,7 +20,7 @@ class CckForms::ParameterTypeClass::Date
   end
 
   # "12.12.2012"
-  def to_s(options = nil)
+  def to_s(_options = nil)
     if value.is_a? Time
       the_value = {
           '(1i)' => value.year,

data/lib/cck_forms/parameter_type_class/date_range.rb

@@ -3,10 +3,6 @@
 class CckForms::ParameterTypeClass::DateRange
   include CckForms::ParameterTypeClass::Base
 
-  def self.name
-    'Диапазон между двумя датами'
-  end
-
   # Converts input hash of type {from: ..., till: ...} to MongoDB format
   def mongoize
     value_from_form = value
@@ -46,9 +42,9 @@ class CckForms::ParameterTypeClass::DateRange
     from, till = types[:from], types[:till]
 
     if from.blank?
-      "до #{till}"
+      [I18n.t('cck_forms.date_range.till'), till].join(' ')
     elsif till.blank?
-      "от #{from}"
+      [I18n.t('cck_forms.date_range.from'), from].join(' ')
     elsif from == till
       from.to_s
     else

data/lib/cck_forms/parameter_type_class/date_time.rb

@@ -4,10 +4,6 @@ class CckForms::ParameterTypeClass::DateTime
   include CckForms::ParameterTypeClass::Base
   include CckForms::DateTime
 
-  def self.name
-    'Дата и время'
-  end
-
   # Date and time SELECTs
   def build_form(form_builder, options)
     set_value_in_hash options

data/lib/cck_forms/parameter_type_class/enum.rb

@@ -3,16 +3,12 @@
 class CckForms::ParameterTypeClass::Enum
   include CckForms::ParameterTypeClass::Base
 
-  def self.name
-    'Значение из списка'
-  end
-
   def mongoize
     value.presence
   end
 
   # Current string value from valid_values
-  def to_s(options = nil)
+  def to_s(_options = nil)
     return '' if value.blank?
     valid_values[value].to_s
   end

data/lib/cck_forms/parameter_type_class/file.rb

@@ -4,10 +4,6 @@ class CckForms::ParameterTypeClass::File
   include CckForms::ParameterTypeClass::Base
   include CckForms::NeofilesDenormalize
 
-  def self.name
-    'Файл'
-  end
-
   def self.file_type
     ::Neofiles::File
   end
@@ -52,8 +48,9 @@ class CckForms::ParameterTypeClass::File
   def self.create_load_form(helper:, file:, input_name:, append_create: false, clean_remove: false, widget_id: nil, disabled: false, multiple: false, with_desc: false)
     cont_id   = 'container_' + (widget_id.presence || form_name_to_id(input_name))
 
-    # создаем временное поле, чтобы пока аяксовый ответ не вернется, мы могли все же отправить родительскую форму
-    # и не потерять при этом данные из поля
+    # create temporary hidden field to keep the value in form context until AJAX request is finished
+    # (otherwise submitting the form before that moment sends it without the file value which can lead to confusion
+    # and consequent data loss in a controller)
     file_id = file.is_a?(file_type) ? file.id.to_s : file.to_s
     temp_field, remove_temp_field = '', ''
     if file_id.present?
@@ -89,12 +86,12 @@ class CckForms::ParameterTypeClass::File
   end
 
   # Returns empty string
-  def to_s
+  def to_s(_options = nil)
     ''
   end
 
   # Returns a file name
-  def to_diff_value(options = {})
+  def to_diff_value(_options = {})
     value.try! :name
   end
 end

data/lib/cck_forms/parameter_type_class/float.rb

@@ -3,15 +3,11 @@
 class CckForms::ParameterTypeClass::Float
   include CckForms::ParameterTypeClass::Base
 
-  def self.name
-    'Дробное число'
-  end
-
   def mongoize
     value.to_f
   end
 
-  def to_s(options = nil)
+  def to_s(_options = nil)
     value.to_f != 0.0 ? value.to_f : ''
   end
 

data/lib/cck_forms/parameter_type_class/image.rb

@@ -1,11 +1,6 @@
 # Represents a single image. A subclass of File.
 #
 class CckForms::ParameterTypeClass::Image < CckForms::ParameterTypeClass::File
-
-  def self.name
-    'Картинка'
-  end
-
   def self.file_type
     ::Neofiles::Image
   end

data/lib/cck_forms/parameter_type_class/integer.rb

@@ -3,15 +3,11 @@
 class CckForms::ParameterTypeClass::Integer
   include CckForms::ParameterTypeClass::Base
 
-  def self.name
-    'Целое число'
-  end
-
   def mongoize
     value.to_i
   end
 
-  def to_s(options = nil)
+  def to_s(_options = nil)
     value.to_i != 0 ? value.to_i.to_s : ''
   end
 

data/lib/cck_forms/parameter_type_class/integer_range.rb

@@ -7,10 +7,6 @@
 class CckForms::ParameterTypeClass::IntegerRange # Rover :)
   include CckForms::ParameterTypeClass::Base
 
-  def self.name
-    'Диапазон между двумя целыми числами'
-  end
-
   # {from: 500, to: 1000, ranges: {"300-600" => true, "601-900" => true, "901-1500" => false}}
   def mongoize
     value_from_form = value
@@ -74,9 +70,9 @@ class CckForms::ParameterTypeClass::IntegerRange # Rover :)
     return '' if from.zero? && till.zero?
 
     if from.zero?
-      "до #{till}"
+      [I18n.t('cck_forms.integer_range.till'), till].join(' ')
     elsif till.zero?
-      "от #{from}"
+      [I18n.t('cck_forms.integer_range.from'), from].join(' ')
     elsif from == till
       from.to_s
     else
@@ -145,16 +141,16 @@ class CckForms::ParameterTypeClass::IntegerRange # Rover :)
   end
 
   def default_integer_range_delimiter
-    "–"
+    '–'
   end
 
   def humanized_integer_ranges_for_select
     @extra_options[:ranges].map do |range_string|
       low, high = range_string.split(range_string_delimiter)
       if low.to_i.to_s != low.to_s
-        option_text = "до #{high}"
+        option_text = [I18n.t('cck_forms.integer_range.less_than'), high].join(' ')
       elsif high.to_i.to_s != high.to_s
-        option_text = "свыше #{low}"
+        option_text = [I18n.t('cck_forms.integer_range.more_than'), low].join(' ')
       else
         option_text = [low, high].join(default_integer_range_delimiter)
       end

data/lib/cck_forms/parameter_type_class/map.rb

@@ -12,10 +12,6 @@ class CckForms::ParameterTypeClass::Map
 
   mattr_accessor :google_maps_api_key
 
-  def self.name
-    'Точка на карте'
-  end
-
   # In MongoDB: {latlon: [x, y], zoom: z}
   #
   # In application: {
@@ -23,7 +19,7 @@ class CckForms::ParameterTypeClass::Map
   #   longitude: y,
   #   zoom: z
   # }
-  def self.demongoize_value(value, parameter_type_class=nil)
+  def self.demongoize_value(value, _parameter_type_class=nil)
     value = value.to_h
     value.stringify_keys!
     latlon = value['latlon'] || []
@@ -145,8 +141,6 @@ class CckForms::ParameterTypeClass::Map
   #
   # 1 click on a map places a point (writing to hidden fields). 1 click on a point removes it (emptying fields).
   #
-  # TODO: remove listeners of "city" events, it's out of scope for this gem
-  #
   # options:
   #
   #   value     - current point
@@ -180,21 +174,8 @@ class CckForms::ParameterTypeClass::Map
       inputs << value_builder.hidden_field(:type,      value: value['type'])
     end
 
-    city_id = id.clone
-    city_id['map'] = 'city'
-    city_id += '_value'
-
-    cities_js = []
-    city_class = if defined?(KazakhstanCities::City) then KazakhstanCities::City elsif defined?(City) then City end
-    if city_class
-      city_class.all.each do |city|
-        cities_js.push [city.id, {lat: city.lat.to_f, lon: city.lon.to_f, zoom: city.zoom.to_i}]
-      end
-    end
-    cities_js = Hash[cities_js].to_json
-
     allowed_maps = @@map_providers
-    map_names = {'google' => 'Google', 'yandex' => 'Яндекс'}
+    map_names = {'google' => 'Google', 'yandex' => 'Yandex'}
     selected_map_type = value['type'].in?(allowed_maps) ? value['type'] : allowed_maps.first
 
     switchers = []
@@ -242,7 +223,7 @@ class CckForms::ParameterTypeClass::Map
       window.onload = loadMapScripts;
       </script>
 
-      <div data-map-data-source data-options='#{options.to_json}' data-id="#{id}" data-cities='#{cities_js}' data-cityid="#{city_id}" data-allowed-maps='#{allowed_maps.to_json}' style="width: #{options[:width]}px; height: #{options[:height]}px">
+      <div data-map-data-source data-options='#{options.to_json}' data-id="#{id}" data-allowed-maps='#{allowed_maps.to_json}' style="width: #{options[:width]}px; height: #{options[:height]}px">
         #{map_html_containers.join}
       </div>
 
@@ -252,7 +233,7 @@ class CckForms::ParameterTypeClass::Map
   end
 
   # Returns a 64x64 IMG with a marker (see #img_tag)
-  def to_diff_value(options = {})
+  def to_diff_value(_options = {})
     demongoize_value!
     img_tag(64, 64, marker_size: :small)
   end

data/lib/cck_forms/parameter_type_class/phones.rb

@@ -8,10 +8,6 @@ class CckForms::ParameterTypeClass::Phones
   PREFIX              = Rails.application.config.cck_forms.phones.prefix
   NUMBER_PARTS_GLUE   = Rails.application.config.cck_forms.phones.number_parts_glue
 
-  def self.name
-    'Телефоны'
-  end
-
   # Filters input array for phone-like Hashes: prefix: ..., code: ..., number: ...
   # Cleans them up and returns.
   #
@@ -46,7 +42,7 @@ class CckForms::ParameterTypeClass::Phones
   end
 
   # Cleanup phone format
-  def self.demongoize_value(value, parameter_type_class=nil)
+  def self.demongoize_value(value, _parameter_type_class=nil)
     if value
       value.map do |phone|
         phone = phone.stringify_keys!
@@ -124,7 +120,7 @@ class CckForms::ParameterTypeClass::Phones
           $phones.children("p:last").after($newPhone);
         }
 
-        $phones.append('<a href="#" class="add_more">Добавить еще телефонов</a>');
+        $phones.append('<a href="#" class="add_more">#{I18n.t 'cck_forms.phones.add_more'}</a>');
         $phones.children(".add_more").click(function() {
           for(var i = 0; i < doTimes; ++ i) {
             createPhone();
@@ -163,7 +159,8 @@ HTML
   end
 
   def to_s(options = {})
-    HTML::FullSanitizer.new.sanitize to_html(options)
+    sanitizer = defined?(Rails::Html::FullSanitizer) ? Rails::Html::FullSanitizer : HTML::FullSanitizer
+    sanitizer.new.sanitize to_html(options)
   end
 
 

data/lib/cck_forms/parameter_type_class/string.rb

@@ -3,10 +3,6 @@
 class CckForms::ParameterTypeClass::String
   include CckForms::ParameterTypeClass::Base
 
-  def self.name
-    'Строка'
-  end
-
   # HTML input element
   def build_form(form_builder, options)
     set_value_in_hash options

data/lib/cck_forms/parameter_type_class/string_collection.rb

@@ -3,10 +3,6 @@
 class CckForms::ParameterTypeClass::StringCollection
   include CckForms::ParameterTypeClass::Base
 
-  def self.name
-    'Массив строк'
-  end
-
   # String: "aaa\r\nxxx" -> ["aaa", "xxx"]
   # :each: -> array
   def mongoize

data/lib/cck_forms/parameter_type_class/text.rb

@@ -3,19 +3,15 @@
 class CckForms::ParameterTypeClass::Text
   include CckForms::ParameterTypeClass::Base
 
-  def self.name
-    'Текст'
-  end
-
   # Builds a TEXTAREA
   def build_form(form_builder, options)
     set_value_in_hash options
     form_builder.text_area :value, {cols: 50, rows: 5, class: 'form-control'}.merge(options)
   end
 
-  # Wrap DIFF output to .wall (bootstrap-specific bounded block)
+  # Wraps DIFF output to .wall (bootstrap-specific bounded block)
   def to_diff_value(options = {})
-    to_html.presence.try do |html|
+    to_html(options).presence.try do |html|
       "<div class='well well-small'>#{html}</div>".html_safe
     end
   end

data/lib/cck_forms/parameter_type_class/time.rb

@@ -4,10 +4,6 @@ class CckForms::ParameterTypeClass::Time
   include CckForms::ParameterTypeClass::Base
   include CckForms::DateTime
 
-  def self.name
-    'Время'
-  end
-
   # Time SELECT
   def build_form(form_builder, options)
     set_value_in_hash options
@@ -19,7 +15,7 @@ class CckForms::ParameterTypeClass::Time
   end
 
   # "19:34"
-  def to_s(options = nil)
+  def to_s(_options = nil)
     if value.is_a? Time
       the_value = {
           '(4i)' => value.hour,

data/lib/cck_forms/parameter_type_class/watermarkless_album.rb

@@ -1,10 +1,6 @@
 # Album without watermark. A subclass of Album.
 #
 class CckForms::ParameterTypeClass::WatermarklessAlbum < CckForms::ParameterTypeClass::Album
-  def self.name
-    'Альбом без водяного знака'
-  end
-
   def cck_image_type
     CckForms::ParameterTypeClass::WatermarklessImage
   end

data/lib/cck_forms/parameter_type_class/watermarkless_image.rb

@@ -1,10 +1,6 @@
 # Image without watermark. A subclass of Image.
 #
 class CckForms::ParameterTypeClass::WatermarklessImage < CckForms::ParameterTypeClass::Image
-  def self.name
-    'Картинка без водяного знака'
-  end
-
   def self.additional_file_attributes
     {no_wm: '1'}
   end

data/lib/cck_forms/parameter_type_class/work_hours.rb

@@ -3,16 +3,11 @@
 class CckForms::ParameterTypeClass::WorkHours
   include CckForms::ParameterTypeClass::Base
 
-  def self.name
-    'Часы работы'
-  end
-
   DAYS = %w{ mon tue wed thu fri sat sun }
-  DAYS_RU_SHORT = %w{ Пн Вт Ср Чт Пт Сб Вс }
 
-  # mon -> Пн
-  def self.day_en_to_ru_short(day)
-    DAYS_RU_SHORT[DAYS.index(day.to_s)]
+  # mon -> Mon
+  def self.day_to_short(day)
+    I18n.t "cck_forms.work_hours.day_short.#{day}"
   end
 
   # mon: {open_time: ..., open_24_hours: ...}, tue: {...}, ... -> MongoDB Hash
@@ -91,13 +86,13 @@ class CckForms::ParameterTypeClass::WorkHours
     groups.each_pair do |hours_description, days|
       if hours_description.present?
         if days.length == 7
-          template = with_tags ? '<span class="workhours-group">%s, <span class="workhours-group-novac">без выходных</span></span>' : '%s, без выходных'
+          template = with_tags ? %Q{<span class="workhours-group">%s, <span class="workhours-group-novac">#{I18n.t 'cck_forms.work_hours.no_vacations'}</span></span>} : "%s, #{I18n.t 'cck_forms.work_hours.no_vacations'}"
           result << sprintf(template, hours_description)
         else
           if days == %w{ mon tue wed thu fri }
-            days_description = 'будние'
+            days_description = I18n.t 'cck_forms.work_hours.work_days'
           elsif days == %w{ sat sun }
-            days_description = 'сб, вс'
+            days_description = I18n.t 'cck_forms.work_hours.sat_sun'
           else
             days_description = CckForms::ParameterTypeClass::WorkHours.grouped_days_string(days).mb_chars.downcase
           end
@@ -110,7 +105,7 @@ class CckForms::ParameterTypeClass::WorkHours
     result.join('; ').html_safe
   end
 
-  def to_s(options = nil)
+  def to_s(_options = nil)
     to_html with_tags: false
   end
 
@@ -127,7 +122,7 @@ class CckForms::ParameterTypeClass::WorkHours
         groups << []
       end
 
-      groups.last << day_en_to_ru_short(day)
+      groups.last << day_to_short(day)
       prev_index = index
     end
 
@@ -190,21 +185,21 @@ class CckForms::ParameterTypeClass::WorkHours
     def to_s_without_day
       result = ''
       if open_24_hours
-        return 'круглосуточно'
+        return I18n.t 'cck_forms.work_hours.24h'
       elsif time_present?(open_time) or time_present?(close_time)
         ots, cts = time_to_s(open_time), time_to_s(close_time)
         if ots and cts
           result = sprintf('%s–%s', ots, cts)
         elsif ots
-          result = sprintf('с %s', ots)
+          result = sprintf("#{I18n.t 'cck_forms.work_hours.from'} %s", ots)
         else
-          result = sprintf('до %s', cts)
+          result = sprintf("#{I18n.t 'cck_forms.work_hours.till'} %s", cts)
         end
       end
 
       if open_until_last_client
         result += ' ' if result.present?
-        result += 'до последнего клиента'
+        result += I18n.t 'cck_forms.work_hours.until_last_client'
       end
 
       result
@@ -223,16 +218,16 @@ class CckForms::ParameterTypeClass::WorkHours
 
       if template
         header = ['<ul class="nav nav-pills">']
-        CckForms::ParameterTypeClass::WorkHours::DAYS.each { |day| header << '<li><a href="#"><input name="' + form_builder.object_name + '[days]" type="checkbox" value="' + day + '" /> ' + CckForms::ParameterTypeClass::WorkHours.day_en_to_ru_short(day) + '</a></li>' }
+        CckForms::ParameterTypeClass::WorkHours::DAYS.each { |day| header << '<li><a href="#"><input name="' + form_builder.object_name + '[days]" type="checkbox" value="' + day + '" /> ' + CckForms::ParameterTypeClass::WorkHours.day_to_short(day) + '</a></li>' }
         header = header.push('</ul>').join
       else
-        header = sprintf '<strong>%s</strong>:%s', CckForms::ParameterTypeClass::WorkHours::day_en_to_ru_short(day), form_builder.hidden_field(:day)
+        header = sprintf '<strong>%s</strong>:%s', CckForms::ParameterTypeClass::WorkHours::day_to_short(day), form_builder.hidden_field(:day)
       end
 
       open_until_last_client_html = unless options[:hide_open_until_last_client]
                                       <<HTML
                     <div class="checkbox">
-                      <label class="form_work_hours_option">#{ form_builder.check_box :open_until_last_client } до&nbsp;последнего&nbsp;клиента</label>
+                      <label class="form_work_hours_option">#{form_builder.check_box :open_until_last_client} <nobr>#{I18n.t 'cck_forms.work_hours.until_last_client'}</nobr></label>
                     </div>
 HTML
                                     end
@@ -246,12 +241,12 @@ HTML
             <table width="100%">
               <tr>
                 <td width="60%" class="form-inline">
-                  с #{ open_time_form }
-                  по #{ close_time_form }
+                  #{I18n.t 'cck_forms.work_hours.time_from'} #{open_time_form}
+                  #{I18n.t 'cck_forms.work_hours.time_till'} #{close_time_form}
                 </td>
                 <td width="40%">
                     <div class="checkbox">
-                      <label class="form_work_hours_option">#{ form_builder.check_box :open_24_hours } круглосуточно</label>
+                      <label class="form_work_hours_option">#{form_builder.check_box :open_24_hours} #{I18n.t 'cck_forms.work_hours.24h'}</label>
                     </div>
                     #{open_until_last_client_html}
                 </td>

data/lib/cck_forms/version.rb

@@ -1,3 +1,3 @@
 module CckForms
-  VERSION = '3.1.1'
+  VERSION = '3.1.2'
 end

data/vendor/assets/javascripts/cck_forms/jquery.workhours.js

@@ -1,8 +1,7 @@
 $(function() {
 
     /**
-     * Виджет-помогайка для установки и получения значения из ХТМЛ-объекта 1 дня или группы (поля у низ почти одинаковые).
-     * При установке значения событи change не вызывается!
+     * Helper widget to set & obtain data from HTML of a single day or days group (almost identical)
      */
     $.widget("cck.workhoursday", {
 
@@ -30,14 +29,14 @@ $(function() {
         },
 
         /**
-         * Получить название дня (mod, tue).
+         * Day name (mod, tue)
          */
         day: function() {
             return this._$day.val();
         },
 
         /**
-         * Получить или установить галку "круглосуточно".
+         * Gets or sets "open 24 hours" checkbox state
          */
         open24Hours: function(value) {
             if(value !== undefined) {
@@ -48,7 +47,7 @@ $(function() {
         },
 
         /**
-         * Получить или установить галку "до последнего клиента".
+         * Gets or sets "open until last client" checkbox state
          */
         openUntilLastClient: function(value) {
             if(value !== undefined) {
@@ -59,8 +58,7 @@ $(function() {
         },
 
         /**
-         * Преобразовать значения переданных инпутов в строку вида "20:35". Если не получится, или одно из значений пустое,
-         * вернет null.
+         * Converts passed input values to string of form "20:35". If failed, or any of values is empty, returns null
          */
         inputsToTime: function($hoursInput, $minutesInput) {
             var h = $hoursInput.val() == '' ? null : $hoursInput.val() * 1;
@@ -74,7 +72,7 @@ $(function() {
         },
 
         /**
-         * Преобразовать хэш вида {hours: 20, minutes: 5} в значения переданных инпутов. Можно по одному.
+         * Converts a hash of form {hours: 20, minutes: 5} into input values
          */
         timeToInputs: function(value, $hoursInput, $minutesInput) {
             if(value.hours !== undefined) {
@@ -87,7 +85,7 @@ $(function() {
         },
 
         /**
-         * Вернуть (строка) или установить (из хэша) значения времени открытия.
+         * Get (as a string) or set (from a hash) open time values
          */
         openTime: function(value) {
             if(value !== undefined) {
@@ -98,7 +96,7 @@ $(function() {
         },
 
         /**
-         * Вернуть (строка) или установить (из хэша) значения времени закрытия.
+         * Get (as a string) or set (from a hash) close time values
          */
         closeTime: function(value) {
             if(value !== undefined) {
@@ -109,12 +107,15 @@ $(function() {
         },
 
         /**
-         * Пустой ли день? Пустой тогда, когда не выбрано времени и чекбоксы не включены.
+         * Returns true, if the day is empty, that is neither open/close times are set nor checkboxes are checked
          */
         isEmpty: function() {
             return !this.open24Hours() && !this.openUntilLastClient() && !this.openTime() && !this.closeTime();
         },
 
+        /**
+         * If this object is equal to another
+         */
         equalTo: function($other) {
             return (
                 this.open24Hours() == $other.workhoursday("open24Hours") &&
@@ -125,9 +126,8 @@ $(function() {
         },
 
         /**
-         * Вернет или установит текущее значение скопом. Через массив, порядок элементов: "круглосуточно", "до послед.
-         * клиента", "время открытия", "время закрытия". Время открытия и закрытия устанавливается через строку "ЧЧ:ММ",
-         * в отличие от метода closeTime({hours: ..., minutes: ...}).
+         * Gets or sets current state from array: [is_open_24h, is_open_until_last_client, open_time, close_time]
+         * Open and close times are string of form "hh:mm"
          */
         value: function(value) {
             if(value) {
@@ -158,26 +158,26 @@ $(function() {
 
 
 
-    // последний использованный ID группы
+    // the last group ID taken
     var cckWorkhoursGroupId = 0;
 
     /**
-     * Виджет блока "режим работы". Навешивается на весь ДИВ со строками для каждого дня недели, прячет их и подменяет
-     * сгруппированными строками, когда на несколько дней (определяется включенными галочками) идет один режим работы:
+     * Work hours widget. Accepts a series of DIVs for each week day and converts it into a grouped series, where
+     * days are grouped by the same "work value" (checkboxes & open/close time):
      *
-     *   > [^] Пн   [^] Вт   [^] Ср   [^] Чт   [^] Пт   [_] Сб   [_] Вс
-     *   > c [09]:[00] по [20]:[00]   [_] круглосуточно   [_] до последнего клиента
+     *   > [^] Mon   [^] Tue   [^] Wed   [^] Thu   [^] Fri   [_] Sat   [_] Sun
+     *   > from [09]:[00] till [20]:[00]   [_] open 24 hours   [_] open until last client
      *
-     *   > [_] Пн   [_] Вт   [_] Ср   [_] Чт   [_] Пт   [^] Сб   [^] Вс
-     *   > c [10]:[00] по [__]:[__]   [_] круглосуточно   [^] до последнего клиента
+     *   > [_] Mon   [_] Tue   [_] Wed   [_] Thu   [_] Fri   [^] Sat   [^] Sun
+     *   > from [10]:[00] till [__]:[__]   [_] open 24 hours    [^] open until last client
      *
-     *   [ Добавить дней ]
+     *   [ Add days ]
      *
-     * Ссылка "Добавить дней" добавляет еще одну пусту группу. При указании в группе дня, который уже есть в другой,
-     * оттуда он удаляется. Если группа опустела (нет дней), она удаляется.
+     * "Add days" link add an empty group. If a day is added to a group, it is removed from the previous group where it
+     * was. If a group is emptied (no more days left), it is deleted.
      *
-     * Реально данные из группы при всех манипуляциях переносятся в спрятанные строки, так что для бэкенда ничего
-     * не меняется - профит!
+     * The key is, when manipulating groups all the data is propagated to the hidden inputs in original HTML, thus
+     * the overall form data stays in the same format and a controller will not even now the widget was constructed.
      */
     $.widget("cck.workhours", {
 
@@ -189,12 +189,13 @@ $(function() {
         _$template: null,
 
         /**
-         * Инициализируем виджет.
+         * Constructor
          */
         _create: function() {
             var $form =  this.element;
             var $widget = this;
 
+            // TODO: i18n in JS?
             var $addGroupLink = $('<a href="#add-group">Добавить дней</a>').click(function() {
                 $widget.createGroup([], []);
             });
@@ -206,7 +207,7 @@ $(function() {
             var groups = [];
             $widget._days = [];
 
-            // существующие дни распихаем по группам и спрячем
+            // split existing days into groups and hide their DIVs
             $form.children(".form_work_hours_day:not(.form_work_hours_day_template)").each(function() {
                 var $day = $(this).workhoursday().hide();
                 var dayName = $day.workhoursday("day");
@@ -230,53 +231,52 @@ $(function() {
         },
 
         /**
-         * Создаем новый ДОМ-элемент "группы" дней из шаблона. Навешиваем нужных слушателей.
+         * Creates new DOM node for a group & bind event listeners
          */
         createGroup: function(days, value) {
 
-            // сделаем ХТМЛ новой группы, заменив id|name ~= template на какой-то новый ID
+            // make HTML, substitute id|name ~= template for a real ID (new one)
             var newHtml = this._$template[0].outerHTML.replace(/((?:id|name)="[^"]*)template([^"]*")/g, "$1" + this.newGroupId() + "$2");
             var $newGroup = $(newHtml);
 
-            // отметим чекбоксы, соотв. выбранным дням
+            // mark checkboxes
             for(var i = 0, ic = days.length; i < ic; ++ i) {
                 $newGroup.find("input[name$=\"[days]\"][value=" + days[i] + "]").prop("checked", true).closest("li").addClass("active");
             }
 
-            // спрячем чекбоксы, вместо них будут ссылко-кнопки
+            // hide checkboxes, link-o-buttons will be in their place
             $newGroup.find("input[name$=\"[days]\"]").hide();
             $newGroup.find(".nav-pills").on("click", "a", function(event) {
-                // событие могло произойти на чекбоксе внутри этой ссылки, это нам не нужно
+                // skip events originated at checkbox inside this link
                 if(event.target == this) {
                     $(this).children("input:checkbox").click();
                     return false;
                 }
             });
 
-            // будем слушать изменения в формах, чтобы транслировать в соотв. группе скрытые поля дней
+            // listen for the form changes to propagate to the original DOM nodes
             var $widget = this;
             $newGroup.on("change", "input, select", function() {
                 $widget.groupChangeListener(this);
             });
 
-            // добавим ХТМЛ и его виджет workhoursday
+            // make HTML & instantiate widget workhoursday
             $newGroup.workhoursday().workhoursday("value", value).insertBefore(this._lastP).show();
         },
 
         /**
-         * Слушаем изменения полей группы.
+         * Listens for a group events
          */
         groupChangeListener: function(input) {
 
             var $group = $(input).closest(".form_work_hours_day");
 
-            // если событие произошло на чекбоксах "Пн", "Вт"
+            // an event from day checkboxes?
             if(input.getAttribute("name").substr(-6) == '[days]') {
 
                 if(input.checked) {
 
-                    // "включили" этот день, значит, "выключим" его в другой группе (группах)
-                    // сделаем себе временную метку, чтобы себя не выключить тоже
+                    // this day is on in this group, so make it off in another groups
                     var dayName = input.value;
 
                     if (!$group.data("multiDays")) {
@@ -285,18 +285,18 @@ $(function() {
                         input.removeAttribute("data-hold-check");
                     }
 
-                    // теперь обновим все поля из нашей группы для нового (включенного) дня
+                    // propagate all the fields from this group to the day's original fields
                     this._days[dayName].workhoursday("value", $group.workhoursday("value"));
 
-                    // выделим метку этого инпута
+                    // mark the day as active
                     input.parentNode.parentNode.className += " active";
 
                 } else {
 
-                    // "выключили" этот день в нашей группе, значит, обнулим его поля
+                    // propagate the off state to the day's original fields
                     this._days[input.value].workhoursday("value", []);
 
-                    // удалим группу, если в ней не осталось включенных дней, иначе уберем метку активного дня
+                    // remove this group if no more days left inside
                     if($group.find("input:checked[name$='[days]']").size() == 0) {
                         $group.remove();
                     } else {
@@ -304,57 +304,56 @@ $(function() {
                     }
                 }
 
-            // если событие НЕ на чекбоксах "Пн", "Вт" и т. п., то транслируем в соотв. скрытые поля
+            // an event not from day checkboxes -> propagate the value to the original node
             } else {
 
                 var match = input.getAttribute("name").match(/(?:\[(open_time|close_time)\])?\[([^\]]*)\]$/);
                 var fieldName = (match[1] ? match[1] + '_' : '') + match[2];
                 var inputNodeName = input.nodeName.toLowerCase();
 
-                // сначала получим текущие отмеченные дни в группе
+                // active days in this group
                 var days = [];
                 $group.find("input:checked[name$='[days]']").each(function() {
                     days.push(this.value);
                 });
 
-                // сделаем несколько простых проверок и отреагируем, меняя значения полей для удобства пользователя
-                // если текущее поле выбрано, то...
+                // some UX friendly checks
                 if(inputNodeName == "input" && input.checked || inputNodeName == "select" && input.value != '') {
                     switch(fieldName) {
 
-                        // открыто 24 часа? обнулим время работы
+                        // open 24 hours? nullify open/close time
                         case "open_24_hours":
                             $group.workhoursday("openTime", {hours: '', minutes: ''}).workhoursday("closeTime", {hours: '', minutes: ''});
                         break;
 
-                        // открыто до последнего клиента? обнулим время закрытия
+                        // open until last client? nullify close time
                         case "open_until_last_client":
                             $group.workhoursday("closeTime", {hours: '', minutes: ''});
                         break;
 
-                        // выбрано время закрытия? уберем галку "открыто до последнего клиента"
+                        // close time set? uncheck "open until last" checkbox
                         case "close_time_hours":
                         case "close_time_minutes":
                             $group.workhoursday("openUntilLastClient", false);
-                        // break пропущен намеренно
+                        // break skipped on purpose!
 
-                        // выбрано к. л. время? уберем галку "открыто 24 часа"
+                        // any time set? uncheck "open 24h" checkbox
                         case "open_time_hours":
                         case "open_time_minutes":
                             $group.workhoursday("open24Hours", false);
                         break;
                     }
 
-                    // указан час, но не указаны минуты? установим :00
+                    // hour is set but minutes are not? set :00
                     var camelCasedTime = this.camelCaseTime(fieldName, 6);
                     if((fieldName == "open_time_hours" || fieldName == "close_time_hours") && !$group.workhoursday(camelCasedTime)) {
                         $group.workhoursday(camelCasedTime, {minutes: 0})
                     }
 
-                // если текущее поле не выбрано...
+                // not input
                 } else {
 
-                    // устанавливаем часы или минуты? обнулим минуты или часы соотв.
+                    // setting hours or minutes? nullify minutes or hours respectively
                     if(fieldName == "open_time_hours" || fieldName == "close_time_hours") {
                         $group.workhoursday(this.camelCaseTime(fieldName, 6), {minutes: ''});
                     } else if(fieldName == "open_time_minutes" || fieldName == "close_time_minutes") {
@@ -362,7 +361,7 @@ $(function() {
                     }
                 }
 
-                // теперь найдем соотв. скрытые дни и установим у них значение группы
+                // all checks are made, propagate the value to days inside this group
                 var value = $group.workhoursday("value");
                 for(var i = 0, ic = days.length; i < ic; ++ i) {
                     this._days[days[i]].workhoursday("value", value);
@@ -384,14 +383,14 @@ $(function() {
         },
 
         /**
-         * Сгенерируем новый уникальный ID группы.
+         * Generates a unique sequential group DOM ID
          */
         newGroupId: function() {
             return 'group' + (cckWorkhoursGroupId ++);
         },
 
         /**
-         * Затычка, чтобы у предыдущих полей объекта всегда ставить в конце запятую.
+         * To not worry about redundant comma after the last hash key
          */
         slashzero: null
 

data/vendor/assets/javascripts/cck_forms/map.js.coffee

@@ -54,10 +54,10 @@ class CckFormsMap.AbstractMap
 
         @.setMarkerToPoint latlon
         @.setCenter latlon
-        @.setZoom 17 # подобран экспериментально
+        @.setZoom 17 # adjusted by experiment for better view
 
     xhr.fail ->
-      alert "Произошла ошибка на сервере"
+      alert "Server error encountered"
 
 
   refresh: ->
@@ -209,7 +209,7 @@ class CckFormsMap.GoogleMap extends CckFormsMap.AbstractMap
     @.refresh()
 
   refresh: ->
-    center = @internalMapAPI.getCenter() # для того, чтобы центр карты не занимался ерундой
+    center = @internalMapAPI.getCenter() # keep the map center rock solid
     google.maps.event.trigger(@internalMapAPI, "resize")
     @internalMapAPI.setCenter(center)
 
@@ -260,32 +260,6 @@ window.init_map = (canvas) ->
 
   $canvas.data('maps', maps)
 
-
-  cityInputSelectorst = [
-        '#' + $source.data("cityid"),
-        '[data-behavior="map_city_change"]'
-  ].join(', ')
-  $cityInput = $(cityInputSelectorst)
-
-  if $cityInput.data('mapCityValue')
-    mapCityValue = $cityInput.data('mapCityValue')
-    $cityInput   = $cityInput.find(mapCityValue)
-
-  handleCityChange = =>
-    cities = $source.data("cities")
-    city   = cities[$cityInput.val()]
-
-    if city
-      latLng = new google.maps.LatLng(city.lat, city.lon) unless hasLatLng
-      zoom   = city.zoom unless hasZoom
-
-    $.each $canvas.data('maps'), (i, map) ->
-      map.setCenter latitude: latLng.lat(), longitude: latLng.lng()
-      map.setZoom zoom
-
-  $cityInput.change handleCityChange
-  handleCityChange() unless $lat.val() or $lon.val()
-
   activeMapType = $canvas
   .find('.cck-map-switchers a.active')
   .data("mapType")

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: cck_forms
 version: !ruby/object:Gem::Version
-  version: 3.1.1
+  version: 3.1.2
 platform: ruby
 authors:
 - Ilya Konanykhin
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-03-06 00:00:00.000000000 Z
+date: 2017-03-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails