Bug Fix: Change default job search sort to start date (primary key), for performance.

Author: jhuckaby

htdocs/js/pages/Search.class.js

@@ -235,6 +235,9 @@ Page.Search = class Search extends Page.PageUtils {
 				// sort
 				html += '<div class="form_cell">';
 					var sort_items = [
+						{ id: 'id_desc', title: 'Newest on Top', icon: 'sort-descending', group: 'Start Date:' },
+						{ id: 'id_asc', title: 'Oldest on Top', icon: 'sort-ascending' },
+						
 						{ id: 'date_desc', title: 'Newest on Top', icon: 'sort-descending', group: 'Completion Date:' },
 						{ id: 'date_asc', title: 'Oldest on Top', icon: 'sort-ascending' },
 
@@ -370,7 +373,7 @@ Page.Search = class Search extends Page.PageUtils {
 		}
 
 		var sort = this.div.find('#fe_s_sort').val();
-		if (sort != 'date_desc') args.sort = sort;
+		if (sort != 'id_desc') args.sort = sort;
 
 		if (!num_keys(args)) return null;
 
@@ -442,6 +445,16 @@ Page.Search = class Search extends Page.PageUtils {
 			compact: 1
 		};
 		switch (args.sort) {
+			case 'id_asc':
+				sopts.sort_by = '_id'; 
+				sopts.sort_dir = 1;
+			break;
+			
+			case 'id_desc':
+				sopts.sort_by = '_id'; 
+				sopts.sort_dir = -1;
+			break;
+			
 			case 'date_asc':
 				sopts.sort_by = 'completed'; 
 				sopts.sort_dir = 1;
@@ -720,6 +733,16 @@ Page.Search = class Search extends Page.PageUtils {
 		var title = 'Bulk Export Jobs';
 
 		switch (args.sort) {
+			case 'id_asc':
+				sopts.sort_by = '_id'; 
+				sopts.sort_dir = 1;
+			break;
+			
+			case 'id_desc':
+				sopts.sort_by = '_id'; 
+				sopts.sort_dir = -1;
+			break;
+			
 			case 'date_asc':
 				sopts.sort_by = 'completed'; 
 				sopts.sort_dir = 1;

htdocs/js/pages/ServerUtils.class.js

@@ -2219,8 +2219,6 @@ Page.ServerUtils = class ServerUtils extends Page.PageUtils {
 
 		args.offset = args.offset || 0;
 		args.limit = config.alt_items_per_page;
-		args.sort_by = 'completed'; 
-		args.sort_dir = -1;
 
 		// add epoch date range (don't use .. range shorthand because end is inclusive)
 		args.query += ' date:>=' + this.epochStart + ' date:<' + this.epochEnd;

lib/api/search.js

@@ -24,7 +24,7 @@ class Search {
 		params.offset = parseInt( params.offset || 0 );
 		params.limit = parseInt( params.limit || 1 );
 
-		if (!params.sort_by) params.sort_by = 'completed';
+		if (!params.sort_by) params.sort_by = '_id';
 		if (!params.sort_dir) params.sort_dir = -1;
 
 		this.loadSession(args, function (err, session, user) {