Ticket #1296: slugfield.patch

django/contrib/admin/views/doc.py

@@ -260 +260 @@
     'PhoneNumberField'          : _('Phone number'),
     'PositiveIntegerField'      : _('Integer'),
     'PositiveSmallIntegerField' : _('Integer'),
-    'SlugField'                 : _('String (up to 50)'),
+    'SlugField'                 : _('String (up to %(maxlength)s)'),
     'SmallIntegerField'         : _('Integer'),
     'TextField'                 : _('Text'),
     'TimeField'                 : _('Time'),

django/core/db/backends/ado_mssql.py

@@ -153 +153 @@
     'PhoneNumberField':  'varchar(20)',
     'PositiveIntegerField': 'int CONSTRAINT [CK_int_pos_%(column)s] CHECK ([%(column)s] > 0)',
     'PositiveSmallIntegerField': 'smallint CONSTRAINT [CK_smallint_pos_%(column)s] CHECK ([%(column)s] > 0)',
-    'SlugField':         'varchar(50)',
+    'SlugField':         'varchar(%(maxlength)s)',
     'SmallIntegerField': 'smallint',
     'TextField':         'text',
     'TimeField':         'time',

django/core/db/backends/postgresql.py

@@ -176 +176 @@
     'PhoneNumberField':  'varchar(20)',
     'PositiveIntegerField': 'integer CHECK ("%(column)s" >= 0)',
     'PositiveSmallIntegerField': 'smallint CHECK ("%(column)s" >= 0)',
-    'SlugField':         'varchar(50)',
+    'SlugField':         'varchar(%(maxlength)s)',
     'SmallIntegerField': 'smallint',
     'TextField':         'text',
     'TimeField':         'time',

django/core/db/backends/sqlite3.py

@@ -177 +177 @@
     'PhoneNumberField':             'varchar(20)',
     'PositiveIntegerField':         'integer unsigned',
     'PositiveSmallIntegerField':    'smallint unsigned',
-    'SlugField':                    'varchar(50)',
+    'SlugField':                    'varchar(%(maxlength)s)',
     'SmallIntegerField':            'smallint',
     'TextField':                    'text',
     'TimeField':                    'time',

django/core/db/backends/mysql.py

@@ -174 +174 @@
     'PhoneNumberField':  'varchar(20)',
     'PositiveIntegerField': 'integer UNSIGNED',
     'PositiveSmallIntegerField': 'smallint UNSIGNED',
-    'SlugField':         'varchar(50)',
+    'SlugField':         'varchar(%(maxlength)s)',
     'SmallIntegerField': 'smallint',
     'TextField':         'longtext',
     'TimeField':         'time',

django/core/meta/fields.py

@@ -604 +604 @@
 
 class SlugField(Field):
     def __init__(self, *args, **kwargs):
-        kwargs['maxlength'] = 50
+        # Default to a maxlength of 50 but allow overrides
+        if not kwargs.has_key('maxlength'):
+            kwargs['maxlength'] = 50
         kwargs.setdefault('validator_list', []).append(validators.isSlug)
         # Set db_index=True unless it's been set manually.
         if not kwargs.has_key('db_index'):

docs/model-api.txt

@@ -409 +409 @@
     "Slug" is a newspaper term. A slug is a short label for something,
     containing only letters, numbers, underscores or hyphens. They're generally
     used in URLs.
+    
+    If ``maxlength`` is specified, it will override the default length of 50. 
 
-    Implies ``maxlength=50`` and ``db_index=True``.
+    Implies ``db_index=True``.
 
     Accepts an extra option, ``prepopulate_from``, which is a list of fields
     from which to auto-populate the slug, via JavaScript, in the object's admin