From ec50882cb58b05686db93e562e346eeb4bb2feac Mon Sep 17 00:00:00 2001
From: Galen Charlton <gmc@esilibrary.com>
Date: Tue, 22 Mar 2016 13:50:29 -0400
Subject: [PATCH] LP#1549505: comment out 'Least Popular' catalog sort option

Support for sorting by ascending popularity remains in the
back end, however, since if one's configured badges such that
almost every record has a badge score, that sort order could
be useful for examining the long tail of the collection.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
---
 Open-ILS/src/templates/opac/parts/filtersort.tt2 | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/Open-ILS/src/templates/opac/parts/filtersort.tt2 b/Open-ILS/src/templates/opac/parts/filtersort.tt2
index 5bc8ed0485..29903524f8 100644
--- a/Open-ILS/src/templates/opac/parts/filtersort.tt2
+++ b/Open-ILS/src/templates/opac/parts/filtersort.tt2
@@ -16,7 +16,14 @@
     </optgroup>
     <optgroup label='[% l("Sort by Popularity") %]'>
         <option value='popularity'[% value == 'popularity' ? ' selected="selected"' : '' %]>[% l("Most Popular") %]</option>
+<!-- Sorting by least popular items first is probably not an expected
+     choice in production, but could be useful in cases where every
+     record has at least one badge value assigned and you want
+     to investigate the long tail.
+-->
+<!--
         <option value='popularity.descending'[% value == 'popularity.descending' ? ' selected="selected"' : '' %]>[% l("Least Popular") %]</option>
+-->
         <option value='poprel'[% value == 'poprel' ? ' selected="selected"' : '' %]>[% l("Popularity Adjusted Relevance") %]</option>
     </optgroup>
 </select>
-- 
2.11.0