trying out resource routing. Works decently. Added lots of notes.
[zzz-floof.git] / floof / controllers / art.py
index f08cd60..7ca1285 100644 (file)
@@ -1,14 +1,14 @@
 import logging
 
 from pylons import request, response, session, tmpl_context as c, h
-from pylons.controllers.util import abort, redirect_to
-
+from pylons.controllers.util import abort, redirect
+from pylons import url
 from floof.lib.base import BaseController, render
 
 log = logging.getLogger(__name__)
 
 import elixir
-from floof.model.art import Art
+from floof.model.art import Art, Rating
 
 class ArtController(BaseController):
     def __before__(self, id=None):
@@ -26,7 +26,7 @@ class ArtController(BaseController):
         return render("/art/new.mako")
 
     # TODO: login required
-    def upload(self):
+    def create(self):
         Art(uploaded_by=c.user, **request.params)
         elixir.session.commit()
         redirect_to(controller="main", action="index")
@@ -37,16 +37,23 @@ class ArtController(BaseController):
             c.your_score = c.art.user_score(c.user)
         return render("/art/show.mako")
         
-    # TODO: login required
-    def tag(self, id):
-        # c.art = h.get_object_or_404(Art, id=id)
-        c.art.add_tags(request.params["tags"], c.user)
-        elixir.session.commit()
-        redirect_to('show_art', id=c.art.id)
-    
+    # TODO: login required
+    def tag(self, id):
+        # c.art = h.get_object_or_404(Art, id=id)
+        c.art.add_tags(request.params["tags"], c.user)
+        elixir.session.commit()
+        redirect_to('show_art', id=c.art.id)
+    # 
     # TODO: login required
     def rate(self, id):
         # c.art = h.get_object_or_404(Art, id=id)
-        c.art.rate(request.params["score"], c.user)
+        score = request.params.get("score")
+        if score and score.isnumeric():
+            score = int(score)
+        else:
+            score = Rating.reverse_options.get(score)
+        
+        c.art.rate(score, c.user)
         elixir.session.commit()
-        redirect_to('show_art', id=c.art.id)
+            
+        redirect(url('art', id=c.art.id))