Remove use of the permalink decorator

This commit is contained in:
Roberto Rosario
2015-04-06 01:36:22 -04:00
parent 4fde4b3aa7
commit 4d2891b361
5 changed files with 13 additions and 15 deletions

View File

@@ -4,6 +4,7 @@ import logging
from django.contrib.contenttypes import generic from django.contrib.contenttypes import generic
from django.contrib.contenttypes.models import ContentType from django.contrib.contenttypes.models import ContentType
from django.core.urlresolvers import reverse
from django.db import models from django.db import models
from django.utils.encoding import python_2_unicode_compatible from django.utils.encoding import python_2_unicode_compatible
from django.utils.translation import ugettext_lazy as _ from django.utils.translation import ugettext_lazy as _
@@ -52,9 +53,8 @@ class DocumentCheckout(models.Model):
event_document_check_out.commit(actor=self.user_object, target=self.document) event_document_check_out.commit(actor=self.user_object, target=self.document)
return result return result
@models.permalink
def get_absolute_url(self): def get_absolute_url(self):
return ('checkout:checkout_info', [self.document.pk]) reverse('checkout:checkout_info', args=[self.document.pk])
class Meta: class Meta:
verbose_name = _('Document checkout') verbose_name = _('Document checkout')

View File

@@ -1,5 +1,6 @@
from __future__ import unicode_literals from __future__ import unicode_literals
from django.core.urlresolvers import reverse
from django.db import models from django.db import models
from django.utils.encoding import python_2_unicode_compatible from django.utils.encoding import python_2_unicode_compatible
from django.utils.translation import ugettext, ugettext_lazy as _ from django.utils.translation import ugettext, ugettext_lazy as _
@@ -33,9 +34,8 @@ class Index(models.Model):
def __str__(self): def __str__(self):
return self.title return self.title
@models.permalink
def get_absolute_url(self): def get_absolute_url(self):
return ('indexing:index_instance_node_view', [self.instance_root.pk]) reverse('indexing:index_instance_node_view', args=[self.instance_root.pk])
def get_document_types_not_in_index(self): def get_document_types_not_in_index(self):
return DocumentType.objects.exclude(pk__in=self.document_types.all()) return DocumentType.objects.exclude(pk__in=self.document_types.all())
@@ -94,9 +94,8 @@ class IndexInstanceNode(MPTTModel):
def index(self): def index(self):
return self.index_template_node.index return self.index_template_node.index
@models.permalink
def get_absolute_url(self): def get_absolute_url(self):
return ('indexing:index_instance_node_view', [self.pk]) reverse('indexing:index_instance_node_view', args=[self.pk])
@property @property
def children(self): def children(self):

View File

@@ -8,9 +8,10 @@ import os
import tempfile import tempfile
import uuid import uuid
from django.db import models
from django.contrib.auth.models import User from django.contrib.auth.models import User
from django.core.exceptions import ValidationError from django.core.exceptions import ValidationError
from django.core.urlresolvers import reverse
from django.db import models
from django.utils.encoding import python_2_unicode_compatible from django.utils.encoding import python_2_unicode_compatible
from django.utils.translation import ugettext_lazy as _ from django.utils.translation import ugettext_lazy as _
@@ -109,9 +110,8 @@ class Document(models.Model):
def __str__(self): def __str__(self):
return self.label return self.label
@models.permalink
def get_absolute_url(self): def get_absolute_url(self):
return ('documents:document_preview', [self.pk]) reverse('documents:document_preview', args=[self.pk])
def save(self, *args, **kwargs): def save(self, *args, **kwargs):
user = kwargs.pop('user', None) user = kwargs.pop('user', None)
@@ -513,9 +513,8 @@ class DocumentPage(models.Model):
verbose_name = _('Document page') verbose_name = _('Document page')
verbose_name_plural = _('Document pages') verbose_name_plural = _('Document pages')
@models.permalink
def get_absolute_url(self): def get_absolute_url(self):
return ('documents:document_page_view', [self.pk]) reverse('documents:document_page_view', args=[self.pk])
@property @property
def siblings(self): def siblings(self):

View File

@@ -1,6 +1,7 @@
from __future__ import unicode_literals from __future__ import unicode_literals
from django.contrib.auth.models import User from django.contrib.auth.models import User
from django.core.urlresolvers import reverse
from django.db import models from django.db import models
from django.utils.encoding import python_2_unicode_compatible from django.utils.encoding import python_2_unicode_compatible
from django.utils.translation import ugettext_lazy as _ from django.utils.translation import ugettext_lazy as _
@@ -18,9 +19,8 @@ class Folder(models.Model):
def __str__(self): def __str__(self):
return self.title return self.title
@models.permalink
def get_absolute_url(self): def get_absolute_url(self):
return ('folders:folder_view', [self.pk]) reverse('folders:folder_view', args=[self.pk])
class Meta: class Meta:
unique_together = ('title', 'user') unique_together = ('title', 'user')

View File

@@ -6,6 +6,7 @@ from django.contrib.auth.models import User
from django.contrib.contenttypes import generic from django.contrib.contenttypes import generic
from django.contrib.contenttypes.models import ContentType from django.contrib.contenttypes.models import ContentType
from django.core.exceptions import PermissionDenied from django.core.exceptions import PermissionDenied
from django.core.urlresolvers import reverse
from django.db import models from django.db import models
from django.utils.encoding import python_2_unicode_compatible from django.utils.encoding import python_2_unicode_compatible
from django.utils.translation import ugettext from django.utils.translation import ugettext
@@ -224,9 +225,8 @@ class Role(models.Model):
def __str__(self): def __str__(self):
return self.label return self.label
@models.permalink
def get_absolute_url(self): def get_absolute_url(self):
return ('permissions:role_list',) reverse('permissions:role_list')
def add_member(self, member): def add_member(self, member):
member = AnonymousUserSingleton.objects.passthru_check(member) member = AnonymousUserSingleton.objects.passthru_check(member)