Improve sorting of wiki pages

Sorting is completely broken. Also align the sorting UI with other pages.

plugins/wiki/app/controllers/pages_controller.rb

@@ -108,22 +108,23 @@ class PagesController < ApplicationController
 
   def all
     @pages = Page.non_redirected
-    @partial = params[:view] || 'recent_changes'
+    @partial = params[:view] || 'title_list'
 
     if params[:name]
-      @pages = @pages.where("title LIKE ?", "%#{params[:name]}%").limit(20).order('updated_at DESC')
+      @pages = @pages.where("title LIKE ?", "%#{params[:name]}%").limit(20)
       @partial = 'title_list'
+    end
+    if params[:sort]
+      sort = case params[:sort]
+               when "title"                then "title"
+               when "title_reverse"        then "title DESC"
+               when "last_updated"         then "updated_at DESC"
+               when "last_updated_reverse" then "updated_at"
+               end
     else
-      order = case @partial
-        when 'recent_changes' then
-          'updated_at DESC'
-        when 'site_map' then
-          'created_at DESC'
-        when 'title_list' then
-          'title DESC'
-              end
-      @pages.order(order)
+      sort = "title"
     end
+    @pages = @pages.order(sort)
     respond_to do |format|
       format.html
       format.rss { render :layout => false }

plugins/wiki/app/views/pages/_recent_changes.html.haml

@@ -1,8 +0,0 @@
-%table.table.table-striped
-  %thead
-    %tr
-      %th= t 'pages.title'
-      %th= t 'pages.last_updated'
-  %tbody
-    - for page in @pages
-      = render :partial => "page_list_item", :locals => {:page => page, :level => 0, :siteMap => 0}

plugins/wiki/app/views/pages/_site_map.html.haml

@@ -8,5 +8,5 @@
       = render :partial => 'page_list_item', :locals => {:page => homepage, :level => 0, :siteMap => 1}
   %tbody
     - for page in @pages
-      - if page.id != homepage.try(:id)
+      - if page.id != homepage.try(:id) && !page.parent_id?
         = render :partial => 'page_list_item', :locals => {:page => page, :level => 0, :siteMap => 1}

plugins/wiki/app/views/pages/_title_list.html.haml

@@ -1,8 +1,8 @@
 %table.table.table-striped
   %thead
     %tr
-      %th= t 'pages.title'
-      %th= t 'pages.last_updated'
+      %th= sort_link_helper t('pages.title'), "title", :remote => false
+      %th= sort_link_helper t('pages.last_updated'), "last_updated", :remote => false
   %tbody
     - for page in @pages
       = render :partial => "page_list_item", :locals => {:page => page, :level => 0, :siteMap => 0}

plugins/wiki/app/views/pages/all.html.haml

@@ -6,7 +6,6 @@
 .navbar
   .navbar-inner
     %ul.nav
-      %li= link_to t('.recent_changes'), all_pages_path(:view => 'recent_changes')
       %li= link_to t('.title_list'), all_pages_path(:view => 'title_list')
       %li= link_to t('.site_map'), all_pages_path(:view => 'site_map')
       %li= link_to image_tag('icons/feed-icon-14x14.png', :alt => 'RSS Feed'), all_pages_rss_url