add artist when posting

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

diff --git a/floof/model/art.py b/floof/model/art.py
index e462d0b..ff8035a 100644 (file)
--- a/floof/model/art.py
+++ b/floof/model/art.py
@@ -11,32 +11,25 @@ import elixir
 from pylons import config
 
 from floof.lib.file_storage import get_path, save_file
 from pylons import config
 
 from floof.lib.file_storage import get_path, save_file

-
-from floof.lib.dbhelpers import find_or_create
-
-
+from floof.lib.dbhelpers import find_or_create, update_or_create
+import floof.model.comments

 
 class Art(Entity):
     title = Field(Unicode(120))
     original_filename = Field(Unicode(120))
 
 class Art(Entity):
     title = Field(Unicode(120))
     original_filename = Field(Unicode(120))

-    hash = Field(String)
+    hash = Field(String, unique=True, required=True)

 
 

-    uploaded_by = ManyToOne('User')    
+    uploader = ManyToOne('User', required=True)

     tags = OneToMany('Tag')
     tags = OneToMany('Tag')

+    discussion = ManyToOne('Discussion')
+
+    user_relations = OneToMany('UserRelation')

 
 

-    # def __init__(self, **kwargs):
-    #     # I wanted to check for the existence of the file, but...
-    #     # for some reason this FieldStorage object always conditions as falsey.
-    #     # self.hash = save_file("art", kwargs.pop('file'))
-    #     super(Art, self).__init__(**kwargs)
-    #     # this is what super is doing, pretty much.
-    #     # for key, value in kwargs.items():
-    #     #     setattr(self, key, value)
-    # left for posterity.

 
     def set_file(self, file):
         self.hash = save_file("art", file)
 
     def set_file(self, file):
         self.hash = save_file("art", file)

-        
+        self.original_filename = file.filename
+

     file = property(get_path, set_file)
 
     def get_path(self):
     file = property(get_path, set_file)
 
     def get_path(self):


@@ -54,46 +47,88 @@ class Art(Entity):
                     if tag:
                         elixir.session.delete(tag)
 
                     if tag:
                         elixir.session.delete(tag)
 

-            else: 
+            else:

                 if len(text) > 50:
                     raise "Long Tag!" # can we handle this more gracefully?
                 # sqlite seems happy to store strings much longer than the supplied limit...
 
                 if len(text) > 50:
                     raise "Long Tag!" # can we handle this more gracefully?
                 # sqlite seems happy to store strings much longer than the supplied limit...
 

-
-

                 # elixir should really have its own find_or_create.
                 tagtext = find_or_create(TagText, text=text)
                 tag     = find_or_create(Tag, art=self, tagger=user, tagtext=tagtext)
 
 
                 # elixir should really have its own find_or_create.
                 tagtext = find_or_create(TagText, text=text)
                 tag     = find_or_create(Tag, art=self, tagger=user, tagtext=tagtext)
 
 

+
+
+    def rate(self, score, user):
+        return update_or_create(Rating, {"rater":user, "art":self}, {"score":score})
+
+    def user_score(self, user):
+        rating = Rating.get_by(rater=user, art=self)
+        if rating:
+            return rating.score
+        return Rating.default
+

     def __unicode__(self):
         return self.get_path()
 
 
 class Tag(Entity):
     def __unicode__(self):
         return self.get_path()
 
 
 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')
     art = ManyToOne('Art', ondelete='cascade')

-    tagger = ManyToOne('User')
+    tagger = ManyToOne('User', ondelete='cascade')

     tagtext = ManyToOne('TagText')
 
     tagtext = ManyToOne('TagText')
 

-    # this text setter is no longer useful since I changed the way Art#add_tags works
-    # but I'll leave it in here just for several minutes nostalgia.
-    # def set_text(self, text):
-    #     self.tagtext = TagText.get_by(text=text)
-    #     if not self.tagtext:
-    #         self.tagtext = TagText(text=text)
-    #
-    # text = property(lambda self: self.tagtext.text, set_text)
-

     def __unicode__(self):
         if not self.tagtext:
             return "(broken)"
         return unicode(self.tagtext)
     def __unicode__(self):
         if not self.tagtext:
             return "(broken)"
         return unicode(self.tagtext)

-    
-    
+
+

 class TagText(Entity):
     text = Field(Unicode(50)) # gotta enforce this somehow
     tags = OneToMany('Tag')
 class TagText(Entity):
     text = Field(Unicode(50)) # gotta enforce this somehow
     tags = OneToMany('Tag')

+
+    def __unicode__(self):
+        return self.text
+
+
+class Rating(Entity):
+    art = ManyToOne('Art', ondelete='cascade')
+    rater = ManyToOne('User', ondelete='cascade')
+    score = Field(Integer)
+
+    options = {-1:"sucks", 0:"undecided", 1:"good", 2:"great"}
+    default = 0
+
+Rating.reverse_options = dict (zip(Rating.options.values(), Rating.options.keys()))
+
+
+
+class UserRelation(Entity):
+    user = ManyToOne("User")
+    art = ManyToOne("Art")
+    kind = Field(String) # by for of
+    creator = ManyToOne("User")
+    confirmed_by_related_user = Field(Boolean)
+
+    # it is useful to record which authority figure on a given artwork
+    # confirmed the validity of this relation.
+    confirmed_by_authority = ManyToOne("User")

     
     

+    def __init__(self, **kwargs):
+        super(UserRelation, self).__init__(**kwargs)
+        assert self.user and self.art and self.kind and self.creator
+        
+        if self.creator == self.user:
+            self.confirmed_by_related_user = True
+        # TODO: implement authorities
+        # if self.creator in self.art.authorities
+        #     self.confirmed_by_authority = self.creator
+

     def __unicode__(self):
     def __unicode__(self):

-        return self.text
\ No newline at end of file
+        return "%s: %s" % (self.kind, self.related_user)
+
+    
+    
+# class CharacterRelation(Entity):
+#     pass