Task order is governed by nested_set instead of acts_as_list, so next…

…_higher becomes right_sibling
backlogs · Aug 25, 2010 · c088f1c · c088f1c
1 parent 43b3f65
commit c088f1c
Show file tree

Hide file tree

Showing 6 changed files with 6 additions and 2 deletions.
diff --git a/app/views/tasks/_impediment.html.erb b/app/views/tasks/_impediment.html.erb
diff --git a/app/views/tasks/_task.html.erb b/app/views/tasks/_task.html.erb
@@ -11,12 +11,12 @@
   <div class="meta">
     <div class="story_id"><%= task.parent_id %></div>
     <div class="status_id"><%= task.status_id %></div>
-    <div class="previous"><%= task.higher_item.nil? ? '' : task.higher_item.id %></div>
+    <div class="previous"><%= task.right_sibling.nil? ? '' : task.right_sibling.id %></div>
     <div class="errors">
       <%- task.errors.each_error do |a, err| %>
       <div><%= "#{l(a)} #{l(err.type)}" %></div>
       <%- end %>
     </div>
   </div>
   <%- end %>
-</div>
+</div>
diff --git a/features/step_definitions/common_steps.rb b/features/step_definitions/common_steps.rb
diff --git a/features/team_member.feature b/features/team_member.feature
diff --git a/features/team_member_steps.rb b/features/team_member_steps.rb
diff --git a/lib/backlogs_query_patch.rb b/lib/backlogs_query_patch.rb
@@ -21,6 +21,10 @@ def self.included(base) # :nodoc:
                                         # sprint name, in case start dates are the same
                                         "(select name from versions where versions.id = issues.fixed_version_id)",
 
+                                        # make sure stories with NULL
+                                        # position sort-last
+                                        "(select case when root.position is null then 1 else 0 end from issues root where issues.root_id = root.id)",
+
                                         # story position
                                         "(select root.position from issues root where issues.root_id = root.id)",