fix: avoid smart feature to be load more one time (#706)

Author: nicolasalexandre9

lib/forest_liana/collection.rb

@@ -24,11 +24,15 @@ def collection(collection_name, opts = {})
     end
 
     def action(name, opts = {})
+      return if model.actions.find { |action| action.name == name }
+
       opts[:name] = name
       model.actions << ForestLiana::Model::Action.new(opts)
     end
 
     def segment(name, opts = {}, &block)
+      return if model.segments.find { |segment| segment.name == name }
+
       opts[:name] = name
       model.segments << ForestLiana::Model::Segment.new(opts, &block)
     end
@@ -74,12 +78,14 @@ def field(name, opts, &block)
       opts[:validations] = [] unless opts.has_key?(:validations)
       opts[:is_virtual] = true unless opts.has_key?(:polymorphic_key) && opts[:polymorphic_key]
 
-      model.fields << opts.merge({
+      field = opts.merge({
         field: name,
         is_filterable: !!opts[:is_filterable],
         is_sortable: !!opts[:is_sortable],
       })
 
+      add_field(field)
+
       define_method(name) { self.data[name] } if smart_collection?
 
       if serializer_name && ForestLiana::UserSpace.const_defined?(
@@ -108,13 +114,15 @@ def field(name, opts, &block)
     end
 
     def has_many(name, opts, &block)
-      model.fields << opts.merge({
+      field = opts.merge({
         field: name,
         is_virtual: true,
         is_searchable: false,
         type: ['String']
       })
 
+      add_field(field)
+
       define_method(name) { self.data[name] } if smart_collection?
 
       if serializer_name && ForestLiana::UserSpace.const_defined?(
@@ -126,13 +134,15 @@ def has_many(name, opts, &block)
     end
 
     def belongs_to(name, opts, &block)
-      model.fields << opts.merge({
+      field = opts.merge({
         field: name,
         is_virtual: true,
         is_searchable: false,
         type: 'String'
       })
 
+      add_field(field)
+
       define_method(name) { self.data[name] } if smart_collection?
 
       if serializer_name && ForestLiana::UserSpace.const_defined?(
@@ -145,6 +155,12 @@ def belongs_to(name, opts, &block)
 
     private
 
+    def add_field(field)
+      return if model.fields.find { |field| field[:field] == name }
+
+      model.fields << field
+    end
+
     def find_name(collection_name)
       # TODO: Remove once lianas prior to 2.0.0 are not supported anymore.
       model = ForestLiana.models.find { |collection| collection.try(:table_name) == collection_name.to_s }