Merge pull request #538 from alphagov/fix-accordion-functionality-on-…

…learning-to-drive-in-sidebar-navigation Fix "open step" on TaskList Sidebar
alphagov · Nov 15, 2017 · 0a65ef2 · 0a65ef2
2 parents 736b79f + 8874f9d
commit 0a65ef2
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/app/controllers/application_controller.rb b/app/controllers/application_controller.rb
@@ -31,14 +31,14 @@ def configure_current_task(config)
 
   def set_task_as_active_if_current_page(tasklist)
     counter = 0
-    tasklist[:steps].each do |grouped_steps|
+    tasklist[:groups].each do |grouped_steps|
       grouped_steps.each do |step|
         counter = counter + 1
 
         step[:panel_links].each do |link|
           if link[:href] == request.path
             link[:active] = true
-            tasklist[:open_section] = counter
+            tasklist[:open_step] = counter
             return tasklist
           end
         end

diff --git a/config/tasklists/learn-to-drive-a-car.json b/config/tasklists/learn-to-drive-a-car.json
@@ -12,7 +12,7 @@
   "tasklist": {
     "small": true,
     "heading_level": 3,
-    "steps": [
+    "groups": [
       [
         {
           "title": "Check you're allowed to drive",

diff --git a/test/unit/tasklist_content_test.rb b/test/unit/tasklist_content_test.rb
@@ -17,7 +17,7 @@ class TasklistContentTest < ActiveSupport::TestCase
   end
 
   test "have a link in the correct structure" do
-    first_link = @config[:tasklist][:steps][0][0][:panel_links][0]
+    first_link = @config[:tasklist][:groups][0][0][:panel_links][0]
     assert_equal "/vehicles-can-drive", first_link[:href]
     assert_equal "Check what age you can drive", first_link[:text]
   end