merged in my branch 'resources', which is not aptly named anymore since it no longer...

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

diff --git a/floof/controllers/art.py b/floof/controllers/art.py
index 7ca1285..5186daa 100644 (file)
--- a/floof/controllers/art.py
+++ b/floof/controllers/art.py
@@ -10,6 +10,9 @@ log = logging.getLogger(__name__)
 import elixir
 from floof.model.art import Art, Rating
 
+from sqlalchemy.exceptions import IntegrityError
+
+
 class ArtController(BaseController):
     def __before__(self, id=None):
         super(ArtController, self).__before__()
@@ -27,9 +30,19 @@ class ArtController(BaseController):
 
     # TODO: login required
     def create(self):
-        Art(uploaded_by=c.user, **request.params)
-        elixir.session.commit()
-        redirect_to(controller="main", action="index")
+        c.art = Art(uploader=c.user, **request.params)
+
+        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 show(self, id):
         # c.art = h.get_object_or_404(Art, id=id)
@@ -37,13 +50,7 @@ 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 rate(self, id):
         # c.art = h.get_object_or_404(Art, id=id)
@@ -56,4 +63,4 @@ class ArtController(BaseController):
         c.art.rate(score, c.user)
         elixir.session.commit()
             
-        redirect(url('art', id=c.art.id))
+        redirect(url('show_art', id=c.art.id))