fix(nested-level-input): include only items that belong to parent

When using collection option

Author: difernandez

app/inputs/nested_level_input.rb

@@ -48,7 +48,7 @@ def load_collection_data
 
     collection_options = collection_to_select_options do |item, option|
       if @options[:parent_attribute].present?
-        option[@options[:parent_id_attribute]] = item.send(@options[:parent_id_attribute])
+        option[@options[:parent_attribute]] = item.send(@options[:parent_id_attribute])
       end
     end
 

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

@@ -25,10 +25,12 @@ function collectionSearch(search, args, collection) {
     return Promise.reject('Search term too short');
   }
 
-  const data = JSON.parse(collection).map(
+  const data = JSON.parse(collection).filter(
+    (item) => (!args.parent || item[args.parent] === Number(args.parentId)) &&
+      String(item.text).toLowerCase().includes(search.toLowerCase()),
+  ).map(
     (item) => ({ value: String(item.id), text: item.text }),
-  ).filter(
-    (item) => String(item.text).toLowerCase().includes(search.toLowerCase()));
+  );
 
   return Promise.resolve(data);
 }

spec/features/inputs/nested_select_input_spec.rb

@@ -99,33 +99,48 @@
       end
     end
 
-    context "updating medium level" do
-      before do
-        on_input_ctx("invoice_region") do
-          open_slimselect_options
-          slimselect_search_input.set("Antof")
+    context "when updating medium level (that uses collection option)" do
+      context 'when searching for region that does not belong to selected country' do
+        before do
+          on_input_ctx("invoice_region") do
+            open_slimselect_options
+            slimselect_search_input.set("Cuy")
+          end
         end
-      end
 
-      it "shows results based on entered text", js: true do
-        expect_slimselect_result_text_to_eq(1, "Antofagasta")
+        it "does not show results", js: true do
+          expect_slimselect_no_result
+        end
       end
 
-      context "after click option" do
+      context 'when searching for region that belongs to selected country' do
         before do
-          on_input_ctx("invoice_region") { click_slimselect_option("Antofagasta") }
+          on_input_ctx("invoice_region") do
+            open_slimselect_options
+            slimselect_search_input.set("Antof")
+          end
         end
 
-        it "sets value", js: true do
-          on_input_ctx("invoice_region") { expect_slimselect_selection("Antofagasta") }
+        it "shows results based on entered text", js: true do
+          expect_slimselect_result_text_to_eq(1, "Antofagasta")
         end
 
-        it "preserves parent value", js: true do
-          on_input_ctx("invoice_country") { expect_slimselect_selection("Chile") }
-        end
+        context "when clicking option" do
+          before do
+            on_input_ctx("invoice_region") { click_slimselect_option("Antofagasta") }
+          end
 
-        it "resets children values", js: true do
-          on_input_ctx("invoice_city") { expect_slimselect_empty_selection }
+          it "sets value", js: true do
+            on_input_ctx("invoice_region") { expect_slimselect_selection("Antofagasta") }
+          end
+
+          it "preserves parent value", js: true do
+            on_input_ctx("invoice_country") { expect_slimselect_selection("Chile") }
+          end
+
+          it "resets children values", js: true do
+            on_input_ctx("invoice_city") { expect_slimselect_empty_selection }
+          end
         end
       end
     end