c80_estate 0.1.0.33 → 0.1.0.34

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 54f3f54111b8652b15917d98d7d370854164d342
-  data.tar.gz: e286d30720313421c5ac97e5c13895f199b950af
+  metadata.gz: 832f4348a9023325a7bbb127efb6c10234eb27b8
+  data.tar.gz: a716de7341084ea34df0bf2d2b71f7e70c6fd723
 SHA512:
-  metadata.gz: 4235806db10ac13f87f3e8969819a9391e30667c0c76d738ffa1e4581282e89ae52040a2bf775d724aa19918ae33a84e37356e2bfaf4b6293d7c8cd87610252c
-  data.tar.gz: 5b9fc34a08e000fe7bfbc0b1ad0d4b10553110119c0e17ba64f1c1785ad018aaa1c245999bbf58754aa72c6eaf48af2c4176f3ecc62e4d4c33a76e1dcd75372c
+  metadata.gz: 273b0fd67f31a3b0d1fa0f5e8ce3ccab77996ceb10546154149cf31fa9ceb27677c8b2c5fd262bc3494dee098a1527309e3c637df71c645492e0624d29e21667
+  data.tar.gz: 8b3c499e0e165c338bde6113941723e4477a934cabb5be9b28035dd2100ccd0d53876f3f3dde10aa81b8c025d8396ab37fefe59e18c4043a9377ce289d71bbb5

data/README.md

@@ -43,6 +43,7 @@ Or install it yourself as:
   gem 'friendly_id'
   gem 'babosa'
   gem 'bootstrap-select-rails'
+  gem 'c80_active_record_union'
 ```
 
 ### Seeds

data/app/admin/c80_estate/areas.rb

@@ -46,6 +46,29 @@ ActiveAdmin.register C80Estate::Area, as: 'Area' do
 
   config.clear_action_items!
 
+  controller do
+
+    def scoped_collection
+
+      # [X] 1. оптимизируем выдачу на index, согласно
+              # http://activeadmin.info/docs/2-resource-customization.html#customizing-resource-retrieval
+              # https://github.com/activeadmin/activeadmin/commit/f8ef8ca7f9ce6dc70a4761d6269477f0dff448ec
+              # http://www.rubydoc.info/gems/activeadmin/file/docs/2-resource-customization.md
+
+      # [ ]2. решаем задачу с "чужими незанятыми" и "чужими занятыми".
+              # * http://stackoverflow.com/a/23788089
+              #   способ реализации с дописанным кастомным where
+              #   super.where(account_id: account.id).order(:date_sent)
+
+      # включаем atypes и properties.logos: http://stackoverflow.com/a/24397716
+      C80Estate::Area
+          .includes(:astatuses, :atype, property: :plogos) ##[1]
+          .all_except_busy_alien(current_admin_user)
+
+    end
+
+  end
+
   action_item :new_model, :only => :index do
     if current_admin_user.can_create_areas?
       link_to I18n.t("active_admin.new_model"), '/admin/areas/new', method: :get
@@ -72,6 +95,8 @@ ActiveAdmin.register C80Estate::Area, as: 'Area' do
 
   config.sort_order = 'id_asc'
 
+  # before_filter :skip_sidebar!, :only => :index
+
   filter :atype_id,
          :label => 'Тип площади',
          :as => :select,
@@ -141,14 +166,15 @@ ActiveAdmin.register C80Estate::Area, as: 'Area' do
   scope "All", :all_areas
   scope "Free", :free_areas
   scope "Busy", :busy_areas
+  # scope 'My', :my_areas
 
   index do
-    selectable_column
+    # selectable_column
     column :title do |area|
       link_to area.title, "/admin/areas/#{area.id}", title: I18n.t("active_admin.view")
     end
     column :atype do |area|
-      area.atype_title
+      area.atype.title
     end
     column '<abbr title="За м.кв. в месяц">Цена м.кв.</abbr>'.html_safe do |area|
       "#{area.price_value.to_s(:rounded, :precision => 2)} руб"
@@ -168,8 +194,7 @@ ActiveAdmin.register C80Estate::Area, as: 'Area' do
     column :property do |area|
       "<div class='image_vertical properties_index_logo'>
       <span></span><a href='/admin/areas?utf8=✓&q%5Bproperty_id_eq%5D=#{area.property.id}&commit=Фильтровать&order=id_asc'><img src='#{image_path(area.property.logo_path)}'>
-      </div><span class='properties_index_logo_title'>#{area.property_title}</span></a>".html_safe
-
+      </div><span class='properties_index_logo_title'>#{area.property.title}</span></a>".html_safe
     end
     column :astatuses do |area|
       "<span class='status_#{area.astatus_tag}'>#{area.astatus_title}</span>".html_safe
@@ -177,10 +202,6 @@ ActiveAdmin.register C80Estate::Area, as: 'Area' do
     column :assigned_person do |area|
       area.property.assigned_person_title
     end
-    # actions
-    # column '' do |area|
-    #   link_to I18n.t("active_admin.view"), "/admin/areas/#{area.id}", class: 'member_link'
-    # end
     column '' do |area|
       if current_admin_user.can_edit_area?(area)
         link_to I18n.t("active_admin.edit"), "/admin/areas/#{area.id}/edit", class: 'member_link'

data/app/models/c80_estate/area.rb

@@ -42,6 +42,13 @@ module C80Estate
       self.joins(:astatuses).where(:c80_estate_astatuses => {tag: 'free'})
     end
 
+    # def self.my_areas
+    #   self.joins(:property)
+    #       .where(:c80_estate_properties => {assigned_person_id: current_admin_user.id})
+    # end
+
+    # scope :my_areas, lambda { |t| t.joins(:property).where(:c80_estate_properties => {assigned_person_id: current_admin_user.id}) }
+
     # посчитает кол-во свободных метров
     def self.free_areas_sq
       Rails.logger.debug "<Area.free_areas_sq>"
@@ -188,50 +195,99 @@ module C80Estate
       self.where(:atype_id => atype_id)
     end
 
+    # Не отображать "чужие" занятые площади (для менеждеров) (задача №1748)
+    def self.all_except_busy_alien(admin_user)
+      if admin_user.can_view_statistics?
+        # админам покажем всё
+        self.all
+      else
+
+        # http://stackoverflow.com/questions/9540801/combine-two-activerecordrelation-objects
+        # двумя независимыми запросами получим мои площади и немои свободные площади, сложим их, и отдадим
+
+=begin
+        # ВАРИАНТ 1
+
+        # извлечём немои свободные площади
+        # этот код я написал, смотря на уже существующий where_assigned_person_id
+        not_my_free_areas = self.free_areas
+                      .joins(:property)
+                      .where.not(:c80_estate_properties => {assigned_person_id: admin_user.id})
+
+        # извлечём все мои площади
+        all_my_areas = self.joins(:property)
+                           .where(:c80_estate_properties => {assigned_person_id: admin_user.id})
+
+        # это вернёт только то, что находится в ПЕРЕСЕЧЕНИИ результатов, а нужен union
+        not_my_free_areas.merge(all_my_areas)
+=end
+
+=begin
+        # ВАРИАНТ 2
+        # http://stackoverflow.com/a/28358592
+        # User.where(
+        #     User.arel_table[:first_name].eq('Tobias').or(
+        #         User.arel_table[:last_name].eq('Fünke')
+        #     )
+        # )
+=end
+
+=begin
+        # ВАРИАНТ 3
+        # http://stackoverflow.com/a/31528499
+        # first_name_relation = User.where(:first_name => 'Tobias') # ActiveRecord::Relation
+        # last_name_relation  = User.where(:last_name  => 'Fünke') # ActiveRecord::Relation
+        #
+        # all_name_relations = User.none
+        # first_name_relation.each do |ar|
+        #   all_name_relations.new(ar)
+        # end
+        # last_name_relation.each do |ar|
+        #   all_name_relations.new(ar)
+        # end
+=end
+
+        # попробуем 3-й вариант
+        not_my_free_areas = self.free_areas
+                                .joins(:property)
+                                .where.not(:c80_estate_properties => {assigned_person_id: admin_user.id})
+
+        all_my_areas = self.joins(:property)
+                           .where(:c80_estate_properties => {assigned_person_id: admin_user.id})
+
+        all_my_areas.union(not_my_free_areas)
+
+      end
+    end
+
     def has_astatus?
       errors.add_to_base 'Укажите статус площади' if self.astatuses.blank?
     end
 
+    # --------
+
     def atype_title
-      res = "-"
-      if atype.present?
-        res = atype.title
-      end
-      res
+      atype.title
     end
 
     def property_title
-      res = "-"
-      if property.present?
-        res = property.title
-      end
-      res
+      property.title
     end
 
     def astatus_title
-      res = "-"
-      if astatuses.count > 0
-        res = astatuses.first.title
-      end
-      res
+      astatuses.first.title
     end
 
     def astatus_id
-      res = -1
-      if astatuses.count > 0
-        res = astatuses.first.id
-      end
-      res
+      astatuses.first.id
     end
 
     def astatus_tag
-      res = -1
-      if astatuses.count > 0
-        res = astatuses.first.tag
-      end
-      res
+      astatuses.first.tag
     end
 
+    # --------
+
     def is_free?
       astatus_tag == 'free'
     end
@@ -261,6 +317,7 @@ module C80Estate
     end
 
     # выдать цену за м.кв. в месяц
+    # TODO_MY:: добавить модели Area столбец price_value и before_update метод, который высчитывал бы значение
     def price_value
 
       res = 0.0
@@ -298,6 +355,7 @@ module C80Estate
       res
     end
 
+    # TODO_MY:: добавить модели Area столбец square_value и before_update метод, который высчитывал бы значение
     def square_value
       res = 0.0
       p = item_props.where(:prop_name_id => 9)
@@ -308,6 +366,7 @@ module C80Estate
       res
     end
 
+    # TODO_MY:: добавить модели Area столбец power_price_value и before_update метод, который высчитывал бы значение
     def power_price_value
       price_value * 1.0 * square_value
     end

data/app/models/c80_estate/property.rb

@@ -166,6 +166,7 @@ module C80Estate
       res
     end
 
+    # TODO:: при построении индексной таблицы из 100 строк происходит 100 запросов к базе типа COUNT(*). Добавить before_update метод и поле logo_path и вычислять путь к лого в before update
     def logo_path
       url = 'property_default_logo.png'
       if plogos.count > 0

data/app/views/c80_estate/shared/properties/_single_property_on_page.html.erb

@@ -7,7 +7,7 @@
 <div class="clearfix">
   <div id="div_main_show_area" class="clearfix">
     <%= link_to image_tag('w227x182px.gif',
-                          :alt=>item_title,
+                          :alt => item_title,
                           :data => {:original => image_path(@item.main_image_url)},
                           :class => 'lazy'),
                 image_path(@item.main_image_url),
@@ -37,15 +37,17 @@
   <%= ph_render_tech_props(@item) %>
 </div>
 
-<div id="div_all_areas_table" class="wrap_collapse">
-  <button type="button" class="btn" data-toggle="collapse" data-target="#iid">
-    <span class="fa fa-chevron-right"></span>
-    Площади объекта <%= @item.title %> (<%= @item.areas.count %>)
-  </button>
-  <div class="collapse" id="iid">
-    <%= ph_render_areas_table(@item) %>
-  </div>
-</div>
+<% if current_admin_user.can_edit_property?(@item) %>
+    <div id="div_all_areas_table" class="wrap_collapse">
+      <button type="button" class="btn" data-toggle="collapse" data-target="#iid">
+        <span class="fa fa-chevron-right"></span>
+        Площади объекта <%= @item.title %> (<%= @item.areas.count %>)
+      </button>
+      <div class="collapse" id="iid">
+        <%= ph_render_areas_table(@item) %>
+      </div>
+    </div>
+<% end %>
 
 <div id="related_items" class="hidden">
   <%# smiph_render_related_items(@item) %>

data/lib/c80_estate/version.rb

@@ -1,3 +1,3 @@
 module C80Estate
-  VERSION = '0.1.0.33'
+  VERSION = '0.1.0.34'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: c80_estate
 version: !ruby/object:Gem::Version
-  version: 0.1.0.33
+  version: 0.1.0.34
 platform: ruby
 authors:
 - C80609A
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2016-08-08 00:00:00.000000000 Z
+date: 2016-08-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler