KG - Add 'Sort By' to Protocol Filter

app/controllers/dashboard/protocols_controller.rb

@@ -32,7 +32,7 @@ def index
     admin_orgs   = @user.authorized_admin_organizations
     @admin       = !admin_orgs.empty?
 
-    default_filter_params = { show_archived: 0 }
+    default_filter_params = { show_archived: 0, sorted_by: 'id_asc' }
 
     # if we are an admin we want to default to admin organizations
     if @admin
@@ -49,7 +49,8 @@ def index
         default_filter_params: default_filter_params,
         select_options: {
           with_status: AVAILABLE_STATUSES.invert,
-          with_organization: Dashboard::GroupedOrganizations.new(@organizations).collect_grouped_options
+          with_organization: Dashboard::GroupedOrganizations.new(@organizations).collect_grouped_options,
+          sorted_by: Protocol.options_for_sorted_by
         },
         persistence_id: false #resets filters on page reload
       ) || return

app/models/protocol.rb

@@ -127,13 +127,14 @@ class Protocol < ActiveRecord::Base
   }
 
   filterrific(
-    default_filter_params: { show_archived: 0 },
+    default_filter_params: { show_archived: 0, sorted_by: 'asc' },
     available_filters: [
       :search_query,
       :admin_filter,
       :show_archived,
       :with_status,
-      :with_organization
+      :with_organization,
+      :sorted_by
     ]
   )
 
@@ -203,6 +204,34 @@ class Protocol < ActiveRecord::Base
     where(sub_service_requests: { organization_id: org_id }).distinct
   }
 
+  scope :sorted_by, -> (key) {
+    case key
+    when 'id_asc'
+      order(id: :asc)
+    when 'id_desc'
+      order(id: :desc)
+    when 'short_title_asc'
+      order('TRIM(REPLACE(short_title, CHAR(9), " ")) ASC')
+    when 'short_title_desc'
+      order('TRIM(REPLACE(short_title, CHAR(9), " ")) DESC')
+    when 'pi_asc'
+      joins(project_roles: :identity).where(project_roles: { role: 'primary-pi' }).order('.identities.first_name ASC')
+    when 'pi_desc'
+      joins(project_roles: :identity).where(project_roles: { role: 'primary-pi' }).order('.identities.first_name DESC')
+    end
+  }
+
+  def self.options_for_sorted_by
+    [
+      ['ID (0-9)', 'id_asc'],
+      ['ID (9-0)', 'id_desc'],
+      ['Short Title (A-Z)', 'short_title_asc'],
+      ['Short Title (Z-A)', 'short_title_desc'],
+      ['PI (A-Z)', 'pi_asc'],
+      ['PI (Z-A)', 'pi_desc']
+    ]
+  end
+
   def is_study?
     self.type == 'Study'
   end

app/models/protocol_filter.rb

@@ -33,6 +33,7 @@ class ProtocolFilter < ActiveRecord::Base
   attr_accessible :search_query
   attr_accessible :with_organization
   attr_accessible :with_status
+  attr_accessible :sorted_by
 
   scope :latest_for_user, -> (identity_id, limit) {
     where(identity_id: identity_id).
@@ -48,7 +49,8 @@ def href
         admin_filter: self.admin_filter,
         search_query: self.search_query,
         with_organization: self.with_organization,
-        with_status: self.with_status
+        with_status: self.with_status,
+        sorted_by: self.sorted_by
       }
     )
   end

app/views/dashboard/protocol_filters/_filter_protocols_form.html.haml

@@ -51,6 +51,10 @@
           = form.label :with_organization, t(:dashboard)[:protocol_filters][:organization], class: 'col-lg-3 control-label'
           .col-lg-9.drop-underneath
             = form.select :with_organization, filterrific.select_options[:with_organization], {}, class: "form-control selectpicker core-select", multiple: true, data: { none_selected_text: t(:constants)[:prompts][:any] }
+        .form-group.row
+          = form.label :sorted_by, t(:dashboard)[:protocol_filters][:sort_by], class: 'col-lg-3 control-label'
+          .col-lg-9
+            = form.select :sorted_by, filterrific.select_options[:sorted_by], {}, class: 'form-control selectpicker sort-select'
         - if admin
           %hr
           .form-group.row.identity-protocols

config/locales/en.yml

@@ -419,6 +419,7 @@ en:
       my_protocols: "My Protocols"
       my_admin_protocols: "My Admin Protocols"
       organization: "Organization"
+      sort_by: "Sort By"
       filter: "Filter"
       saved_filters:
         header: "Recently Saved Filters"

db/migrate/20160720151213_add_sorted_by_to_protocol_filter.rb

@@ -0,0 +1,5 @@
+class AddSortedByToProtocolFilter < ActiveRecord::Migration
+  def change
+    add_column :protocol_filters, :sorted_by, :string
+  end
+end

db/schema.rb

@@ -11,7 +11,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 20160719150112) do
+ActiveRecord::Schema.define(version: 20160720151213) do
 
   create_table "admin_rates", force: :cascade do |t|
     t.integer  "line_item_id", limit: 4
@@ -184,6 +184,12 @@
   add_index "charges", ["service_id"], name: "index_charges_on_service_id", using: :btree
   add_index "charges", ["service_request_id"], name: "index_charges_on_service_request_id", using: :btree
 
+  create_table "click_counters", force: :cascade do |t|
+    t.integer  "click_count", limit: 4
+    t.datetime "created_at",            null: false
+    t.datetime "updated_at",            null: false
+  end
+
   create_table "clinical_providers", force: :cascade do |t|
     t.integer  "identity_id",     limit: 4
     t.integer  "organization_id", limit: 4
@@ -657,6 +663,7 @@
     t.datetime "created_at"
     t.datetime "updated_at"
     t.string   "admin_filter",      limit: 255
+    t.string   "sorted_by",         limit: 255
   end
 
   create_table "protocols", force: :cascade do |t|

spec/views/dashboard/protocols/index.html.haml_spec.rb

@@ -8,13 +8,15 @@
     assign(:filterrific, double('filterrific',
       select_options: {
         with_status: [],
-        with_organization: []
+        with_organization: [],
+        sorted_by: "id_asc"
       },
       with_status: [],
       search_query: '',
       show_archived: 0,
       admin_filter: "for_identity #{jug2.id}",
-      with_organization: false
+      with_organization: false,
+      sorted_by: "id_asc"
     ))
   end