Moved all searching to search view.

diff --git a/catalogue/forms.py b/catalogue/forms.py
index 7472eb4..51d7cba 100644 (file)
--- a/catalogue/forms.py
+++ b/catalogue/forms.py
@@ -8,11 +8,13 @@ from catalogue.lib.slughifi import slughifi
 
 class SearchForm(forms.Form):
     q = JQueryAutoCompleteField('/katalog/tags/', {'minChars': 2, 'selectFirst': True, 'cacheLength': 50})
 
 class SearchForm(forms.Form):
     q = JQueryAutoCompleteField('/katalog/tags/', {'minChars': 2, 'selectFirst': True, 'cacheLength': 50})

-    tags = forms.CharField(widget=forms.HiddenInput)
+    tags = forms.CharField(widget=forms.HiddenInput, required=False)

     
     def __init__(self, *args, **kwargs):
     
     def __init__(self, *args, **kwargs):

+        tags = kwargs.pop('tags', [])

         super(SearchForm, self).__init__(*args, **kwargs)
         self.fields['q'].widget.attrs['title'] = u'tytuł utworu, motyw lub kategoria'
         super(SearchForm, self).__init__(*args, **kwargs)
         self.fields['q'].widget.attrs['title'] = u'tytuł utworu, motyw lub kategoria'

+        self.fields['tags'].initial = '/'.join(tag.slug for tag in Tag.get_tag_list(tags))

 
 
 class UserSetsForm(forms.Form):
 
 
 class UserSetsForm(forms.Form):

diff --git a/catalogue/urls.py b/catalogue/urls.py
index e56d822..5827285 100644 (file)
--- a/catalogue/urls.py
+++ b/catalogue/urls.py
@@ -9,6 +9,7 @@ urlpatterns = patterns('catalogue.views',
     url(r'^zestawy/nowy/$', 'new_set'),
     url(r'^lektura/(?P<slug>[a-zA-Z0-9-]+)/$', 'book_detail'),
     url(r'^tags/$', 'tags_starting_with', name='hint'),
     url(r'^zestawy/nowy/$', 'new_set'),
     url(r'^lektura/(?P<slug>[a-zA-Z0-9-]+)/$', 'book_detail'),
     url(r'^tags/$', 'tags_starting_with', name='hint'),

+    url(r'^szukaj/$', 'search', name='search'),

     url(r'^(?P<tags>[a-zA-Z-/]+)/$', 'tagged_book_list', name='tagged_book_list'),
 )
 
     url(r'^(?P<tags>[a-zA-Z-/]+)/$', 'tagged_book_list', name='tagged_book_list'),
 )
 

diff --git a/catalogue/views.py b/catalogue/views.py
index d5b4f69..0166133 100644 (file)
--- a/catalogue/views.py
+++ b/catalogue/views.py
@@ -26,20 +26,26 @@ class LazyEncoder(simplejson.JSONEncoder):
         return obj
 
 
         return obj
 
 

-def catalogue_redirect(request, tags=''):
-    if len(request.GET['q']) > 0:
+def search(request):
+    query = request.GET.get('q', '')
+    tags = request.GET.get('tags', '')
+    if tags == '':
+        tags = []
+    
+    try:
+        tag_list = models.Tag.get_tag_list(tags)
+        tag = models.Tag.objects.get(name=query)
+    except models.Tag.DoesNotExist:
+        print "%r, %r" % (query, tags)

         try:
         try:

-            tag = models.Tag.objects.get(name=request.GET['q'])
-            if len(tags):
-                tags += '/'
-            tags = tags + tag.slug
-        except models.Tag.DoesNotExist:
-            book = get_object_or_404(models.Book, title=request.GET['q'])
-            return HttpResponseRedirect(book.get_absolute_url())
-    if len(tags) > 0:
-        return HttpResponseRedirect(reverse('catalogue.views.tagged_book_list', kwargs=dict(tags=tags)))
+            book = models.Book.objects.get(title=query)
+        except models.Book.DoesNotExist:
+            return HttpResponseRedirect(reverse('catalogue.views.main_page'))

     else:
     else:

-        return HttpResponseRedirect(reverse('catalogue.views.main_page'))
+        tag_list.append(tag)
+        return HttpResponseRedirect(reverse('catalogue.views.tagged_book_list', 
+            kwargs={'tags': '/'.join(tag.slug for tag in tag_list)}
+        ))

 
 
 def tags_starting_with(request):
 
 
 def tags_starting_with(request):


@@ -63,10 +69,7 @@ def tags_starting_with(request):
         return HttpResponse('')
 
 
         return HttpResponse('')
 
 

-def main_page(request):
-    if 'q' in request.GET:
-        return catalogue_redirect(request)
-    
+def main_page(request):    

     if request.user.is_authenticated():
         extra_where = '(NOT catalogue_tag.category = "set" OR catalogue_tag.user_id = %d)' % request.user.id
     else:
     if request.user.is_authenticated():
         extra_where = '(NOT catalogue_tag.category = "set" OR catalogue_tag.user_id = %d)' % request.user.id
     else:


@@ -80,9 +83,6 @@ def main_page(request):
 
 
 def book_list(request):
 
 
 def book_list(request):

-    if 'q' in request.GET:
-        return catalogue_redirect(request)
-        

     books = models.Book.objects.all()
     form = forms.SearchForm()
     
     books = models.Book.objects.all()
     form = forms.SearchForm()
     


@@ -95,9 +95,6 @@ def book_list(request):
 
 
 def tagged_book_list(request, tags=''):
 
 
 def tagged_book_list(request, tags=''):

-    if 'q' in request.GET:
-        return catalogue_redirect(request, tags)
-    

     try:
         tags = models.Tag.get_tag_list(tags)
     except models.Tag.DoesNotExist:
     try:
         tags = models.Tag.get_tag_list(tags)
     except models.Tag.DoesNotExist:


@@ -116,7 +113,7 @@ def tagged_book_list(request, tags=''):
         queryset_or_model=models.Book,
         tags=tags,
         template_name='catalogue/tagged_book_list.html',
         queryset_or_model=models.Book,
         tags=tags,
         template_name='catalogue/tagged_book_list.html',

-        extra_context = {'categories': categories, 'form': forms.SearchForm() },
+        extra_context = {'categories': categories },

     )
 
 
     )
 
 


@@ -124,7 +121,6 @@ def book_detail(request, slug):
     book = get_object_or_404(models.Book, slug=slug)
     tags = list(book.tags.filter(~Q(category='set')))
     categories = split_tags(tags)
     book = get_object_or_404(models.Book, slug=slug)
     tags = list(book.tags.filter(~Q(category='set')))
     categories = split_tags(tags)

-    search_form = forms.SearchForm()

     
     return render_to_response('catalogue/book_detail.html', locals(),
         context_instance=RequestContext(request))
     
     return render_to_response('catalogue/book_detail.html', locals(),
         context_instance=RequestContext(request))

diff --git a/templates/catalogue/book_list.html b/templates/catalogue/book_list.html
index f6c3b75..f4f575c 100644 (file)
--- a/templates/catalogue/book_list.html
+++ b/templates/catalogue/book_list.html
@@ -16,7 +16,7 @@
 
 {% block body %}
     <h1>Alfabetyczny spis utworów</h1>
 
 {% block body %}
     <h1>Alfabetyczny spis utworów</h1>

-    <form action="." method="GET" accept-charset="utf-8" id="search-form">
+    <form action="{% url search %}" method="GET" accept-charset="utf-8" id="search-form">

         <p>{{ form.q }} <input type="submit" value="Szukaj" /> <strong>lub</strong> <a href="{% url main_page %}">wróć do strony głównej</a></p>
     </form>
     
         <p>{{ form.q }} <input type="submit" value="Szukaj" /> <strong>lub</strong> <a href="{% url main_page %}">wróć do strony głównej</a></p>
     </form>
     

diff --git a/templates/catalogue/main_page.html b/templates/catalogue/main_page.html
index 8fe16d1..507fa3a 100644 (file)
--- a/templates/catalogue/main_page.html
+++ b/templates/catalogue/main_page.html
@@ -17,8 +17,8 @@
         {% chunk "site-description" %}
     </div>
     <div class="clearboth"></div>
         {% chunk "site-description" %}
     </div>
     <div class="clearboth"></div>

-    <form action="." method="get" accept-charset="utf-8" id="search-form">
-        <p>{{ form.q }} <input type="submit" value="Szukaj" /> <strong>lub</strong> <a href="{% url catalogue.views.book_list %}">zobacz spis utworów</a> w naszym zbiorze</p>
+    <form action="{% url search %}" method="get" accept-charset="utf-8" id="search-form">
+        <p>{{ form.q }} {{ form.tags }} <input type="submit" value="Szukaj" /> <strong>lub</strong> <a href="{% url catalogue.views.book_list %}">zobacz spis utworów</a> w naszym zbiorze</p>

     </form>
     
     <div id="tags-list">
     </form>
     
     <div id="tags-list">

diff --git a/templates/catalogue/tagged_book_list.html b/templates/catalogue/tagged_book_list.html
index 695a492..68253af 100644 (file)
--- a/templates/catalogue/tagged_book_list.html
+++ b/templates/catalogue/tagged_book_list.html
@@ -54,14 +54,15 @@
 
 {% block body %}
     <h1>{% title_from_tags tags %}</h1>
 
 {% block body %}
     <h1>{% title_from_tags tags %}</h1>

-    <form action="." method="get" accept-charset="utf-8" id="search-form">
-        <ol>
-            {% for tag in tags %}
-            <li class="category"><a href="{% catalogue_url tag %}">{{ tag }}</a> | <a href="{% catalogue_url tags -tag %}">x</a></li>
-            {% endfor %}
-            <li>{{ form.q }} <input type="submit" value="Szukaj"/></li>
-        </ol>
-    </form>
+    {% breadcrumbs tags %}
+    {# <form action="." method="get" accept-charset="utf-8" id="search-form"> #}
+    {#     <ol> #}
+    {#         {% for tag in tags %} #}
+    {#         <li class="category"><a href="{% catalogue_url tag %}">{{ tag }}</a> | <a href="{% catalogue_url tags -tag %}">x</a></li> #}
+    {#         {% endfor %} #}
+    {#         <li>{{ form.q }} <input type="submit" value="Szukaj"/></li> #}
+    {#     </ol> #}
+    {# </form> #}

     
     {% autopaginate object_list 10 %}
     <div id="books-list">
     
     {% autopaginate object_list 10 %}
     <div id="books-list">