From 152d7ed9dce00ca6c0391080e9109f7cb54245af Mon Sep 17 00:00:00 2001
From: Sanj <b@pad.ma>
Date: Sat, 10 Sep 2011 07:10:47 +0530
Subject: [PATCH] sorting works, pagination works.

---
 itf/app/models.py                  |  15 ++--
 itf/insidepages/models.py          |  19 ++++
 itf/insidepages/urls.py            |   1 +
 itf/insidepages/views.py           |  22 ++++-
 itf/static/js/insidepage.js        | 134 ++++++++++++++++++++++++++---
 itf/templates/noel/insidepage.html |   6 +-
 6 files changed, 178 insertions(+), 19 deletions(-)

diff --git a/itf/app/models.py b/itf/app/models.py
index fbad0ae..09407e2 100644
--- a/itf/app/models.py
+++ b/itf/app/models.py
@@ -205,16 +205,16 @@ class ItfModel(models.Model):
   @classmethod
   def get_list(kls, data):
     options = {
-      'page_no': 1,
-      'list_size': 12,
+      'page': 1,
+      'count': 12,
       'search': '',
       'sort': [],
       'range': [0,50]
     }
     options.update(data)
     ret = []
-    page_no = options['page_no']
-    list_size = options['list_size']
+    page_no = options['page']
+    list_size = options['count']
     try:
       qset = kls.get_qset()
     except:
@@ -225,6 +225,7 @@ class ItfModel(models.Model):
       qset = kls.fts(qset, search) 
    
     sort = options['sort']
+
     if sort != []:
       for s in sort:
         if s['operator'] == '-':
@@ -249,7 +250,11 @@ class ItfModel(models.Model):
     for r in results.object_list:
       ret.append(r.list_dict())
 
-    return ret
+    return {
+        'page_no': page_no,
+        'num_pages': paginator.num_pages,
+        'items': ret
+    }
 
   @classmethod
   def get_qset(kls):
diff --git a/itf/insidepages/models.py b/itf/insidepages/models.py
index 4052206..8e438d1 100644
--- a/itf/insidepages/models.py
+++ b/itf/insidepages/models.py
@@ -32,9 +32,22 @@ class ModuleTab(models.Model):
     def model_class(self):
         return self.model.model.model_class()
 
+    def get_absolute_url(self): #FIXME!!
+        return self.slug
+
     def get_list(self, options):
         return self.model_class().get_list(options)
 
+    def get_dict(self):
+        return {
+            'id': self.id,
+            'slug': self.slug,
+            'title': self.title,
+            'text': self.text,
+            'has_list': self.model.has_list,
+            'page': self.get_list({}),
+            'sorts': [s.get_dict() for s in self.model.modelsort_set.all()]
+        }
 
 class ModelExtra(models.Model):
     model = models.ForeignKey(ContentType, help_text="Chose from the list of death at your own peril.")
@@ -57,6 +70,12 @@ class ModelSort(models.Model):
     def __unicode__(self):
         return self.friendly_name
 
+    def get_dict(self):
+        return {
+            'operator': self.operator,
+            'field_name': self.field_name,
+            'friendly_name': self.friendly_name
+        }
 
 class ModelButton(models.Model):
     name = models.CharField(max_length=64, help_text="Name of button, eg. Download")
diff --git a/itf/insidepages/urls.py b/itf/insidepages/urls.py
index 875c3b3..e6e8344 100644
--- a/itf/insidepages/urls.py
+++ b/itf/insidepages/urls.py
@@ -4,5 +4,6 @@ import views
 urlpatterns = patterns('',
     (r'^get_list$', views.get_list),
     (r'^get_details$', views.get_details),
+    (r'^get_tab$', views.get_tab),
     (r'^(?P<module_slug>.*)$', views.main),
 )
diff --git a/itf/insidepages/views.py b/itf/insidepages/views.py
index 0e51643..28a50cd 100644
--- a/itf/insidepages/views.py
+++ b/itf/insidepages/views.py
@@ -17,15 +17,33 @@ def main(request, module_slug):
         'tabs': tabs[1:]
     })
     return render_to_response("noel/insidepage.html", context)
+
+def get_tab(request):
+    tab_id = request.GET.get("tab_id", 0)
+    tab = get_object_or_404(ModuleTab, pk=tab_id)
+    return render_to_json_response(tab.get_dict())
             
     
 def get_list(request):
     tab_id = request.GET.get("tab_id", 0)
     tab = get_object_or_404(ModuleTab, pk=tab_id)
+
+    sortString = request.GET.get("sort", "")
+    if sortString == "":
+        sortArray = []
+    else:
+        sortOperator = sortString[0]
+        sortField = sortString[1:]
+        sortArray = [{
+            'operator': sortOperator,
+            'key': sortField
+        }]
+
     list_options = {
         'search': request.GET.get("search", ""),
-        'sort': request.GET.get("sort", ""),
-        'page': request.GET.get("page_no", 1)
+        'sort': sortArray, 
+        'page': request.GET.get("page", 1),
+#        'count': request.GET.get("count", 12) #FIXME: make list_length either in settings.py or config per model
     }
     object_list = tab.get_list(list_options)
     return render_to_json_response(object_list)
diff --git a/itf/static/js/insidepage.js b/itf/static/js/insidepage.js
index 4bea13f..1b215e9 100644
--- a/itf/static/js/insidepage.js
+++ b/itf/static/js/insidepage.js
@@ -19,6 +19,13 @@ $('#listLeft ul li a').live("click", function() {
     return false;
 });
 
+
+/* Get a jquery <option> row for a sort field, requires s.operator, s.field_name, s.friendly_name */
+function getOption(s) {
+    return $('<option />').val(s.operator + s.field_name).text(s.friendly_name);
+}
+
+/* Get a Jquery List Row object for an item - requires item.id and item.title*/
 function getLi(item) {
     var titleLength = 30;
     var $li = $('<li />').addClass("tabListItem");
@@ -29,32 +36,137 @@ function getLi(item) {
 }
 
 $(function() {
+    /* search button toggle */
+    $('.searchInnerField').hide();
+    $('#searchInner').toggle(function() {
+      $('.searchInnerField').slideDown();
+    }, function() {
+      $('.searchInnerField').slideUp();
+    });
+    /* search button end */
+
+
     $('#listLeft ul li a').eq(0).click();
 //    alert("hi");
+
+
     $('.tabCategory').click(function() {
         var $this = $(this);
         if ($this.hasClass('.innerSelected')) {
             return false;
         }
-
+        
         $('.innerSelected').removeClass("innerSelected");
         $this.addClass("innerSelected");
         $('.displayedTab').removeClass("displayedTab");
         var tabId = $this.attr("data-id");
         $('#tabText_' + tabId).addClass("displayedTab");
-        $('#loadingList').show();
-        $('.tabListItem').remove();
-        $.getJSON("/m/get_list", {
+        doListLoading();
+        $.getJSON("/m/get_tab", {
             'tab_id': tabId,
-            'page_no': 1
+//            'page_no': 1
         }, function(data) {
-            $('#loadingList').hide();
-            
-            for (var i=0; i<data.length; i++) {
-                var $li = getLi(data[i]);
-                $('#listLeft ul').append($li);                
+ //           $('#loadingList').hide();
+            stopListLoading(data.page);
+            /* begin if for has_list, ideally make into separate render function */
+            if (data.has_list) {
+                /* handle sort options on list */
+                $select = $('#orderBySelect');
+                $select.empty();
+                var sorts = data.sorts;
+                if (sorts.length == 0) { $select.hide(); } else { $select.show(); }
+                for (var j=0; j<sorts.length; j++) {
+                    var s = sorts[j];
+                    var $opt = getOption(s);
+                    $select.append($opt);
+                }
+                /* end handle sort */
+
+                /* handle displaying initial list on page */
+                displayList(data.page.items);
+                /* end display initial list */
+
             }
-            $('#listLeft ul li a').eq(0).click();
+            /* end if has_list */
+
+            
         });
     });
+
+    $('#orderBySelect').change(function(e) {
+        var sortString = $(this).val();
+        doListLoading();
+        $.getJSON("/m/get_list", {
+            'tab_id': $('.innerSelected').attr("data-id"),
+            'sort': sortString
+        }, function(data) {
+            stopListLoading(data);
+            displayList(data.items);
+        });
+    });
+
+    $('#triangleUp').click(function() {
+        goPreviousPage();
+    });    
+
+    $('#triangleDown').click(function() {
+        goNextPage();
+    });
+
 });
+
+function displayList(items) {
+    for (var i=0; i<items.length; i++) {
+        var $li = getLi(items[i]);
+        $('#listLeft ul').append($li);                
+    }
+    $('#listLeft ul li a').eq(0).click();
+}
+
+function doListLoading() {
+    $('#loadingList').show();
+    $('.tabListItem').remove();
+}
+
+function stopListLoading(page) {
+    $('#currPageNo').text(page.page_no);
+    $('#totalPages').text(page.num_pages);
+    $('#loadingList').hide();
+    if (page.page_no == 1) {
+        $('#triangleUp').hide();
+    } else {
+        $('#triangleUp').show();
+    }
+    if (page.page_no == page.num_pages) {
+        $('#triangleDown').hide();
+    } else {
+        $('#triangleDown').show();
+    }
+}
+
+function goNextPage() {
+    var currPage = parseInt($('#currPageNo').text());
+    var totalPages = parseInt($('#totalPages').text());
+    if (currPage < totalPages) {
+        getPage(currPage + 1);
+    }
+}
+
+function goPreviousPage() {
+    var currPage = parseInt($('#currPageNo').text());
+    if (currPage > 1) {
+        getPage(currPage - 1);
+    }
+}
+
+function getPage(pageNo) {
+    doListLoading();
+    $.getJSON("/m/get_list", {
+        'tab_id': $('.innerSelected').attr("data-id"),
+        'page': pageNo,
+        'sort': $('#orderBySelect').val()    
+    }, function(page) {
+        stopListLoading(page);
+        displayList(page.items);
+    });
+}
diff --git a/itf/templates/noel/insidepage.html b/itf/templates/noel/insidepage.html
index 8b2b1ac..7464ff3 100644
--- a/itf/templates/noel/insidepage.html
+++ b/itf/templates/noel/insidepage.html
@@ -51,7 +51,7 @@
                 {% endfor %}
                     <!--is this select okay, my forms are a bit weak, do we need action and method here?--><form class="selectMenu" action="" method="post">
                         <span>Order by</span><!--Can i avoid a p tag here to keep it inline-->
-                        <select>Select
+                        <select id="orderBySelect">Select
                             <option value="atoz">A-Z</option>
                             <option value="ztoa">Z-A</option>
                             <option value="title">Title</option>
@@ -76,6 +76,10 @@
         	<div id="triangleDown">&#x25BC;</div>
                 
         </div>
+        <div id="pagination" style="display:none;"> <!-- Please, please let me display pagination info -->
+            <span id="currPageNo">1</span>
+            <span id="totalPages"></span>
+        </div>
 
     </div>