Require spec id for form fields

Closes #163
matthewwithanm · Dec 1, 2012 · 54baa44 · 54baa44
1 parent 4ead0b3
commit 54baa44
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/imagekit/forms/fields.py b/imagekit/forms/fields.py
@@ -5,10 +5,13 @@
 class ProcessedImageField(ImageField, SpecHost):
 
     def __init__(self, processors=None, format=None, options=None,
-                 autoconvert=True, spec=None, spec_id=None, *args, **kwargs):
+                 autoconvert=True, spec_id=None, spec=None, *args, **kwargs):
 
         if spec_id is None:
-            spec_id = '??????'  # FIXME: Wher should we get this?
+            # Unlike model fields, form fields are never told their field name.
+            # (Model fields are done so via `contribute_to_class()`.) Therefore
+            # we can't really generate a good spec id automatically.
+            raise TypeError('You must provide a spec_id')
 
         SpecHost.__init__(self, processors=processors, format=format,
                           options=options, autoconvert=autoconvert, spec=spec,