Fix #953

HISTORY.rst

@@ -15,7 +15,9 @@ Historia zmian
 * wyświetlaj link do PubMedCentral dla prac z PMC ID (#959),
 * poprawki pobierania PubMed ID (#958),
 * poprawne zawężanie do zakresu punktów PK (#967),
-* katalog cache ma nazwę z numerem wersji (#961).
+* katalog cache ma nazwę z numerem wersji (#961),
+* w multiwyszukiwarce w polu "Wydawnictwo nadrzędne" pokazuj wyłącznie rekordy
+  będące już wydawnictwami nadrzędnymi dla rekordów (#953).
 
 202101.53
 ---------

src/bpp/tests/test_autocomplete.py

@@ -5,7 +5,7 @@
 from django.urls import reverse
 from model_mommy import mommy
 
-from bpp.models import Autor_Dyscyplina
+from bpp.models import Autor_Dyscyplina, Wydawnictwo_Zwarte
 from bpp.models.autor import Autor
 from bpp.models.konferencja import Konferencja
 from bpp.views.autocomplete import (
@@ -150,9 +150,22 @@ def test_wydawca_autocomplete(admin_client):
 
 def test_wydawnictwo_nadrzedne_autocomplete(admin_client):
     admin_client.get(reverse("bpp:wydawnictwo-nadrzedne-autocomplete") + "?q=test")
-    admin_client.get(
+
+
+def test_publicwydawnictwo_nadrzedne_autocomplete(admin_client, ksiazka):
+    ksiazka.tytul_oryginalny = "test 123"
+    ksiazka.save()
+
+    res = admin_client.get(
+        reverse("bpp:public-wydawnictwo-nadrzedne-autocomplete") + "?q=test"
+    )
+    assert not json.loads(res.content)["results"]
+
+    mommy.make(Wydawnictwo_Zwarte, wydawnictwo_nadrzedne=ksiazka)
+    res = admin_client.get(
         reverse("bpp:public-wydawnictwo-nadrzedne-autocomplete") + "?q=test"
     )
+    assert len(json.loads(res.content)["results"]) == 1
 
 
 def test_tsquery_bug1(admin_client):

src/bpp/tests/test_views/test_oai.py

@@ -1,9 +1,8 @@
 # -*- encoding: utf-8 -*-
-import pytest
-from django.urls.base import reverse
 import xml.etree.ElementTree as ET
 
-from bpp.models import Rekord
+import pytest
+from django.urls.base import reverse
 
 
 @pytest.mark.django_db
@@ -20,20 +19,6 @@ def test_identify(wydawnictwo_ciagle, client):
     assert res.status_code == 200
 
 
-@pytest.fixture
-def ksiazka(wydawnictwo_zwarte, ksiazka_polska) -> "Wydawnictwo_Zwarte":
-    wydawnictwo_zwarte.charakter_formalny = ksiazka_polska
-    wydawnictwo_zwarte.save()
-    return wydawnictwo_zwarte
-
-
-@pytest.fixture
-def artykul(wydawnictwo_ciagle, artykul_w_czasopismie):
-    wydawnictwo_ciagle.charakter_formalny = artykul_w_czasopismie
-    wydawnictwo_ciagle.save()
-    return wydawnictwo_ciagle
-
-
 def toXML(response):
     return ET.fromstring(response.content.decode("utf-8"))
 
@@ -42,7 +27,7 @@ def test_listRecords(ksiazka, client):
     listRecords = reverse("bpp:oai") + "?verb=ListRecords&metadataPrefix=oai_dc"
     res = client.get(listRecords)
 
-    responseXml = ET.fromstring(res.content.decode("utf-8"))
+    ET.fromstring(res.content.decode("utf-8"))
     assert "Tytul Wydawnictwo" in toXML(res)[2][0][1][0][1].text
 
 

src/bpp/urls.py

@@ -347,7 +347,7 @@
         name="wydawnictwo-nadrzedne-autocomplete",
     ),
     url(
-        r"^wydawnictwo-nadrzedne-autocomplete/$",
+        r"^public-wydawnictwo-nadrzedne-autocomplete/$",
         PublicWydawnictwo_NadrzedneAutocomplete.as_view(),
         name="public-wydawnictwo-nadrzedne-autocomplete",
     ),

src/bpp/views/autocomplete.py

@@ -53,6 +53,24 @@ def get_queryset(self):
 class PublicWydawnictwo_NadrzedneAutocomplete(Wydawnictwo_NadrzedneAutocomplete):
     create_field = None
 
+    def get_queryset(self):
+        """
+        :test: :py:class:`bpp.tests.test_autocomplete`
+        """
+
+        # Publiczna wyszukiwarka dla wydawnictw nadrzędnych powinna wyszukiwać wyłącznie rekordy,
+        # które są już wydawnictwami nadrzędnymi dla jakichś rekordów:
+
+        qs = Wydawnictwo_Zwarte.objects.filter(
+            pk__in=Wydawnictwo_Zwarte.objects.exclude(wydawnictwo_nadrzedne_id=None)
+            .values_list("wydawnictwo_nadrzedne_id")
+            .distinct()
+        )
+
+        if self.q:
+            qs = qs.filter(tytul_oryginalny__icontains=self.q)
+        return qs
+
 
 class JednostkaMixin:
     def get_result_label(self, result):

src/conftest.py

@@ -750,3 +750,17 @@ def wydawnictwo_zwarte_w_trakcie_korekty(statusy_korekt):
 @pytest.fixture
 def wydawnictwo_zwarte_po_korekcie(statusy_korekt):
     return mommy.make(Wydawnictwo_Zwarte, status_korekty=statusy_korekt["po korekcie"])
+
+
+@pytest.fixture
+def ksiazka(wydawnictwo_zwarte, ksiazka_polska) -> "Wydawnictwo_Zwarte":
+    wydawnictwo_zwarte.charakter_formalny = ksiazka_polska
+    wydawnictwo_zwarte.save()
+    return wydawnictwo_zwarte
+
+
+@pytest.fixture
+def artykul(wydawnictwo_ciagle, artykul_w_czasopismie):
+    wydawnictwo_ciagle.charakter_formalny = artykul_w_czasopismie
+    wydawnictwo_ciagle.save()
+    return wydawnictwo_ciagle