From: Nick Retallack <nickretallack@gmil.com>
Date: Sun, 4 Oct 2009 18:12:23 +0000 (-0700)
Subject: Searching for one tag works.  Two tags acts like or and returns multiple copies.
X-Git-Url: http://git.veekun.com/zzz-floof.git/commitdiff_plain/2bb6d6e6d79441c4918bd903eae9099f03a65675?ds=sidebyside

Searching for one tag works.  Two tags acts like or and returns multiple copies.
---

diff --git a/floof/controllers/search.py b/floof/controllers/search.py
new file mode 100644
index 0000000..75926e7
--- /dev/null
+++ b/floof/controllers/search.py
@@ -0,0 +1,35 @@
+import logging
+
+from pylons import request, response, session, tmpl_context as c
+from pylons.controllers.util import abort, redirect_to
+
+from floof.lib.base import BaseController, render
+
+log = logging.getLogger(__name__)
+
+from floof.model.art import Art, Tag, TagText
+import elixir
+
+class SearchController(BaseController):
+
+    def index(self):
+        # Return a rendered template
+        #return render('/search.mako')
+        # or, return a response
+        return 'Hello World'
+
+    def results(self):
+        """ Search, implemented the stupid way! """
+        query = request.params.get('query','')
+        tags = query.split()
+
+        tagtexts = TagText.query.filter(TagText.text.in_(tags))
+        tagtext_ids = map(lambda x:x.id, tagtexts)
+
+        # TODO: this is wrong.  Please fix it so it returns art that has all the tags.
+        art_tag_pairs = elixir.session.query(Art,Tag).filter(Art.id == Tag.art_id).\
+            filter(Tag.tagtext_id.in_(tagtext_ids)).all()
+        
+        # just the art please.
+        c.artwork = map(lambda x: x[0], art_tag_pairs)
+        return render('/index.mako')
\ No newline at end of file
diff --git a/floof/model/art.py b/floof/model/art.py
index fd80d93..420dff8 100644
--- a/floof/model/art.py
+++ b/floof/model/art.py
@@ -10,11 +10,9 @@ from elixir import *
 from pylons import config
 
 from floof.lib.file_storage import get_path, save_file
-
 from floof.lib.dbhelpers import find_or_create
 
 
-
 class Art(Entity):
     title = Field(Unicode(120))
     original_filename = Field(Unicode(120))
@@ -59,7 +57,7 @@ class Art(Entity):
 
 
 class Tag(Entity):
-    # look into how ondelete works.  It just sets a database property.
+    # look into how ondelete works.  This just sets a database property.
     art = ManyToOne('Art', ondelete='cascade')
     tagger = ManyToOne('User')
     tagtext = ManyToOne('TagText')
diff --git a/floof/templates/base.mako b/floof/templates/base.mako
index 5034bf0..19f8455 100644
--- a/floof/templates/base.mako
+++ b/floof/templates/base.mako
@@ -10,6 +10,10 @@
 <div id="header">
 <a href="${h.url_for("/")}">Home</a>
 
+${h.form (h.url_for(controller="search", action="results"))}
+${h.text('query')} ${h.submit('search','Search')}
+${h.end_form()}
+
     <div id="user">
         % if c.user:
         <p>Logged in as ${c.user.name}</p>
diff --git a/floof/tests/functional/test_search.py b/floof/tests/functional/test_search.py
new file mode 100644
index 0000000..62c8cd1
--- /dev/null
+++ b/floof/tests/functional/test_search.py
@@ -0,0 +1,7 @@
+from floof.tests import *
+
+class TestSearchController(TestController):
+
+    def test_index(self):
+        response = self.app.get(url(controller='search', action='index'))
+        # Test response...