X-Git-Url: http://git.veekun.com/zzz-floof.git/blobdiff_plain/3bdc7971c3359efe7f38eb1b94253b416471b87b..422dfbb85e3efd51efd676779012aeb3f9575ba4:/floof/model/art.py diff --git a/floof/model/art.py b/floof/model/art.py index 66cbe0b..451383b 100644 --- a/floof/model/art.py +++ b/floof/model/art.py @@ -6,22 +6,22 @@ # from elixir import Entity, Field, Integer, Unicode from elixir import * +import elixir 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)) - hash = Field(String) + hash = Field(String, unique=True, required=True) - uploaded_by = ManyToOne('User') + uploader = ManyToOne('User', required=True) tags = OneToMany('Tag') + discussion = ManyToOne('Discussion') # def __init__(self, **kwargs): # # I wanted to check for the existence of the file, but... @@ -35,7 +35,8 @@ class Art(Entity): 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): @@ -44,24 +45,44 @@ class Art(Entity): def add_tags(self, tags, user): - for tag in tags.split(): - if len(tag) > 50: - raise "Long Tag!" # can we handle this more gracefully? - # sqlite seems happy to store strings much longer than the supplied limit... + for text in tags.split(): + if text[0] == '-': + # Nega-tags + tagtext = TagText.get_by(text=text[1:]) + if tagtext: + tag = Tag.get_by(art=self, tagger=user, tagtext=tagtext) + if tag: + elixir.session.delete(tag) + + 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... - # elixir should really have its own find_or_create. - tagtext = find_or_create(TagText, text=tag) - 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): - # 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') + tagger = ManyToOne('User', ondelete='cascade') tagtext = ManyToOne('TagText') # this text setter is no longer useful since I changed the way Art#add_tags works @@ -77,11 +98,27 @@ class Tag(Entity): if not self.tagtext: return "(broken)" return unicode(self.tagtext) - - + + class TagText(Entity): text = Field(Unicode(50)) # gotta enforce this somehow tags = OneToMany('Tag') - + def __unicode__(self): - return self.text \ No newline at end of file + return self.text + + +class Rating(Entity): + art = ManyToOne('Art', ondelete='cascade') + rater = ManyToOne('User', ondelete='cascade') + score = Field(Integer) + + # @score.setter + # def score(self, value): + + options = {-1:"sucks", 0:"undecided", 1:"good", 2:"great"} + default = 0 + # options = ["sucks","neutral","good","great"] + + +Rating.reverse_options = dict (zip(Rating.options.values(), Rating.options.keys()))