Fix backwards show_order logic

ElixirTeSS · Oct 25, 2024 · 6eed8ca · 6eed8ca
1 parent e08f87a
commit 6eed8ca
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/app/views/learning_paths/show.html.erb b/app/views/learning_paths/show.html.erb
@@ -80,7 +80,8 @@
 
               <ul style="padding-left: 0; overflow: auto;">
                 <% lpt.topic.material_items.each do |object| %>
-                  <%= render partial: object.to_partial_path, locals: { learning_path_topic_item: object,
+                  <%= render partial: object.to_partial_path, locals: { show_order: false,
+                                                                        learning_path_topic_item: object,
                                                                         topic_link: lpt } %>
                 <% end %>
               </ul>

diff --git a/app/views/materials/_material.html.erb b/app/views/materials/_material.html.erb
@@ -1,7 +1,11 @@
 <% learning_path_topic_item ||= nil %>
 <% collection_item ||= learning_path_topic_item %>
 <% link_params = (defined? topic_link) ? learning_path_breadcrumb_param(topic_link, collection_item) : {} %>
-<% show_order = false unless defined? show_order %>
+<%
+  unless defined? show_order
+    show_order = true
+  end
+%>
 <li class="masonry-brick media-item long">
   <%= link_to material_path(material, **link_params), class: 'link-overlay' do %>
     <%= item_order_badge(collection_item) if show_order && collection_item %>