[Fixes #11579] Use autocomplete API for editing linked resources

geonode/base/forms.py

@@ -20,9 +20,11 @@
 import html
 import json
 import logging
+
 from django.db.models.query import QuerySet
 from bootstrap3_datetime.widgets import DateTimePicker
 from dal import autocomplete
+import dal.forward
 from django import forms
 from django.conf import settings
 from django.contrib.auth import get_user_model
@@ -347,35 +349,45 @@ def _get_thesauro_title_label(item, lang):
 
 
 class LinkedResourceForm(forms.ModelForm):
-    linked_resources = forms.MultipleChoiceField(label=_("Link to"), required=False)
+    linked_resources = forms.ModelMultipleChoiceField(
+        label=_("Related resources"),
+        required=False,
+        queryset=None,
+        widget=autocomplete.ModelSelect2Multiple(url="autocomplete_linked_resource"),
+    )
 
     def __init__(self, *args, **kwargs):
         super().__init__(*args, **kwargs)
-        self.fields["linked_resources"].choices = self.generate_link_choices()
-        self.fields["linked_resources"].initial = LinkedResource.get_target_ids(self.instance)
+
+        # this is used to automatically validate the POSTed back values
+        self.fields["linked_resources"].queryset = ResourceBase.objects.exclude(pk=self.instance.id)
+        # these are the LinkedResource already linked to this resource
+        self.fields["linked_resources"].initial = LinkedResource.get_target_ids(self.instance).all()
+        # this is used by the autocomplete view to exclude current resource
+        self.fields["linked_resources"].widget.forward.append(
+            dal.forward.Const(
+                self.instance.id,
+                "exclude",
+            )
+        )
 
     class Meta:
         model = ResourceBase
         fields = ["linked_resources"]
 
-    def generate_link_choices(self, resources=None):
-        if resources is None:
-            resources = ResourceBase.objects.exclude(pk=self.instance.id).order_by("title")
-
-        return [[obj.id, f"{obj.title} ({obj.polymorphic_ctype.model})"] for obj in resources]
-
     def save_linked_resources(self, links_field="linked_resources"):
         # create and fetch desired links
         target_ids = []
-        for res_id in self.cleaned_data[links_field]:
-            linked, _ = LinkedResource.objects.get_or_create(source=self.instance, target_id=res_id, internal=False)
-            target_ids.append(res_id)
+        for res in self.cleaned_data[links_field]:
+            linked, _ = LinkedResource.objects.get_or_create(source=self.instance, target=res, internal=False)
+            target_ids.append(res.pk)
 
         # delete remaining links
-        # DocumentResourceLink.objects.filter(document_id=self.instance.id).exclude(
-        #     pk__in=[i.pk for i in instances]
-        # ).delete()
-        (LinkedResource.objects.filter(source_id=self.instance.id).exclude(target_id__in=target_ids).delete())
+        (
+            LinkedResource.objects.filter(source_id=self.instance.id, internal=False)
+            .exclude(target_id__in=target_ids)
+            .delete()
+        )
 
 
 class ResourceBaseDateTimePicker(DateTimePicker):

geonode/base/urls.py

@@ -27,6 +27,7 @@
     ResourceBaseAutocomplete,
     HierarchicalKeywordAutocomplete,
     ThesaurusKeywordLabelAutocomplete,
+    LinkedResourcesAutocomplete,
 )
 
 
@@ -36,6 +37,11 @@
         ResourceBaseAutocomplete.as_view(),
         name="autocomplete_base",
     ),
+    url(
+        r"^autocomplete_linked_resource/$",
+        LinkedResourcesAutocomplete.as_view(),
+        name="autocomplete_linked_resource",
+    ),
     url(
         r"^autocomplete_region/$",
         RegionAutocomplete.as_view(),

geonode/base/views.py

@@ -282,6 +282,28 @@ def get_queryset(self):
         )[:100]
 
 
+class LinkedResourcesAutocomplete(autocomplete.Select2QuerySetView):
+    def get_queryset(self):
+        qs = ResourceBase.objects.order_by("title")
+
+        if self.q:
+            qs = qs.filter(title__icontains=self.q)
+
+        if self.forwarded and "exclude" in self.forwarded:
+            qs = qs.exclude(pk=self.forwarded["exclude"])
+
+        return get_visible_resources(
+            qs,
+            self.request.user if self.request else None,
+            admin_approval_required=settings.ADMIN_MODERATE_UPLOADS,
+            unpublished_not_visible=settings.RESOURCE_PUBLISHING,
+            private_groups_not_visibile=settings.GROUP_PRIVATE_RESOURCES,
+        )[:10]
+
+    def get_result_label(self, result):
+        return f"{result.title} [{_(result.polymorphic_ctype.model)}]"
+
+
 class RegionAutocomplete(SimpleSelect2View):
     model = Region
     filter_arg = "name__icontains"

geonode/documents/templates/documents/document_metadata.html

@@ -84,19 +84,3 @@ <h2>{% trans "Metadata Provider" %}</h2>
   </div>
 </div>
 {% endblock body_outer %}
-
-{% block extra_script %}
-{{ block.super }}
-<script type="text/javascript">
-    $("#id_resource-linked_resources").select2({
-        placeholder: "{% trans "Select an option" %}",
-        allowClear: true
-    });
-</script>
-<style>
-  #s2id_id_resource-linked_resources {
-    width: 600px;
-    height: 100%;
-  }
-</style>
-{% endblock extra_script %}

geonode/documents/templates/layouts/doc_panels.html

@@ -315,7 +315,7 @@
                                 {% endblock doc_title %}
                                 {% block doc_linked_resources %}
                                     <div id="req_item">                                  
-                                      <span><label for="{{ document_form.linked_resources|id }}">{{ document_form.linked_resources.label }}</label></span>
+                                      <span><label for="{{ document_form.linked_resources|id }}">{% trans "Related resources" %}</label></span>
                                       {{ document_form.linked_resources }}
                                     </div>
                                 {% endblock doc_linked_resources %}

geonode/documents/tests.py

@@ -655,9 +655,9 @@ def test_create_document_with_links(self):
 
         mixin1 = LinkedResourceForm()
         mixin1.instance = d
-        mixin1.cleaned_data = dict(
-            linked_resources=[r.id for r in resources],
-        )
+        mixin1.cleaned_data = {
+            "linked_resources": resources,
+        }
         mixin1.save_linked_resources()
 
         for resource in resources:
@@ -668,9 +668,9 @@ def test_create_document_with_links(self):
 
         mixin2 = LinkedResourceForm()
         mixin2.instance = d
-        mixin2.cleaned_data = dict(
-            linked_resources=[r.id for r in layers],
-        )
+        mixin2.cleaned_data = {
+            "linked_resources": layers,
+        }
         mixin2.save_linked_resources()
 
         for resource in layers:

geonode/geoapps/templates/apps/app_metadata.html

@@ -85,19 +85,3 @@ <h2>{% trans "Metadata Provider" %}</h2>
 
 {{ block.super }}
 {% endblock body_outer %}
-
-{% block extra_script %}
-{{ block.super }}
-<script type="text/javascript">
-    $("#id_resource-linked_resources").select2({
-        placeholder: "{% trans "Select an option" %}",
-        allowClear: true
-    });
-</script>
-<style>
-  #s2id_id_resource-linked_resources {
-    width: 600px;
-    height: 100%;
-  }
-</style>
-{% endblock extra_script %}

geonode/geoapps/templates/layouts/app_panels.html

@@ -299,7 +299,7 @@
                                     {% endblock %}
                                     {% block geoapp_linked_resources %}
                                         <div id="req_item">
-                                          <span><label for="{{ geoapp_form.linked_resources|id }}">{{ geoapp_form.linked_resources.label }}</label></span>
+                                          <span><label for="{{ geoapp_form.linked_resources|id }}">{% trans "Related resources" %}</label></span>
                                           {{ geoapp_form.linked_resources }}
                                         </div>
                                     {% endblock geoapp_linked_resources %}

geonode/layers/templates/datasets/dataset_metadata.html

@@ -109,19 +109,3 @@ <h2>{% trans "Metadata Provider" %}</h2>
 
 {{ block.super }}
 {% endblock body_outer %}
-
-{% block extra_script %}
-{{ block.super }}
-<script type="text/javascript">
-    $("#id_resource-linked_resources").select2({
-        placeholder: "{% trans "Select an option" %}",
-        allowClear: true
-    });
-</script>
-<style>
-  #s2id_id_resource-linked_resources {
-    width: 600px;
-    height: 100%;
-  }
-</style>
-{% endblock extra_script %}

geonode/layers/templates/layouts/panels.html

@@ -333,7 +333,7 @@
                                     {% endblock dataset_title %}
                                     {% block dataset_linked_resources %}
                                         <div id="req_item">
-                                          <span><label for="{{ dataset_form.linked_resources|id }}">{{ dataset_form.linked_resources.label }}</label></span>
+                                          <span><label for="{{ dataset_form.linked_resources|id }}">{% trans "Related resources" %}</label></span>
                                           {{ dataset_form.linked_resources }}
                                         </div>
                                     {% endblock dataset_linked_resources %}

geonode/locale/it/LC_MESSAGES/django.po

@@ -39,7 +39,7 @@ msgstr ""
 "Project-Id-Version: GeoNode\n"
 "Report-Msgid-Bugs-To: \n"
 "POT-Creation-Date: 2021-07-08 09:02+0300\n"
-"PO-Revision-Date: 2023-07-20 11:35+0200\n"
+"PO-Revision-Date: 2023-10-11 11:47+0200\n"
 "Last-Translator: Julien Collaer <[email protected]>\n"
 "Language-Team: Italian (http://www.transifex.com/geonode/geonode/language/it/)\n"
 "Language: it\n"
@@ -947,6 +947,9 @@ msgstr "Classifica velocità"
 msgid "Link to"
 msgstr "Collegamento a"
 
+msgid "Related resources"
+msgstr "Risorse correlate"
+
 msgid "File"
 msgstr "File"
 
@@ -3283,6 +3286,11 @@ msgid_plural "Maps"
 msgstr[0] "Mappa"
 msgstr[1] "Mappe"
 
+msgid "map"
+msgid_plural "maps"
+msgstr[0] "mappa"
+msgstr[1] "mappe"
+
 msgid "Map Layers"
 msgstr "Livelli della mappa"
 

geonode/maps/templates/layouts/map_panels.html

@@ -319,7 +319,7 @@
                                   {% endblock map_title %}
                                   {% block map_linked_resources %}
                                     <div id="req_item">
-                                      <span><label for="{{ map_form.linked_resources|id }}">{{ map_form.linked_resources.label }}</label></span>
+                                      <span><label for="{{ map_form.linked_resources|id }}">{% trans "Related resources" %}</label></span>
                                       {{ map_form.linked_resources }}
                                     </div>
                                   {% endblock map_linked_resources %}

geonode/maps/templates/maps/map_metadata.html

@@ -87,19 +87,3 @@ <h2>{% trans "Metadata Provider" %}</h2>
 
 {{ block.super }}
 {% endblock body_outer %}
-
-{% block extra_script %}
-{{ block.super }}
-<script type="text/javascript">
-    $("#id_resource-linked_resources").select2({
-        placeholder: "{% trans "Select an option" %}",
-        allowClear: true
-    });
-</script>
-<style>
-  #s2id_id_resource-linked_resources {
-    width: 600px;
-    height: 100%;
-  }
-</style>
-{% endblock extra_script %}