Moving Crop clean method into model, not the form

cropduster/models.py

@@ -9,7 +9,9 @@
 
 CROPDUSTER_UPLOAD_PATH = getattr(settings, "CROPDUSTER_UPLOAD_PATH", settings.MEDIA_ROOT)
 
-IMAGE_SAVE_PARAMS =  {"quality" :95}
+IMAGE_SAVE_PARAMS =  {
+	"quality" :95
+}
 
 MANUALLY_CROP = 0
 AUTO_CROP = 1
@@ -180,6 +182,16 @@ def save(self, *args, **kwargs):
 				# loop through the other sizes of the same aspect ratio, and create those crops
 				for size in sizes:
 					self.image.rescale(cropped_image, size=size)
+	def clean(self):
+
+		if not self.crop_x or not self.crop_y:
+			raise ValidationError("Missing crop values")
+
+		if self.crop_x < 0 or self.crop_y < 0:
+			raise ValidationError("Crop positions must be non-negative")
+
+		if self.crop_w <= 0 or self.crop_h <= 0:
+			raise ValidationError("Crop measurements must be greater than zero")
 
 
 class Image(CachingMixin, models.Model):

cropduster/views.py

@@ -37,23 +37,7 @@ class Meta:
 		widgets = {
 			"image": TextInput(),
 		}
-	def clean(self):
-
-		cleaned_data = super(CropForm, self).clean()
-
-		if "crop_x" not in cleaned_data or "crop_y" not in cleaned_data:
-			self._errors.clear()
-			raise ValidationError("Missing crop values")
 
-		if cleaned_data["crop_x"] < 0 or cleaned_data["crop_y"] < 0:
-			self._errors.clear()
-			raise ValidationError("Crop positions must be non-negative")
-
-		if cleaned_data["crop_w"] <= 0 or cleaned_data["crop_h"] <= 0:
-			self._errors.clear()
-			raise ValidationError("Crop measurements must be greater than zero")
-
-		return self.cleaned_data
 
 @csrf_exempt
 def upload(request):