From: afilsaime <shadow_kungfu@hotmail.fr>
Date: Tue, 9 Jun 2015 13:58:56 +0000 (+0200)
Subject: fix bug on playlist add and add pagination on advance search
X-Git-Tag: 1.6a^2~15^2~24
X-Git-Url: https://git.parisson.com/?a=commitdiff_plain;h=222ca247913f97c4d60c367819fe31fc34744e4f;p=telemeta.git

fix bug on playlist add and add pagination on advance search
---

diff --git a/telemeta/haystack_urls.py b/telemeta/haystack_urls.py
index 0692c714..28f37011 100644
--- a/telemeta/haystack_urls.py
+++ b/telemeta/haystack_urls.py
@@ -11,5 +11,5 @@ urlpatterns = patterns('',
     url(r'^advance/$', HaystackAdvanceSearch(form_class=HayAdvanceForm, template='search/advanceSearch.html'), name='haystack_advance_search'),
     url(r'^advance/(?P<type>[A-Za-z0-9._-]+)/$', HaystackAdvanceSearch(form_class=HayAdvanceForm, template='search/advanceSearch.html'), name='haystack_advance_search_type'),
     url(r'^playlist_add/(?P<type>[A-Za-z0-9._-]+)/$', NewPlaylistView().display, name='haystack_playlist'),
-    url(r'^playlist_confirmation/$',NewPlaylistView().addToPlaylist, name='add_confirmation'),
+    url(r'^playlist_confirmation/(?P<type>[A-Za-z0-9._-]+)/$',NewPlaylistView().addToPlaylist, name='add_confirmation'),
 )
diff --git a/telemeta/templates/search/addplaylist.html b/telemeta/templates/search/addplaylist.html
index 778bb45c..3e3c7b70 100644
--- a/telemeta/templates/search/addplaylist.html
+++ b/telemeta/templates/search/addplaylist.html
@@ -54,7 +54,8 @@
     </style>
 {% endblock %}
 {% block content %}
-    <form method="POST" action="{%url "add_confirmation"%}">{% csrf_token %}
+
+    <form method="POST" action="{%url "add_confirmation" type %}">{% csrf_token %}
     <div class="row">
         <div class="col-md-4">
             <table class="listing" style="width:100%">
diff --git a/telemeta/templates/search/advanceSearch.html b/telemeta/templates/search/advanceSearch.html
index f7a7fb5d..973ca031 100644
--- a/telemeta/templates/search/advanceSearch.html
+++ b/telemeta/templates/search/advanceSearch.html
@@ -1,6 +1,7 @@
 {% extends 'telemeta/base.html' %}
 {% load telemeta_utils %}
 {% load i18n %}
+{% load bootstrap_pagination %}
 
 
 {% block title %}
@@ -180,7 +181,17 @@
     </form>
 
         {% if query %}
-            <h1>{% trans "Results" %}</h1>
+
+            <div class="row">
+                <div class="col-md-3">
+                    <img src="{{ STATIC_URL }}telemeta/images/search_red.png" alt="search-results" style="vertical-align:middle" /> <h1>{% trans "Search Results" %}</h1>
+                </div>
+                <div class="col-md-7">
+                    {% bootstrap_paginate page range=10 show_first_last="True" %}
+                </div>
+                <div class="col-md-0">&nbsp;</div>
+            </div>
+
             {% ifequal type 'item' %}
                  <p><b>Items ({{item_count}}) | <a href="{% url "haystack_advance_search_type" "collection" %}?{{request.GET.urlencode}}">Collections ({{collection_count}})</a></b></p>
              {% else %}
@@ -188,7 +199,7 @@
                     <p><b><a href="{% url "haystack_advance_search_type" "item" %}?{{request.GET.urlencode}}">Items ({{item_count}}) </a>| Collections ({{collection_count}})</b></p>
                 {% endifequal %}
              {% endifequal %}
-             <form method="POST" action="{% url "haystack_playlist"%} ">{% csrf_token %}
+             <form method="POST" action="{% url "haystack_playlist" type%} ">{% csrf_token %}
              {% with object_list as items %}
                 <div class="fullpage">
                     {% ifequal type 'item' %}
@@ -200,13 +211,6 @@
                     {%endifequal%}
                 </div>
 
-                {% if page.has_previous or page.has_next %}
-                    <div>
-                        {% if page.has_previous %}<a href="?{{request.GET.urlencode}};page={{ page.previous_page_number }}">{% endif %}&laquo; Previous{% if page.has_previous %}</a>{% endif %}
-                        |
-                        {% if page.has_next %}<a href="?{{request.GET.urlencode}};page={{ page.next_page_number }}">{% endif %}Next &raquo;{% if page.has_next %}</a>{% endif %}
-                    </div>
-                {% endif %}
             {% endwith %}
             {% if user.is_authenticated %}
                 <input type="hidden" name="lastquerypath" value="{{request.get_full_path}}"/>
diff --git a/telemeta/views/new_playlist.py b/telemeta/views/new_playlist.py
index b60fc855..2ddb67e3 100644
--- a/telemeta/views/new_playlist.py
+++ b/telemeta/views/new_playlist.py
@@ -13,7 +13,7 @@ class NewPlaylistView(object):
 
         idlist = request.POST.getlist('selected_items_list')
         itemlist = []
-        if self.type =="items":
+        if self.type =="item":
             for itemid in idlist:
                 itemlist.append(MediaItem.objects.all().get(id=itemid))