X-Git-Url: https://git.openstreetmap.org./osqa.git/blobdiff_plain/4cf907a4492224ac83151ce54a845f72363fc4a9..50218f3c76fc43de83da77dda0501b5aabffa4d4:/forum/models/node.py diff --git a/forum/models/node.py b/forum/models/node.py index 4f6b6d1..b375ec4 100644 --- a/forum/models/node.py +++ b/forum/models/node.py @@ -1,4 +1,5 @@ from base import * +import re from tag import Tag import markdown @@ -79,13 +80,21 @@ class NodeMetaClass(BaseMetaClass): class NodeQuerySet(CachedQuerySet): + def obj_from_datadict(self, datadict): + cls = NodeMetaClass.types.get(datadict.get("node_type", ""), None) + if cls: + obj = cls() + obj.__dict__.update(datadict) + return obj + else: + return super(NodeQuerySet, self).obj_from_datadict(datadict) + def get(self, *args, **kwargs): - node = super(NodeQuerySet, self).get(*args, **kwargs) - cls = NodeMetaClass.types.get(node.node_type, None) + return super(NodeQuerySet, self).get(*args, **kwargs).leaf - if cls and (node.__class__ is not cls): - return node.leaf - return node + def filter_state(self, **kwargs): + apply_bool = lambda q, b: b and q or ~q + return self.filter(*[apply_bool(models.Q(state_string__contains="(%s)" % s), b) for s, b in kwargs.items()]) class NodeManager(CachedManager): @@ -103,6 +112,60 @@ class NodeManager(CachedManager): kwargs['node_type__in'] = [t.get_type() for t in types] return self.get(*args, **kwargs) + def filter_state(self, **kwargs): + return self.all().filter_state(**kwargs) + + +class NodeStateDict(object): + def __init__(self, node): + self.__dict__['_node'] = node + + def __getattr__(self, name): + if self.__dict__.get(name, None): + return self.__dict__[name] + + try: + node = self.__dict__['_node'] + action = NodeState.objects.get(node=node, state_type=name).action + self.__dict__[name] = action + return action + except: + return None + + def __setattr__(self, name, value): + current = self.__getattr__(name) + + if value: + if current: + current.action = value + current.save() + else: + node = self.__dict__['_node'] + state = NodeState(node=node, action=value, state_type=name) + state.save() + self.__dict__[name] = value + + if not "(%s)" % name in node.state_string: + node.state_string = "%s(%s)" % (node.state_string, name) + node.save() + else: + if current: + node = self.__dict__['_node'] + node.state_string = "".join("(%s)" % s for s in re.findall('\w+', node.state_string) if s != name) + node.save() + current.node_state.delete() + del self.__dict__[name] + + +class NodeStateQuery(object): + def __init__(self, node): + self.__dict__['_node'] = node + + def __getattr__(self, name): + node = self.__dict__['_node'] + return "(%s)" % name in node.state_string + + class Node(BaseModel, NodeContent): __metaclass__ = NodeMetaClass @@ -114,8 +177,7 @@ class Node(BaseModel, NodeContent): added_at = models.DateTimeField(default=datetime.datetime.now) score = models.IntegerField(default=0) - deleted = models.ForeignKey('Action', null=True, unique=True, related_name="deleted_node") - in_moderation = models.ForeignKey('Action', null=True, unique=True, related_name="moderated_node") + state_string = models.TextField(default='') last_edited = models.ForeignKey('Action', null=True, unique=True, related_name="edited_node") last_activity_by = models.ForeignKey(User, null=True) @@ -126,10 +188,8 @@ class Node(BaseModel, NodeContent): extra_ref = models.ForeignKey('Node', null=True) extra_count = models.IntegerField(default=0) - extra_action = models.ForeignKey('Action', null=True, related_name="extra_node") - + marked = models.BooleanField(default=False) - wiki = models.BooleanField(default=False) comment_count = DenormalizedField("children", node_type="comment", canceled=False) flag_count = DenormalizedField("flags") @@ -140,7 +200,7 @@ class Node(BaseModel, NodeContent): @classmethod def cache_key(cls, pk): - return '%s.node:%s' % (settings.APP_URL, pk) + return '%s:node:%s' % (settings.APP_URL, pk) @classmethod def get_type(cls): @@ -157,12 +217,36 @@ class Node(BaseModel, NodeContent): leaf.__dict__ = self.__dict__ return leaf + @property + def nstate(self): + state = self.__dict__.get('_nstate', None) + + if state is None: + state = NodeStateDict(self) + self._nstate = state + + return state + + @property + def nis(self): + nis = self.__dict__.get('_nis', None) + + if nis is None: + nis = NodeStateQuery(self) + self._nis = nis + + return nis + + @property + def deleted(self): + return self.nis.deleted + @property def absolute_parent(self): if not self.abs_parent_id: - return self.leaf + return self - return self.abs_parent.leaf + return self.abs_parent @property def summary(self): @@ -203,47 +287,70 @@ class Node(BaseModel, NodeContent): self.save() - def get_tag_list_if_changed(self): + def _list_changes_in_tags(self): dirty = self.get_dirty_fields() - active_user = self.last_edited and self.last_edited.by or self.author - if 'tagnames' in dirty: - new_tags = self.tagname_list() - old_tags = dirty['tagnames'] - - if old_tags is None or not old_tags: - old_tags = [] + if not 'tagnames' in dirty: + return None + else: + if self._original_state['tagnames']: + old_tags = set(name for name in self._original_state['tagnames'].split(u' ')) else: - old_tags = [name for name in dirty['tagnames'].split(u' ')] + old_tags = set() + new_tags = set(name for name in self.tagnames.split(u' ') if name) + + return dict( + current=list(new_tags), + added=list(new_tags - old_tags), + removed=list(old_tags - new_tags) + ) - tag_list = [] + def _last_active_user(self): + return self.last_edited and self.last_edited.by or self.author - for name in new_tags: + def _process_changes_in_tags(self): + tag_changes = self._list_changes_in_tags() + + if tag_changes is not None: + for name in tag_changes['added']: try: tag = Tag.objects.get(name=name) except: - tag = Tag.objects.create(name=name, created_by=active_user or self.author) - - tag_list.append(tag) + tag = Tag.objects.create(name=name, created_by=self._last_active_user()) - if not name in old_tags: - tag.used_count = tag.used_count + 1 - if tag.deleted: - tag.unmark_deleted() + if not self.nis.deleted: + tag.used_count = models.F('used_count') + 1 tag.save() - for name in [n for n in old_tags if not n in new_tags]: - tag = Tag.objects.get(name=name) - tag.used_count = tag.used_count - 1 - if tag.used_count == 0: - tag.mark_deleted(active_user) - tag.save() + if not self.nis.deleted: + for name in tag_changes['removed']: + try: + tag = Tag.objects.get(name=name) + tag.used_count = models.F('used_count') - 1 + tag.save() + except: + pass - return tag_list + return True - return None + return False + + def mark_deleted(self, action): + self.nstate.deleted = action + self.save() + + if action: + for tag in self.tags.all(): + tag.used_count = models.F('used_count') - 1 + tag.save() + else: + for tag in Tag.objects.filter(name__in=self.tagname_list()): + tag.used_count = models.F('used_count') + 1 + tag.save() def save(self, *args, **kwargs): + tags_changed = self._process_changes_in_tags() + if not self.id: self.node_type = self.get_type() super(BaseModel, self).save(*args, **kwargs) @@ -252,10 +359,9 @@ class Node(BaseModel, NodeContent): if self.parent_id and not self.abs_parent_id: self.abs_parent = self.parent.absolute_parent - - tags = self.get_tag_list_if_changed() + super(Node, self).save(*args, **kwargs) - if tags is not None: self.tags = tags + if tags_changed: self.tags = list(Tag.objects.filter(name__in=self.tagname_list())) class Meta: app_label = 'forum' @@ -272,3 +378,13 @@ class NodeRevision(BaseModel, NodeContent): app_label = 'forum' +class NodeState(models.Model): + node = models.ForeignKey(Node, related_name='states') + state_type = models.CharField(max_length=16) + action = models.OneToOneField('Action', related_name="node_state") + + class Meta: + unique_together = ('node', 'state_type') + app_label = 'forum' + +