Ticket #8440: auto_id.diff

forms/formsets.py

@@ -28 +28 @@
     """
     A collection of instances of the same Form class.
     """
-    def __init__(self, data=None, files=None, auto_id='id_%s', prefix=None,
+    def __init__(self, data=None, files=None, auto_id='id-%s', prefix=None,
                  initial=None, error_class=ErrorList):
         self.is_bound = data is not None or files is not None
         self.prefix = prefix or 'form'

forms/models.py

@@ -184 +184 @@
         return new_class
 
 class BaseModelForm(BaseForm):
-    def __init__(self, data=None, files=None, auto_id='id_%s', prefix=None,
+    def __init__(self, data=None, files=None, auto_id='id-%s', prefix=None,
                  initial=None, error_class=ErrorList, label_suffix=':',
                  empty_permitted=False, instance=None):
         opts = self._meta
@@ -240 +240 @@
     """
     model = None
 
-    def __init__(self, data=None, files=None, auto_id='id_%s', prefix=None,
+    def __init__(self, data=None, files=None, auto_id='id-%s', prefix=None,
                  queryset=None, **kwargs):
         self.queryset = queryset
         defaults = {'data': data, 'files': files, 'auto_id': auto_id, 'prefix': prefix}

forms/forms.py

@@ -68 +68 @@
     # class is different than Form. See the comments by the Form class for more
     # information. Any improvements to the form API should be made to *this*
     # class, not to the Form class.
-    def __init__(self, data=None, files=None, auto_id='id_%s', prefix=None,
+    def __init__(self, data=None, files=None, auto_id='id-%s', prefix=None,
                  initial=None, error_class=ErrorList, label_suffix=':',
                  empty_permitted=False):
         self.is_bound = data is not None or files is not None