Compatibility up to Django 2.1, dropped < 1.8. Removed unused code, fixed tests.

[django-cas-provider.git] / cas_provider / models.py

diff --git a/cas_provider/models.py b/cas_provider/models.py
index 12fed4c..0dd8b9a 100644 (file)
--- a/cas_provider/models.py
+++ b/cas_provider/models.py
@@ -1,3 +1,5 @@
+from __future__ import unicode_literals
+

 from django.conf import settings
 from django.db import models
 from django.utils.translation import ugettext_lazy as _
 from django.conf import settings
 from django.db import models
 from django.utils.translation import ugettext_lazy as _


@@ -8,10 +10,7 @@ try:
 except ImportError:
     from urllib import urlencode
     from urlparse import urlparse, ParseResult
 except ImportError:
     from urllib import urlencode
     from urlparse import urlparse, ParseResult

-    try:
-        from urlparse import parse_qs
-    except: # Python <2.6 compatibility
-        from cgi import parse_qs
+    from urlparse import parse_qs

 
 __all__ = ['ServiceTicket', 'LoginTicket', 'ProxyGrantingTicket', 'ProxyTicket', 'ProxyGrantingTicketIOU']
 
 
 __all__ = ['ServiceTicket', 'LoginTicket', 'ProxyGrantingTicket', 'ProxyTicket', 'ProxyGrantingTicketIOU']
 


@@ -32,11 +31,11 @@ class BaseTicket(models.Model):
 
     def _generate_ticket(self, length=ticket.max_length, chars=string.ascii_letters + string.digits):
         """ Generates a random string of the requested length. Used for creation of tickets. """
 
     def _generate_ticket(self, length=ticket.max_length, chars=string.ascii_letters + string.digits):
         """ Generates a random string of the requested length. Used for creation of tickets. """

-        return u"%s-%s" % (self.prefix, ''.join(Random().sample(chars, length - (len(self.prefix) + 1))))
+        return "%s-%s" % (self.prefix, ''.join(Random().sample(chars, length - (len(self.prefix) + 1))))

 
 
 class ServiceTicket(BaseTicket):
 
 
 class ServiceTicket(BaseTicket):

-    user = models.ForeignKey(settings.AUTH_USER_MODEL, verbose_name=_('user'))
+    user = models.ForeignKey(settings.AUTH_USER_MODEL, verbose_name=_('user'), on_delete=models.CASCADE)

     service = models.URLField(_('service'))
 
     prefix = 'ST'
     service = models.URLField(_('service'))
 
     prefix = 'ST'


@@ -65,15 +64,15 @@ class LoginTicket(BaseTicket):
 
 
 class ProxyGrantingTicket(BaseTicket):
 
 
 class ProxyGrantingTicket(BaseTicket):

-    user = models.ForeignKey(settings.AUTH_USER_MODEL, verbose_name=_('user'))
+    user = models.ForeignKey(settings.AUTH_USER_MODEL, verbose_name=_('user'), on_delete=models.CASCADE)

     service = models.URLField(_('service'), null=True)
     service = models.URLField(_('service'), null=True)

-    pgt = models.ForeignKey('self', null=True)
+    pgt = models.ForeignKey('self', null=True, on_delete=models.CASCADE)

     pgtiou = models.CharField(max_length=256, verbose_name=_('PGTiou'))
     prefix = 'PGT'
 
     def __init__(self, *args, **kwargs):
         if 'pgtiou' not in kwargs:
     pgtiou = models.CharField(max_length=256, verbose_name=_('PGTiou'))
     prefix = 'PGT'
 
     def __init__(self, *args, **kwargs):
         if 'pgtiou' not in kwargs:

-            kwargs['pgtiou'] = u"PGTIOU-%s" % (''.join(Random().sample(string.ascii_letters + string.digits, 50)))
+            kwargs['pgtiou'] = "PGTIOU-%s" % (''.join(Random().sample(string.ascii_letters + string.digits, 50)))

         super(ProxyGrantingTicket, self).__init__(*args, **kwargs)
 
     class Meta:
         super(ProxyGrantingTicket, self).__init__(*args, **kwargs)
 
     class Meta:


@@ -82,7 +81,7 @@ class ProxyGrantingTicket(BaseTicket):
 
 
 class ProxyTicket(ServiceTicket):
 
 
 class ProxyTicket(ServiceTicket):

-    proxyGrantingTicket = models.ForeignKey(ProxyGrantingTicket, verbose_name=_('Proxy Granting Ticket'))
+    proxyGrantingTicket = models.ForeignKey(ProxyGrantingTicket, verbose_name=_('Proxy Granting Ticket'), on_delete=models.CASCADE)

 
     prefix = 'PT'
 
 
     prefix = 'PT'
 


@@ -92,7 +91,7 @@ class ProxyTicket(ServiceTicket):
 
 
 class ProxyGrantingTicketIOU(BaseTicket):
 
 
 class ProxyGrantingTicketIOU(BaseTicket):

-    proxyGrantingTicket = models.ForeignKey(ProxyGrantingTicket, verbose_name=_('Proxy Granting Ticket'))
+    proxyGrantingTicket = models.ForeignKey(ProxyGrantingTicket, verbose_name=_('Proxy Granting Ticket'), on_delete=models.CASCADE)

 
     prefix = 'PGTIOU'
 
 
     prefix = 'PGTIOU'