Revert "Revert "Merge pull request #1760 from laws-africa/source-file""

This reverts commit 9e18d6b.

peachjam/admin.py

@@ -322,11 +322,6 @@ def clean_content_html(self):
             return self.instance.content_html
         return self.cleaned_data["content_html"]
 
-    def _save_m2m(self):
-        super()._save_m2m()
-        # update document text
-        self.instance.update_text_content()
-
 
 class AttachedFilesInline(BaseAttachmentFileInline):
     model = AttachedFiles
@@ -504,7 +499,9 @@ def save_related(self, request, form, formsets, change):
             cp.queue_re_extract_citations(form.instance.date)
 
         super().save_related(request, form, formsets, change)
+
         form.instance.save()
+        form.instance.update_text_content()
 
     def get_urls(self):
         return [

peachjam/forms.py

@@ -188,6 +188,17 @@ def _save_m2m(self):
                 self.instance.file_as_pdf.delete()
                 self.instance.ensure_file_as_pdf()
 
+    def clean(self):
+        cleaned_data = super().clean()
+        if "file" in self.changed_data:
+            # clear all the fields that are derived from the file
+            self.instance.mimetype = None
+            self.instance.size = None
+            self.instance.file_as_pdf = None
+            self.instance.filename = None
+
+        return cleaned_data
+
 
 class AttachedFilesForm(AttachmentFormMixin, forms.ModelForm):
     class Meta: