From dbcd2fe985498889e1206f54c196f00879e79dae Mon Sep 17 00:00:00 2001 From: Luke Plant Date: Thu, 1 Jun 2006 20:47:34 +0000 Subject: [PATCH] Fixed #2045 - TypeError thrown if a form does not have the correct enctype for uploading files. It throws a ValidationError now, as it should. git-svn-id: http://code.djangoproject.com/svn/django/trunk@3048 bcc190cf-cafb-0310-a4f2-bffc1f526a37 --- django/core/validators.py | 14 +++++++++++--- django/forms/__init__.py | 6 +++++- 2 files changed, 16 insertions(+), 4 deletions(-) diff --git a/django/core/validators.py b/django/core/validators.py index 69c8e0f701..27505b7d5a 100644 --- a/django/core/validators.py +++ b/django/core/validators.py @@ -146,7 +146,11 @@ def isValidImage(field_data, all_data): from PIL import Image from cStringIO import StringIO try: - Image.open(StringIO(field_data['content'])) + content = field_data['content'] + except TypeError: + raise ValidationError, gettext("No file was submitted. Check the encoding type on the form.") + try: + Image.open(StringIO(content)) except IOError: # Python Imaging Library doesn't recognize it as an image raise ValidationError, gettext("Upload a valid image. The file you uploaded was either not an image or a corrupted image.") @@ -366,9 +370,13 @@ class HasAllowableSize: self.max_error_message = max_error_message or lazy_inter(gettext_lazy("Make sure your uploaded file is at most %s bytes big."), max_size) def __call__(self, field_data, all_data): - if self.min_size is not None and len(field_data['content']) < self.min_size: + try: + content = field_data['content'] + except TypeError: + raise ValidationError, gettext_lazy("No file was submitted. Check the encoding type on the form.") + if self.min_size is not None and len(content) < self.min_size: raise ValidationError, self.min_error_message - if self.max_size is not None and len(field_data['content']) > self.max_size: + if self.max_size is not None and len(content) > self.max_size: raise ValidationError, self.max_error_message class MatchesRegularExpression: diff --git a/django/forms/__init__.py b/django/forms/__init__.py index 52092aff7b..b67e1d0f84 100644 --- a/django/forms/__init__.py +++ b/django/forms/__init__.py @@ -641,7 +641,11 @@ class FileUploadField(FormField): self.validator_list = [self.isNonEmptyFile] + validator_list def isNonEmptyFile(self, field_data, all_data): - if not field_data['content']: + try: + content = field_data['content'] + except TypeError: + raise validators.CriticalValidationError, gettext("No file was submitted. Check the encoding type on the form.") + if not content: raise validators.CriticalValidationError, gettext("The submitted file is empty.") def render(self, data):