feat(nested-select): adjust to work with association name

Author: difernandez

app/inputs/nested_level_input.rb

@@ -1,4 +1,4 @@
-class NestedLevelInput < ActiveAdminAddons::InputBase
+class NestedLevelInput < ActiveAdminAddons::SelectInputBase
   include ActiveAdminAddons::SelectHelpers
 
   def render_custom_input
@@ -38,15 +38,17 @@ def load_parent_data_options
     return unless @options[:parent_attribute]
 
     load_data_attr(:parent, value: @options[:parent_attribute])
-    load_data_attr(:parent_id, value: @object.send(@options[:parent_attribute]), default: -1)
+    load_data_attr(
+      :parent_id, value: @object.send(@options[:parent_id_attribute]), default: -1
+    )
   end
 
   def load_collection_data
     return unless @options[:collection]
 
     collection_options = collection_to_select_options do |item, option|
-      if !!@options[:parent_attribute]
-        option[@options[:parent_attribute]] = item.send(@options[:parent_attribute])
+      if @options[:parent_attribute].present?
+        option[@options[:parent_id_attribute]] = item.send(@options[:parent_id_attribute])
       end
     end
 

app/inputs/nested_select_input.rb

@@ -59,32 +59,38 @@ def set_parent_attributes(hierarchy)
       parent_level_data = hierarchy[next_idx] if hierarchy.count != next_idx
       if parent_level_data
         level_data[:parent_attribute] = parent_level_data[:attribute]
+        level_data[:parent_id_attribute] = "#{level_data[:parent_attribute]}_id"
         set_parent_value(level_data)
       end
     end
   end
 
   def set_parent_value(level_data)
-    parent_attribute = level_data[:parent_attribute]
-    build_virtual_attr(parent_attribute)
-    instance = instance_from_attribute_name(level_data[:attribute])
-    if instance&.respond_to?(parent_attribute)
-      @object.send("#{parent_attribute}=", instance.send(parent_attribute))
+    build_virtual_attr(level_data[:parent_attribute])
+    build_virtual_attr(level_data[:parent_id_attribute])
+    instance = instance_from_attribute_id("#{level_data[:attribute]}_id")
+    if instance.respond_to?(level_data[:parent_id_attribute])
+      @object.send(
+        "#{level_data[:parent_attribute]}=", instance.send(level_data[:parent_attribute])
+      )
+      @object.send(
+        "#{level_data[:parent_id_attribute]}=", instance.send(level_data[:parent_id_attribute])
+      )
     end
   end
 
-  def instance_from_attribute_name(attribute)
-    return unless attribute
+  def instance_from_attribute_id(attribute_id)
+    return unless attribute_id
 
-    attribute_value = @object.send(attribute)
-    return unless attribute_value
+    attribute_id_value = @object.send(attribute_id)
+    return unless attribute_id_value
 
-    klass = class_from_attribute(attribute)
-    klass.find_by(id: attribute_value)
+    klass = class_from_attribute_id(attribute_id)
+    klass.find_by(id: attribute_id_value)
   end
 
-  def class_from_attribute(attribute)
-    association_name = attribute.to_s.chomp("_id")
+  def class_from_attribute_id(attribute_id)
+    association_name = attribute_id.to_s.chomp("_id")
     association_name.camelize.constantize
   rescue NameError
     object_class.reflect_on_association(association_name).klass

app/javascript/activeadmin_addons/inputs/slim-select-nested.js

@@ -12,7 +12,7 @@ function ajaxSearch(search, currentData, args) {
   });
 
   if (!!args.parent) {
-    args.query.q[`${args.parent}_eq`] = args.parentId;
+    args.query.q[`${args.parent}_id_eq`] = args.parentId;
   }
 
   args.query.q = { ...args.query.q, ...args.filters };
@@ -48,7 +48,7 @@ function settings(el) {
         const { model, association } = el.dataset;
         const child = el.closest('.nested_level')
           .parentNode
-          .querySelector(`.nested_level [data-model=${model}][data-parent=${association}_id]`);
+          .querySelector(`.nested_level [data-model=${model}][data-parent=${association}]`);
 
         if (child) {
           child.slim.setSelected();