add artist when posting

[zzz-floof.git] / floof / controllers / art.py

diff --git a/floof/controllers/art.py b/floof/controllers/art.py
index 6c5e454..5216da7 100644 (file)
--- a/floof/controllers/art.py
+++ b/floof/controllers/art.py
@@ -1,48 +1,76 @@
 import logging
 
 from pylons import request, response, session, tmpl_context as c, h
 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.lib.base import BaseController, render
 
 log = logging.getLogger(__name__)
 
 import elixir

-from floof.model.art import Art
+from floof.model.users import User
+from floof.model.art import Art, Rating, UserRelation
+from floof.model.comments import Discussion

 
 

-class ArtController(BaseController):
+from sqlalchemy.exceptions import IntegrityError

 
 

-    # def index():
-    #     c.artwork = Art.query.order_by(Art.id.desc()).all()
-    #     return render
+
+class ArtController(BaseController):
+    def __before__(self, id=None):
+        super(ArtController, self).__before__()
+        # Awesome refactoring!
+        if id:
+            c.art = h.get_object_or_404(Art, id=id)

 
     def new(self):
         """ New Art! """
         return render("/art/new.mako")
 
 
     def new(self):
         """ New Art! """
         return render("/art/new.mako")
 

+    # TODO: login required
+    def create(self):
+        # if 'file' not in request.params or not request.params['file']:
+        #     return "Validation Error: Needs a File"
+
+
+        c.art = Art(uploader=c.user, **request.params)
+        c.art.discussion = Discussion(count=0)
+        
+
+        artist = User.get_by(name=request.params['artist'])
+        if not artist:
+            return "Validation Error: Artist not found"
+
+        relation = UserRelation(user=artist, kind="by", creator=c.user, art=c.art)
+
+        try:
+            elixir.session.commit()
+            redirect(url('show_art', id=c.art.id))
+        except IntegrityError:
+            # hurr, there must be a better way to do this but I am lazy right now
+            hash = c.art.hash
+            elixir.session.rollback()
+            duplicate_art = Art.get_by(hash=hash)
+            h.flash("We already have that one.")
+            redirect(url('show_art', id=duplicate_art.id))

 
 

-    def upload(self):
-        print "PARAMS", request.params
-        Art(uploaded_by=c.user, **request.params)
-        elixir.session.commit()
-        redirect_to(controller="main", action="index")

 
     def show(self, id):
 
     def show(self, id):

-        c.art = h.get_object_or_404(Art, id=id)
+        # c.art = h.get_object_or_404(Art, id=id)

         if c.user:
             c.your_score = c.art.user_score(c.user)
         return render("/art/show.mako")
         if c.user:
             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(action="show", id=c.art.id)
-    
+
+

     # TODO: login required
     def rate(self, 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)
+        # c.art = h.get_object_or_404(Art, id=id)
+        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()
         elixir.session.commit()

-        redirect_to(action="show", id=c.art.id)
+
+        redirect(url('show_art', id=c.art.id))