c80_push 0.1.0.2 → 0.1.0.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: b4d991e9e1a4d40ab4a2788f801fc5bda42e61d2
-  data.tar.gz: 37d7c1a99822e278f8b2b4db16255fdd29a187a8
+  metadata.gz: b2bc7b9516f5f7edb8dffce344f1c2465c7dca6d
+  data.tar.gz: 5684422a46e41045c3de9a5ba094c8a8b9c2ab3c
 SHA512:
-  metadata.gz: c88589031569da99e0158285b792473c6b0c1a580106b6830ad1ffaee861546c219a760983054d5d126da6acf185fb4bdfb8060ad8c2a6eceb171381de09d5f1
-  data.tar.gz: 9a7b1e94d5105aed752b535222df57341519ae0e7fde738650edda28ccc362185b010eea44616551fbc6b039440b050f3d2bffdb2a59b05609be3d9bd67f595c
+  metadata.gz: 5bbd6b017f98a1d3c469f237d1aef3d936749e7a757db8c07f318e9375f6ada96b699a3d441f8566cc2674787bdca379187d9b5e5fe27a119afbe025116a8f9a
+  data.tar.gz: 1b5bcdc9f2b603a910bf9209ba040d0853dd91a5c7a9639a994b8f0e2aeae26f124dabd3154b0b0c25e0de116d0fdb43c9158f16234b60e56f21f881e6fd99a8

data/README.md

@@ -2,13 +2,14 @@
 
 Добавляет функционал дилеров и офисов продаж. 
 Это *ActiveAdmin* plugin.
-Использует *Bootstrap* и *Bootstrap Select*. 
+Использует *Bootstrap* и *Bootstrap Select*: как во *frontend*,
+так и в *backend*.
 
 Содержит модели: 
 
-* Дилер
-* Офис
-* Регион
+* *Регион* (habtm Dealers)
+* *Дилер* (has_many Offices)
+* *Офис* (belongs_to Dealer)
 
 Добавляет пункт меню:
 
@@ -21,7 +22,7 @@
 Available helpers:
 
 * `c80_push_render_page_dealers`: отрисует js-компонент с yandex картой
-и фильтруемым списком дилеров слева.
+и фильтруемым, кликабельным списком офисов слева.
 
 ## Installation
 
@@ -36,6 +37,8 @@ And then execute:
 ```bash
 $ bundle
 $ rake db:migrate
+$ rails generate c80_push:install
+$ rake db:seed:c80_push_02_fill_presets
 ```
 
 Add this line to host app's `application_controller.rb`:
@@ -49,7 +52,6 @@ Add these lines to host app's `active_admin.js.coffee`:
 ```js
 #= require bootstrap
 #= require bootstrap-select
-#= require c80_yax
 #= require c80_push_backend
 ```
 
@@ -67,10 +69,61 @@ Add these lines to host app's 'application.scss':
 @import "bootstrap-sprockets";
 @import "bootstrap";
 @import "bootstrap-select";
-@include "c80_push";
+@import "c80_push";
 ```
 
-## Процесс разработки JS функционала 
+## Customization
+
+Т.к. исходя из ТЗ было не ясно, есть ли у *Офисов* такие поля, как
+сайт, email и регион, они были добавлены (на всякий случай), и поля
+сделаны необязательными. И их можно "отключить", добавив в host app
+такой файл `app/assets/stylesheets/custom/backend/c80_push/dealers.scss`:
+
+```scss
+/* у Офисов скроем неиспользуемые поля */
+body.admin_dealers {
+  &.edit, &.new {
+
+    li[id*='dealer_offices_attributes_'] {
+      &[id*='site_input'] {
+        display: none;
+      }
+      &[id*='email_input'] {
+        display: none;
+      }
+      &[id*='regions_input'] {
+        display: none;
+      }
+    }
+  }
+}
+```
+
+А сам файл подключить в `active_admin.scss`:
+
+```scss
+@import "custom/backend/**/*";
+```
+
+# TODO
+
+* [X] Добавить валидацию поля `gps` модели *Office*.
+
+    - [ ] Попробовать подключить к `backend` *client side validation*.
+
+* [X] Реализовать функционал цветных маркеров.
+
+    - [ ] Внутри `bootstrap selectpicker` должны рисоваться цветные прямоугольники: 
+    [/formtastic/inputs/select_input.rb](https://github.com/justinfrench/formtastic/blob/master/lib/formtastic/inputs/select_input.rb),
+    [bootstrap selectpicker: option with data-content](https://stackoverflow.com/questions/26069286/bootstrap-with-select-list-that-contains-only-images)
+
+* [ ] Вынести `*.rb` файлы, подлежащие кастомизации в `templates` (предварительно
+определив, что может еще быть кастомизировано), наваять инсталлятор,
+отметить это в `README`.
+
+    - [X] seeds
+
+# Процесс разработки JS функционала 
 
 Теперь пора заняться js.
 
@@ -92,13 +145,36 @@ Add these lines to host app's 'application.scss':
 * [X] При наведении/клике на `.li_office` - должна подсвечиваться соответствующая 
 точка на карте.       
  
-* [ ] Вставить из `psd` синий маркер.
+* [X] Реализовать функционал цветных маркеров.
 
 * [X] Когда меняется регион:
 
     * [X] Должны отобразиться релевантные `.li_office`
     * [X] Должны отобразиться релевантные точки на карте.
 
+## Процесс добавления функционала "Цвет маркеров дилеров"
+
+* [X] добавляем таблицу `Presets` с полями `preset_name`
+и `color`, причём `color` я должен забить руками с помощью color picker
+со страницы докуметации Яндекс.карты (`color` используется только в админке
+для иллюстрации "что такое preset").
+
+* [X] Таблица `Presets` должна быть habtm-связана с таблицей `:dealers`
+
+* [X] Таблицу заполнить содержимым с помощью seed файла, пункт в админке не делать.
+
+* [X] при сборке хеша для Яндекс.карт - собираем в него свойства меток
+
+    - [X] Если цвет не задан - по-умолчанию добавляем голубой 
+
+* в админке:
+
+    - [ ] в форме редактирования/создания: у каждого дилера должен быть селект, в котором видно как
+    и название `preset-а`, так и его цвет
+
+    * [X] на индексной странице: должен быть столбец, в котором выводится название
+    `preset-a` и его цвет
+
 ## Цитата из ТЗ
 
 > Данный раздел представляет собой список компаний, у которых указаны
@@ -114,11 +190,11 @@ Add these lines to host app's 'application.scss':
 > Выбрав тот или иной регион, пользователь видит слева от
 > интерактивной карты список компаний, а также адреса их офисов продаж.
 > 
-> Офисы продаж ORIMA и копании-дилеры (и адреса их офисов продаж)
+> Офисы продаж компании и копании-дилеры (и адреса их офисов продаж)
 > разграничены на интерактивной карте маркерами разного цвета.
 > 
 > В списке слева от интерактивной карты сначала отображаются офисы
-> продаж компании ORIMA, а затем список компаний-дилеров
+> продаж компании, а затем список компаний-дилеров
 > (и адресов их офисов продаж).
 > 
 > С помощью инструментов панели управления сайтом можно создавать

data/app/admin/c80_push/dealers.rb

@@ -8,6 +8,7 @@ ActiveAdmin.register C80Push::Dealer, as: 'Dealer' do
                 :ord,
                 :site,
                 :email,
+                :preset_ids => [],
                 :region_ids => [],
                 :offices_attributes => [
                     :id,
@@ -49,8 +50,8 @@ ActiveAdmin.register C80Push::Dealer, as: 'Dealer' do
     f.inputs 'Свойства' do
       f.input :title
       f.input :ord
-      f.input :site, hint: I18n.t('c80_push.active_admin.hints.site')
-      f.input :email, hint: I18n.t('c80_push.active_admin.hints.email')
+      f.input :site, hint: I18n.t('c80_push.active_admin.hints.dealer.site')
+      f.input :email, hint: I18n.t('c80_push.active_admin.hints.dealer.email')
       f.input :regions,
               :as => :select,
               :input_html => {
@@ -63,6 +64,24 @@ ActiveAdmin.register C80Push::Dealer, as: 'Dealer' do
                   :multiple => false
               },
               :include_blank => true
+      f.input :presets,
+              :as => :select,
+              :collection => C80Push::Preset.all.map {|p| [p.title, p.id]},
+              # :collection => C80Push::Preset.all.map { |p|
+              #   "<option value='#{p.id}'>#{p.title}</option>".html_safe
+              # },
+              # :collection => "<option></option>",
+              :input_html => {
+                  :title => '',
+                  :class => 'selectpicker',
+                  :data => {
+                      :size => 10,
+                      :width => '600px'
+                  },
+                  :multiple => false
+              },
+              :include_blank => true,
+              :hint => "<a href='https://tech.yandex.ru/maps/doc/jsapi/2.1/ref/reference/option.presetStorage-docpage/' target='_blank'>Справка по цветам</a>".html_safe
 
       f.has_many :offices,
                  :allow_destroy => true,
@@ -71,9 +90,9 @@ ActiveAdmin.register C80Push::Dealer, as: 'Dealer' do
         office.input :ord
         office.input :addr
         office.input :gps
-        office.input :tel, hint: I18n.t('c80_push.active_admin.hints.tel')
-        office.input :site, hint: I18n.t('c80_push.active_admin.hints.site')
-        office.input :email, hint: I18n.t('c80_push.active_admin.hints.email')
+        office.input :tel, hint: I18n.t('c80_push.active_admin.hints.office.tel')
+        office.input :site, hint: I18n.t('c80_push.active_admin.hints.office.site')
+        office.input :email, hint: I18n.t('c80_push.active_admin.hints.office.email')
         office.input :regions,
                 :as => :select,
                 :input_html => {

data/app/assets/javascripts/c80_push/frontend/c80_push_page_dealers/_map_set_objects.js

@@ -9,11 +9,13 @@ function map_set_objects(region_id) {
 
     // соберём коллекцию точек
     var c = new ymaps.GeoObjectCollection();
-    var icoords, iprops; // loop vars
+    var icoords, iprops, isetts; // loop vars
     for (var i = 0; i < n; i++) {
         icoords = arr['coords'][i];
         iprops = arr['props'][i];
-        c.add(new ymaps.Placemark(icoords, iprops));
+        isetts = arr['setts'][i];
+        console.log(isetts);
+        c.add(new ymaps.Placemark(icoords, iprops, isetts));
     }
 
     // сначала уберём все точки

data/app/assets/stylesheets/c80_push/backend/dealer_preset_span.scss

@@ -0,0 +1,5 @@
+span.dealer_preset_span {
+  display: inline-block;
+  width: 100px;
+  height: 25px;
+}

data/app/assets/stylesheets/c80_push_backend.scss

@@ -2,4 +2,4 @@
  *= require_self
  */
 
-//@import "c80_push/backend/**/*";
+@import "c80_push/backend/**/*";

data/app/helpers/c80_push/admin_helper.rb

@@ -42,10 +42,11 @@ module C80Push
       res = "#{dealer.title}"
       res += "<br><sub>email: #{format_val(dealer.email)}</sub>"
       res += "<br><sub>site: #{format_val(dealer.site)}</sub>"
+      res += "<br><sub>цвет: #{format_dealer_preset_val(dealer)}</sub>"
       res.html_safe
     end
 
-    private
+    # private
 
     def format_val(val)
       res = '-'
@@ -55,6 +56,19 @@ module C80Push
       "<span class='c80_md_link_color'>#{res}</span>".html_safe
     end
 
+    def format_dealer_preset_val(dealer)
+      res = '-'
+      if dealer.presets.count > 0
+        p = dealer.presets.first
+        res = format_preset_val(p)
+      end
+      res.html_safe
+    end
+
+    def format_preset_val(preset)
+      "<span class='dealer_preset_span' style='background-color:#{preset.color}'></span>"
+    end
+
   end
 
 end

data/app/helpers/c80_push/application_helper.rb

@@ -32,12 +32,14 @@ module C80Push
     #
     # {
     #   all: {          - здесь собираем все офисы всех регионов (для лаконичности js)
-    #       coords: [],
-    #       props: []
+    #       coords: [], - массив двух координат, например: [12.345, 12.345]
+    #       props: [],  - свойства объекта (офиса): тот текст, который выводится в balloon-e
+    #       setts: []   - свойства метки (цвет)
     #   }
     #   <region_id>: [  - данные офисов разложены по регионам
     #       coords: [],
-    #       props: []
+    #       props: [],
+    #       setts: []
     #   ]
     # }
     #
@@ -46,14 +48,15 @@ module C80Push
       res = {
           all: {
               coords: [],
-              props: []
+              props: [],
+              setts: []
           }
       }
 
       rdo.each do |region|
         region.dealers.each do |dealer|
           dealer.offices.each do |office|
-            res[region.id] = { coords:[], props:[] } if res[region.id].nil?
+            res[region.id] = { coords:[], props:[], setts:[] } if res[region.id].nil?
 
             # координаты точки (это массив двух точек)
             gps = office.gps_arr
@@ -66,13 +69,20 @@ module C80Push
                 id:                   office.id
             }
 
+            # свойства метки - цвет
+            setts = {
+                preset: "islands##{prepare_dealer_preset(dealer)}"
+            }
+
             # фиксируем в хэше региона
             res[region.id][:coords] << gps
             res[region.id][:props] << props
+            res[region.id][:setts] << setts
 
             # фиксируем в all-хэше
             res[:all][:coords] << gps
             res[:all][:props] << props
+            res[:all][:setts] << setts
 
           end
         # для удобства в js: зафиксируем кол-во офисов региона
@@ -104,5 +114,16 @@ module C80Push
       res
     end
 
+    # сформировать название preset-а Дилера
+    # для цветного маркера.
+    # По умолчанию вернёт `blueDotIcon`
+    def prepare_dealer_preset(dealer)
+      res = 'blueDotIcon'
+      unless dealer.presets.count.zero?
+        res = dealer.presets.first.title
+      end
+      res
+    end
+
   end
 end

data/app/models/c80_push/dealer.rb

@@ -1,6 +1,7 @@
 module C80Push
   class Dealer < ActiveRecord::Base
     has_and_belongs_to_many :regions
+    has_and_belongs_to_many :presets
     has_many :offices, :dependent => :destroy
     accepts_nested_attributes_for :offices,
                                   :reject_if => lambda { |attributes|

data/app/models/c80_push/office.rb

@@ -15,7 +15,8 @@ module C80Push
               :presence => true
 
     validates :gps,
-              :presence => true
+              :presence => true,
+              :format => { with: /\A\d+\.\d+,\s*\d+\.\d+\z/ }
 
     validates :tel,
               :presence => true

data/app/models/c80_push/preset.rb

@@ -0,0 +1,5 @@
+module C80Push
+  class Preset < ActiveRecord::Base
+    has_and_belongs_to_many :dealer
+  end
+end

data/app/views/c80_push/shared/_page_dealers.html.erb

@@ -24,4 +24,4 @@
   var offices = <%= ymap_hash.to_json.html_safe %>;
 </script>
 
-<script src="https://api-maps.yandex.ru/2.1/?lang=ru_RU" type="text/javascript"></script>
+<script src="https://api-maps.yandex.ru/2.1/?load=package.standard&lang=ru-RU" type="text/javascript"></script>

data/config/locales/dealer/ru.yml

@@ -2,8 +2,9 @@ ru:
   c80_push:
     active_admin:
       hints:
-        site: 'Если сайтов несколько, перечислите их через запятую'
-        email: 'Если адресов несколько, перечислите их через запятую'
+        dealer:
+          site: 'Если сайтов несколько, перечислите их через запятую'
+          email: 'Если адресов несколько, перечислите их через запятую'
       menu:
         dealer: 'Список дилеров'
   activerecord:
@@ -19,4 +20,13 @@ ru:
         site: 'Сайт(ы)'
         email: 'Email адрес(а)'
         regions: 'Регион'
-        offices: 'Офисы'
+        presets: 'Цвет маркера на карте'
+        offices: 'Офисы'
+    errors:
+      models:
+        c80_push/dealer:
+          attributes:
+            ord:
+              blank: 'Не может быть пустым'
+            title:
+              blank: 'Не может быть пустым'

data/config/locales/office/ru.yml

@@ -2,9 +2,10 @@ ru:
   c80_push:
     active_admin:
       hints:
-        tel: 'Если номеров несколько, перечислите их через запятую'
-        site: 'Если сайтов несколько, перечислите их через запятую'
-        email: 'Если адресов несколько, перечислите их через запятую'
+        office:
+          tel: 'Если номеров несколько, перечислите их через запятую'
+          site: 'Если сайтов несколько, перечислите их через запятую'
+          email: 'Если адресов несколько, перечислите их через запятую'
   activerecord:
     models:
       office:
@@ -20,3 +21,18 @@ ru:
         site: 'Сайт(ы)'
         email: 'Email адрес(а)'
         regions: 'Регион'
+    errors:
+      models:
+        c80_push/office:
+          attributes:
+            ord:
+              blank: 'Не может быть пустым'
+            title:
+              blank: 'Не может быть пустым'
+            addr:
+              blank: 'Не может быть пустым'
+            gps:
+              blank: 'Не может быть пустым'
+              invalid: 'Неверный формат. Пример правильного значения: 12.1234,12.1234'
+            tel:
+              blank: 'Не может быть пустым'

data/db/migrate/20170630061313_create_c80_push_presets.rb

@@ -0,0 +1,9 @@
+class CreateC80PushPresets < ActiveRecord::Migration[5.0]
+  def change
+    create_table :c80_push_presets do |t|
+      t.string :title, null: false, limit: 90
+      t.string :color, null: false, limit: 10
+      t.timestamps null: false
+    end
+  end
+end

data/db/migrate/20170630061515_create_c80_push_join_table_dealers_presets.rb

@@ -0,0 +1,12 @@
+class CreateC80PushJoinTableDealersPresets < ActiveRecord::Migration[5.0]
+  def change
+    create_table :c80_push_dealers_presets, :id => false do |t|
+      t.integer :dealer_id, :null => false
+      t.integer :preset_id, :null => false
+    end
+
+    add_index :c80_push_dealers_presets, [:dealer_id, :preset_id], name: 'too_long_push11'
+    add_index :c80_push_dealers_presets, [:preset_id, :dealer_id], name: 'too_long_push12'
+
+  end
+end

data/lib/c80_push/version.rb

@@ -1,3 +1,3 @@
 module C80Push
-  VERSION = '0.1.0.2'
+  VERSION = '0.1.0.3'
 end

data/lib/generators/c80_push/install_generator.rb

@@ -0,0 +1,20 @@
+require 'rails/generators/base'
+require 'rails/generators/resource_helpers'
+
+module C80Push
+  module Generators
+    class InstallGenerator < Rails::Generators::Base
+      source_root File.expand_path('../templates', __FILE__)
+
+      def create_root_folder
+      #   empty_directory 'app/helpers/c80_yax'
+      end
+
+      def copy_template_file
+        template 'seeds/c80_push_02_fill_presets.rb',
+                 File.join('db/seeds', 'c80_push_02_fill_presets.rb')
+      end
+
+    end
+  end
+end

data/lib/generators/c80_push/templates/seeds/c80_push_02_fill_presets.rb

@@ -0,0 +1,21 @@
+# rake db:seed:c80_push_02_fill_presets
+
+C80Push::Preset.delete_all
+C80Push::Preset.create!([
+                            {title:'blueDotIcon', color:'#1E98FF'},
+                            {title:'redDotIcon', color:'#ED4543'},
+                            {title:'darkOrangeDotIcon', color:'#E6761B'},
+                            {title:'nightDotIcon', color:'#0E4779'},
+                            {title:'darkBlueDotIcon', color:'#177BC9'},
+                            {title:'pinkDotIcon', color:'#F371D1'},
+                            {title:'grayDotIcon', color:'#B3B3B3'},
+                            {title:'brownDotIcon', color:'#793D0E'},
+                            {title:'darkGreenDotIcon', color:'#1BAD03'},
+                            {title:'violetDotIcon', color:'#B51EFF'},
+                            {title:'blackDotIcon', color:'#595959'},
+                            {title:'yellowDotIcon', color:'#FFD21E'},
+                            {title:'greenDotIcon', color:'#56DB40'},
+                            {title:'orangeDotIcon', color:'#FF931E'},
+                            {title:'lightBlueDotIcon', color:'#82CDFF'},
+                            {title:'oliveDotIcon', color:'#97A100'}
+                        ])

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: c80_push
 version: !ruby/object:Gem::Version
-  version: 0.1.0.2
+  version: 0.1.0.3
 platform: ruby
 authors:
 - C80609A
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-06-28 00:00:00.000000000 Z
+date: 2017-06-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -24,6 +24,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: activeadmin
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 description: Добавляет функционал дилеров и офисов продаж
 email:
 - c080609a@gmail.com
@@ -49,6 +63,7 @@ files:
 - app/assets/javascripts/c80_push/lib_backend/init_select_picker.js
 - app/assets/javascripts/c80_push_backend.js.coffee
 - app/assets/stylesheets/c80_push.scss
+- app/assets/stylesheets/c80_push/backend/dealer_preset_span.scss
 - app/assets/stylesheets/c80_push/c80_push_page_dealers/_layout.scss
 - app/assets/stylesheets/c80_push/c80_push_page_dealers/_style.scss
 - app/assets/stylesheets/c80_push/c80_push_page_dealers/map_dealers_list.scss
@@ -59,6 +74,7 @@ files:
 - app/models/c80_push/application_record.rb
 - app/models/c80_push/dealer.rb
 - app/models/c80_push/office.rb
+- app/models/c80_push/preset.rb
 - app/models/c80_push/region.rb
 - app/views/c80_push/shared/_page_dealers.html.erb
 - app/views/c80_push/shared/_select_region.html.erb
@@ -72,9 +88,13 @@ files:
 - db/migrate/20170626125858_create_c80_push_regions.rb
 - db/migrate/20170626130000_create_c80_push_join_table_dealers_regions.rb
 - db/migrate/20170626130303_create_c80_push_join_table_offices_regions.rb
+- db/migrate/20170630061313_create_c80_push_presets.rb
+- db/migrate/20170630061515_create_c80_push_join_table_dealers_presets.rb
 - lib/c80_push.rb
 - lib/c80_push/engine.rb
 - lib/c80_push/version.rb
+- lib/generators/c80_push/install_generator.rb
+- lib/generators/c80_push/templates/seeds/c80_push_02_fill_presets.rb
 - lib/tasks/c80_push_tasks.rake
 homepage: https://github.com/c80609a/c80_push
 licenses: