klastera 1.2.1 → 1.2.4.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3d1ff69c5abf0d989e26c1bfa2be9da2f8c9f5e8100bd4670f1c70d05c91cb13
-  data.tar.gz: 3bda3826a8d07270008bd4789ecc0eee72706984d9e3065f64bcc7475b844296
+  metadata.gz: 149afa24eba1225396134f74b791f7d1e960480af16f5785d7f1a76570eea5d3
+  data.tar.gz: e801acbf180a965cf9a922bfc0e7ad3006df21a780b7d9cbb279cad0c33a4f22
 SHA512:
-  metadata.gz: 39e4ec148db926bd2881d1e88e45b4c6071eccbd9aee45964e5c8d547d83f7513e6b1fc9882dfd6c1b06fe2bd146f0153aa935f4869809e09c554736ca8288e3
-  data.tar.gz: '09cbc4eb782c613ddfa68c4704dc6511ba77c49137539b2fc7d52c084e25a5ecfc28e52b38231081ddbea920468eac23aadca10449009196bd25a7d7c36e46d0'
+  metadata.gz: e43ab98889edd3d9ece42a1f496bfc614ee77ac4721344694adf1e084d05192bcc97a43e69be19247095d9b18c3131a5d86d123be9a69ef85544965af6bb419b
+  data.tar.gz: e994dfd1ae541212674faed964c526a38ee3ef14020af970893f54f04d054bf6b9abe36822cf5456ade0cb47a5570576c0cd790604b5d0e70ef71e41f12ae039

data/Rakefile

@@ -14,7 +14,7 @@ RDoc::Task.new(:rdoc) do |rdoc|
   rdoc.rdoc_files.include('lib/**/*.rb')
 end
 
-#APP_RAKEFILE = File.expand_path("../test/dummy/Rakefile", __FILE__)
+APP_RAKEFILE = File.expand_path("../test/dummy/Rakefile", __FILE__)
 load 'rails/tasks/engine.rake'
 
 

data/app/models/klastera/concerns/clusterizable.rb

@@ -9,9 +9,10 @@ module Klastera::Concerns::Clusterizable
 
     validates :cluster_id, presence: true, if: proc { self.try(:cluster_id) && self.organization.required_suborganization_mode? }
     validate :at_least_one_cluster_entity, if: proc { self.organization.required_suborganization_mode? }
+    validate :uniqueness_of_cluster_entity_record
 
     scope :related_clusters, ->() {
-      includes(:cluster_entities).where("cluster_entities.entity_id = #{self.table_name}.id")
+      joins(:cluster_entities).where("cluster_entities.entity_id = #{self.table_name}.id")
     }
 
     def at_least_one_cluster_entity
@@ -20,8 +21,15 @@ module Klastera::Concerns::Clusterizable
       end
     end
 
-    def clusters_string_separated_by(separator)
-      self.cluster_entities.map{|ce|ce.cluster.name}.join(separator)
+    ##
+     # This is a legacy method and we don't recommend using it.
+     # Implement directly Klastera.entity_clusters_string_list instead of this method.
+     # TODO: In order to deprecate it, you will need to perform some changes in the main app
+     ##
+    def clusters_string_separated_by(separator,attribute=:name)
+      Klastera.entity_clusters_string_list!(
+        self.cluster_entities, separator, attribute
+      )
     end
 
     def has_one_cluster_entity
@@ -29,6 +37,12 @@ module Klastera::Concerns::Clusterizable
         return errors.add(:cluster_entities, I18n.t('klastera.messages.has_one_cluster_entity'))
       end
     end
+
+    def uniqueness_of_cluster_entity_record
+      if cluster_entities.map(&:cluster_id).uniq.size != cluster_entities.size
+        return errors.add(:cluster_entities, I18n.t('klastera.messages.duplicated_cluster_entity') )
+      end
+    end
   end
   
   module ClassMethods

data/app/views/layouts/klastera/_cluster_entity_fields.html.erb

@@ -1,6 +1,6 @@
 <tr class="nested-fields">
   <td class="field">
-    <%= f.select :cluster_id, cluster_clusters.map{|c|[c.name,c.id]}, { include_blank: true }, { class: 'form-control' } %>
+    <%= f.select :cluster_id, @cluster_clusters.map{|c|[c.name,c.id]}, { include_blank: true }, { class: 'form-control' } %>
   </td>
   <td class="action vertical-align-middle text-center" width="44">
     <%= link_to_remove_association f, class: 'btn btn-danger btn-xs text-danger' do %>

data/app/views/layouts/klastera/_nested_cluster_entity.html.erb

@@ -1,4 +1,5 @@
 <% if cluster_organization.is_in_cluster_mode? %>
+  <% @cluster_clusters = cluster_clusters %>
   <div class="col-xs-12">
     <% if hide_title||=false %>
       <div class="form-group file required <%=f.object.class.name.parameterize%>_cluster_entity<%=' has-error' if f.object.errors.has_key?(:cluster_entities)%>">
@@ -26,6 +27,7 @@
           <table id="cluster-entities" class="table table-striped">
             <tbody class="cluster-entity-rows">
               <%= f.fields_for :cluster_entities do |cluster_entity|%>
+                <% next unless @cluster_clusters.map(&:id).include?(cluster_entity.try(:object).try(:cluster_id)) %>
                 <%= render 'layouts/klastera/cluster_entity_fields', f: cluster_entity %>
               <% end %>
             </body>

data/config/locales/es.yml

@@ -61,6 +61,7 @@ es:
       at_least_one_cluster_entity: Debe agregar al menos un cluster
       record_action_successfully: Registro %{a} exitosamente
       cant_delete_the_last_record_in_required_suborganization_mode: No se puede eliminar el único cluster de esta organización
+      duplicated_cluster_entity: Hay un cluster duplicado
       new_cluster_id:
         nil: Cluster no existe
         same: Cluster no puede ser el mismo

data/lib/klastera.rb

@@ -5,8 +5,36 @@ module Klastera
 
   extend ActiveSupport::Concern
 
+  KLSTR_HELPERS = %i[
+    cluster_user cluster_organization user_has_more_than_one_cluster
+    cluster_scope cluster_clusters cluster_scope_filtered clusters_from
+    user_clusters_string_list
+  ]
+
   class << self
 
+    def set_cluster_entities_attributes!(entity,array_cluster_ids)
+      cluster_entities_attributes = {}
+      entity_cluster_entities = entity.try(:cluster_entities) || []
+      entity_clusters = entity_cluster_entities.map(&:cluster_id).sort
+      array_cluster_ids = array_cluster_ids.map(&:id).sort
+      if array_cluster_ids != entity_clusters
+        now = DateTime.now
+        entity_cluster_entities.each_with_index do |ec,index|
+          remove_cluster = true
+          if array_cluster_ids.include?(ec.cluster_id)
+            remove_cluster = false
+            array_cluster_ids.delete(ec.cluster_id)
+          end
+          cluster_entities_attributes[index] = { id: ec.id, cluster_id: ec.cluster_id, _destroy: remove_cluster }
+        end
+        array_cluster_ids.each_with_index do |cluster_id,index|
+          cluster_entities_attributes[now.to_i+index] = { cluster_id: cluster_id, _destroy: false }
+        end
+      end
+      cluster_entities_attributes
+    end
+
     #
     # I like to wrap methods
     #
@@ -30,7 +58,7 @@ module Klastera
           # A force cluster organization won't return an entity out of a cluster, but
           # we don't know if the clusterized entities are fully definition-compliant.
           cluster_array << nil if cluster_organization.optional_suborganization_mode?
-          active_record_collection = active_record_collection.where("cluster_entities.cluster_id": cluster_array)
+          active_record_collection = active_record_collection.joins(:cluster_entities).where("cluster_entities.cluster_id": cluster_array)
         end
         # you may use a block only with clusterizable data
         yield(active_record_collection) if block_given?
@@ -81,12 +109,12 @@ module Klastera
           if organization.required_suborganization_mode?
             scope_klass = scope_klass.joins(:cluster_entities).where( cluster_entities: { cluster_id: cluster_ids } )
           else
+            or_these_cluster_ids = cluster_ids.present? ? " OR cluster_entities.cluster_id IN (#{cluster_ids.join(",")})" : ""
             scope_klass = scope_klass.joins("
               LEFT OUTER JOIN cluster_entities
-                ON cluster_entities.cluster_id IN (#{cluster_ids.join(",")})
-                AND entity_id = #{scope.table_name}.id 
+                ON entity_id = #{scope.table_name}.id
                 AND entity_type = '#{scope}'
-            ")
+            ").where("cluster_entities.id IS NULL#{or_these_cluster_ids}")
           end
           # Provisional fix to avoid SQL clashes due to DISTINCT ON clause
           scope_klass = scope_class(scope).where(organization_id: organization).where(id: scope_klass.map(&:id))
@@ -121,38 +149,42 @@ module Klastera
       end
       clusters
     end
-  end
 
-  included do
-    if respond_to?(:helper_method)
-      helper_method :cluster_scope
-      helper_method :user_cluster
-      helper_method :cluster_user
-      helper_method :cluster_organization
-      helper_method :cluster_clusters
-      helper_method :cluster_scope_filtered
-      helper_method :user_has_more_than_one_cluster
-      helper_method :clusters_from
+    ##
+     # Return a string with cluster attribute separated by separator argument
+     # A array of cluster ids can be passed fo filter the result
+     #
+    def entity_clusters_string_list!(cluster_entities,separator,attribute=:name,allowed_cluster_ids=nil)
+      _cluster_entities = cluster_entities.reject(&:nil?)
+      if allowed_cluster_ids.is_a?(Array)
+        _cluster_entities.select!{|ce| allowed_cluster_ids.include?(ce.cluster_id)}
+      end
+      _cluster_entities.map do |ce|
+        ce.cluster.try(attribute)
+      end.compact.join(separator)
     end
-    if respond_to?(:hide_action)
-      hide_action :cluster_scope
-      hide_action :cluster_scope=
-      hide_action :user_cluster
-      hide_action :user_cluster=
-      hide_action :cluster_user
-      hide_action :cluster_user=
-      hide_action :cluster_organization
-      hide_action :cluster_organization=
-      hide_action :cluster_clusters
-      hide_action :cluster_clusters=
-      hide_action :cluster_scope_filtered
-      hide_action :cluster_scope_filtered=
-      hide_action :user_has_more_than_one_cluster
-      hide_action :user_has_more_than_one_cluster=
-      hide_action :clusters_from
-      hide_action :clusters_from=
+
+    ##
+     # cluster_clusters needs the logged user and its organization
+     # that why, we perfomed this logic here
+     #
+    def user_clusters_string_list!(user,organization,cluster_entities,separator,attribute=:name)
+      @_session_clusters ||= self.session_clusters(user,organization)
+      self.entity_clusters_string_list!(cluster_entities, separator, attribute, @_session_clusters.map(&:id))
     end
-    before_action :set_the_lonely_cluster, only: %i[ create update ]
+  end
+
+  def cluster_user
+    current_user
+  end
+
+  def cluster_organization
+    current_organization
+  end
+
+  def user_has_more_than_one_cluster
+    @cluster_clusters ||= cluster_clusters
+    @cluster_clusters.size > 1
   end
 
   def set_the_lonely_cluster
@@ -164,16 +196,22 @@ module Klastera
     end
   end
 
-  def cluster_scope(scope,includes=[])
-    Klastera.cluster_scope!(cluster_user,cluster_organization,scope,includes)
+  def set_cluster_filter
+    @filter = ::ClusterFilter.new(cluster_filter_params)
   end
 
-  def cluster_user
-    current_user
+  def cluster_filter_params
+    parameters = params.require(:cluster_filter) rescue nil
+    return {} if parameters.blank?
+    parameters.permit(*cluster_filter_permit_params)
   end
 
-  def cluster_organization
-    current_organization
+  def cluster_filter_permit_params
+    [ :cluster_id ].concat( ::ClusterFilter.attributes )
+  end
+
+  def cluster_scope(scope,includes=[])
+    Klastera.cluster_scope!(cluster_user,cluster_organization,scope,includes)
   end
 
   def cluster_clusters
@@ -190,26 +228,30 @@ module Klastera
     )
   end
 
-  def user_has_more_than_one_cluster
-    @cluster_clusters ||= cluster_clusters
-    @cluster_clusters.size > 1
-  end
-
   def clusters_from(scope,includes=[])
     Klastera.clusters_from!(cluster_user,cluster_organization,scope,includes)
   end
 
-  def set_cluster_filter
-    @filter = ::ClusterFilter.new(cluster_filter_params)
-  end
-
-  def cluster_filter_params
-    parameters = params.require(:cluster_filter) rescue nil
-    return {} if parameters.blank?
-    parameters.permit(*cluster_filter_permit_params)
+  def user_clusters_string_list(object_entity,separator,attribute=:name)
+    Klastera.user_clusters_string_list!(
+      cluster_user,
+      cluster_organization,
+      object_entity.try(:cluster_entities),
+      separator,
+      attribute
+    )
   end
 
-  def cluster_filter_permit_params
-    [ :cluster_id ].concat( ::ClusterFilter.attributes )
+  included do
+    Klastera::KLSTR_HELPERS.each do |action|
+      if respond_to?(:helper_method)
+        helper_method(action)
+      end
+      if respond_to?(:hide_action)
+        hide_action(helper)
+        hide_action("#{helper}=")
+      end
+    end
+    before_action :set_the_lonely_cluster, only: %i[ create update ]
   end
 end

data/lib/klastera/version.rb

@@ -1,3 +1,3 @@
 module Klastera
-  VERSION = "1.2.1"
+  VERSION = "1.2.4.1"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: klastera
 version: !ruby/object:Gem::Version
-  version: 1.2.1
+  version: 1.2.4.1
 platform: ruby
 authors:
 - Gino Barahona
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-06-01 00:00:00.000000000 Z
+date: 2020-06-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -54,7 +54,6 @@ files:
 - app/controllers/klastera/application_controller.rb
 - app/controllers/klastera/clusters_controller.rb
 - app/helpers/klastera/application_helper.rb
-- app/helpers/klastera/clusters_helper.rb
 - app/models/klastera/cluster.rb
 - app/models/klastera/cluster_entity.rb
 - app/models/klastera/cluster_filter.rb
@@ -128,7 +127,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.8
+rubygems_version: 3.1.2
 signing_key: 
 specification_version: 4
 summary: Clusterization Engine

data/app/helpers/klastera/clusters_helper.rb

@@ -1,4 +0,0 @@
-module Klastera
-  module ClustersHelper
-  end
-end