[Fixes #11703] Documents can now be uploaded without specifying title via the REST API.

geonode/documents/api/serializers.py

@@ -19,6 +19,7 @@
 import logging
 
 from dynamic_rest.fields.fields import DynamicComputedField
+from rest_framework import serializers
 from geonode.base.api.serializers import ResourceBaseSerializer
 from geonode.documents.models import Document
 
@@ -39,6 +40,7 @@ class DocumentSerializer(ResourceBaseSerializer):
     def __init__(self, *args, **kwargs):
         # Instantiate the superclass normally
         super().__init__(*args, **kwargs)
+        self.fields["title"] = serializers.CharField(required=False)
 
     file_path = GeonodeFilePathField(required=False)
     doc_file = DocumentFieldField(required=False)

geonode/documents/models.py

@@ -79,7 +79,8 @@ def compact_permission_labels(cls):
     @property
     def name(self):
         if not self.title:
-            return str(self.id)
+            if len(self.files) > 0:
+                return self.files[0].split("/")[-1]
         else:
             return self.title
 

geonode/documents/tests.py

@@ -383,6 +383,23 @@ def test_document_isuploaded(self):
         )
         self.assertEqual(response.status_code, 200)
 
+    def test_document_upload_without_title(self):
+        """/documents/upload -> Test uploading a document without a title"""
+
+        f = SimpleUploadedFile("test_img_file.gif", self.imgfile.read(), "image/gif")
+
+        self.client.login(username="admin", password="admin")
+        response = self.client.post(
+            f"{reverse('document_upload')}?no__redirect=true",
+            data={
+                "doc_file": f,
+                "type": "document",
+            },
+        )
+        self.assertEqual(response.status_code, 200)
+        d = Document.objects.get(title="test_img_file.gif")
+        self.assertIsNotNone(d)
+
     # Permissions Tests
 
     def test_set_document_permissions(self):