CHANGELOG.md

@@ -1,15 +0,0 @@
-## Version 0.4.0
-
-Version 0.4.0 has added support for search-as-you-type when searching for animals to adopt. Furthermore, the display of
-maps in the search has been majorly improved.
-
-Photon has been added as geocoding source option which allows to use this functionality.
-
-Further improvements include the representation of rescue organizations and tooltips.
-
-One of the biggest features is the addition of search subscriptions. These allow you to not only
-search for currently active adoption notices but to subscribe to that search so that you get notified if there are new
-rats in your search area in the future.
-
-For developers the new API documentation might come in handy, it can be found at
-[/api/schema/swagger-ui/](https://notfellchen.org/api/schema/swagger-ui/)

README.md

@@ -80,13 +80,17 @@ docker run -p8000:7345 moanos/notfellchen:latest
 ## Geocoding
 
 Geocoding services (search map data by name, address or postcode) are provided via the
-either [Nominatim](https://nominatim.org/) or [photon](https://github.com/komoot/photon) API, powered by [OpenStreetMap](https://openstreetmap.org) data.
+[Nominatim](https://nominatim.org/) API, powered by [OpenStreetMap](https://openstreetmap.org) data. Notfellchen uses
-Notfellchen uses a selfhosted Photon instance to avoid overburdening the publicly hosted instance.
+a selfhosted Nominatim instance to avoid overburdening the publicly hosted instance. Due to ressource constraints
+geocoding is only supported for Germany right now.
+
+ToDos
+* [ ] Implement a report that shows the number of location strings that could not be converted into a location
+* [x] Add a management command to re-query location strings to fill location
 
 ## Maps
 
 The map on the main homepage is powered by [Versatiles](https://versatiles.org), and rendered using [Maplibre](https://maplibre.org/).
-The Versatiles server is self-hosted and does not send data to third parties.
 
 ## Translation
 
@@ -121,20 +125,3 @@ Start beat
 ```zsh
  celery -A notfellchen.celery beat
 ```
-
-# Contributing
-
-This project is currently solely developed by me, moanos. I'd like that to change and will be very happy for contributions
-and shared responsibilities. Some ideas where you can look for contributing first
-
-* CSS structure: It's a hot mess right now, and I'm happy it somehow works. As you might see, there is much room for improvement. Refactoring this and streamlining the look across the app would be amazing.
-* Docker: If you know how to build a docker container that is a) smaller or b) utilizes staged builds this would be amazing. Any improvement welcome
-* Testing: Writing tests is always welcome, and it's likely you discover a few bugs
-
-I'm also very happy for all other contributions. Before you do large refactoring efforts or features, best write a short
-issue for it before you spend a lot of work.
-
-Send PRs either to [codeberg](https://codeberg.org/moanos/notfellchen) (preferred) or [Github](https://github.com/moan0s/notfellchen).
-CI (currently only for dcumentation) runs via [git.hyteck.de](https://git.hyteck.de), you can also ask moanos for an account there.
-
-Also welcome are new issues with suggestions or bugs and additions to the documentation.

notfellchen.cfg

@@ -24,7 +24,4 @@ console-only=true
 app_log_level=INFO
 django_log_level=INFO
 
-[geocoding]
-api_url=https://photon.hyteck.de/api
-api_format=photon
 

pyproject.toml

@@ -39,8 +39,7 @@ dependencies = [
     "django-crispy-forms",
     "crispy-bootstrap4",
     "djangorestframework",
-    "celery[redis]",
+    "celery[redis]"
-    "drf-spectacular[sidecar]"
 ]
 
 dynamic = ["version", "readme"]

src/fellchensammlung/admin.py

@@ -6,11 +6,10 @@ from django.utils.html import format_html
 from django.urls import reverse
 from django.utils.http import urlencode
 
-from .models import User, Language, Text, ReportComment, ReportAdoptionNotice, Log, Timestamp, SearchSubscription, \
+from .models import User, Language, Text, ReportComment, ReportAdoptionNotice, Log, Timestamp
-    SpeciesSpecificURL
 
 from .models import Animal, Species, RescueOrganization, AdoptionNotice, Location, Rule, Image, ModerationAction, \
-    Comment, Report, Announcement, AdoptionNoticeStatus, User, Subscriptions, BaseNotification
+    Comment, Report, Announcement, AdoptionNoticeStatus, User, Subscriptions
 from django.utils.translation import gettext_lazy as _
 
 
@@ -67,7 +66,6 @@ class UserAdmin(admin.ModelAdmin):
 
     export_as_csv.short_description = _("Ausgewählte User exportieren")
 
-
 def _reported_content_link(obj):
     reported_content = obj.reported_content
     return format_html(f'<a href="{reported_content.get_absolute_url}">{reported_content}</a>')
@@ -94,39 +92,22 @@ class ReportAdoptionNoticeAdmin(admin.ModelAdmin):
 
     reported_content_link.short_description = "Reported Content"
 
-class SpeciesSpecificURLInline(admin.StackedInline):
-    model = SpeciesSpecificURL
 
 @admin.register(RescueOrganization)
 class RescueOrganizationAdmin(admin.ModelAdmin):
-    search_fields = ("name","description", "internal_comment", "location_string")
+    search_fields = ("name__icontains",)
     list_display = ("name", "trusted", "allows_using_materials", "website")
     list_filter = ("allows_using_materials", "trusted",)
 
-    inlines = [
-        SpeciesSpecificURLInline,
-    ]
-
 
 @admin.register(Text)
 class TextAdmin(admin.ModelAdmin):
     search_fields = ("title__icontains", "text_code__icontains",)
 
-
 @admin.register(Comment)
 class CommentAdmin(admin.ModelAdmin):
     list_filter = ("user",)
 
-
-@admin.register(BaseNotification)
-class BaseNotificationAdmin(admin.ModelAdmin):
-    list_filter = ("user", "read")
-
-@admin.register(SearchSubscription)
-class SearchSubscriptionAdmin(admin.ModelAdmin):
-    list_filter = ("owner",)
-
-
 admin.site.register(Animal)
 admin.site.register(Species)
 admin.site.register(Location)

src/fellchensammlung/api/serializers.py

@@ -14,11 +14,6 @@ class AnimalCreateSerializer(serializers.ModelSerializer):
         model = Animal
         fields = ["name", "date_of_birth", "description", "species", "sex", "adoption_notice"]
 
-class RescueOrgSerializer(serializers.ModelSerializer):
-    class Meta:
-        model = RescueOrganization
-        fields = ["name", "location_string", "instagram", "facebook", "fediverse_profile", "email", "phone_number",
-                  "website", "description", "external_object_identifier", "external_source_identifier"]
 
 class AnimalGetSerializer(serializers.ModelSerializer):
     class Meta:

src/fellchensammlung/api/views.py

@@ -1,8 +1,12 @@
 from rest_framework.views import APIView
 from rest_framework.response import Response
+from rest_framework import status
+from rest_framework import permissions
+from rest_framework.decorators import api_view, permission_classes
+from rest_framework.permissions import IsAuthenticated
 from django.db import transaction
 from fellchensammlung.models import AdoptionNotice, Animal, Log, TrustLevel
-from fellchensammlung.tasks import post_adoption_notice_save
+from fellchensammlung.tasks import add_adoption_notice_location
 from rest_framework import status
 from rest_framework.permissions import IsAuthenticated
 from .serializers import (
@@ -11,25 +15,14 @@ from .serializers import (
     RescueOrganizationSerializer,
     AdoptionNoticeSerializer,
     ImageCreateSerializer,
-    SpeciesSerializer, RescueOrgSerializer,
+    SpeciesSerializer,
 )
 from fellchensammlung.models import Animal, RescueOrganization, AdoptionNotice, Species, Image
-from drf_spectacular.utils import extend_schema
+
 
 class AdoptionNoticeApiView(APIView):
     permission_classes = [IsAuthenticated]
 
-    @extend_schema(
-        parameters=[
-            {
-                'name': 'id',
-                'required': False,
-                'description': 'ID of the adoption notice to retrieve.',
-                'type': int
-            },
-        ],
-        responses={200: AdoptionNoticeSerializer(many=True)}
-    )
     def get(self, request, *args, **kwargs):
         """
         Retrieve adoption notices with their related animals and images.
@@ -47,13 +40,9 @@ class AdoptionNoticeApiView(APIView):
         return Response(serializer.data, status=status.HTTP_200_OK)
 
     @transaction.atomic
-    @extend_schema(
-        request=AdoptionNoticeSerializer,
-        responses={201: 'Adoption notice created successfully!'}
-    )
     def post(self, request, *args, **kwargs):
         """
-        API view to add an adoption notice.
+        API view to add an adoption notice.b
         """
         serializer = AdoptionNoticeSerializer(data=request.data, context={'request': request})
         if not serializer.is_valid():
@@ -62,7 +51,7 @@ class AdoptionNoticeApiView(APIView):
         adoption_notice = serializer.save(owner=request.user)
 
         # Add the location
-        post_adoption_notice_save.delay_on_commit(adoption_notice.pk)
+        add_adoption_notice_location.delay_on_commit(adoption_notice.pk)
 
         # Only set active when user has trust level moderator or higher
         if request.user.trust_level >= TrustLevel.MODERATOR:
@@ -84,7 +73,6 @@ class AdoptionNoticeApiView(APIView):
         )
 
 
-
 class AnimalApiView(APIView):
     permission_classes = [IsAuthenticated]
 
@@ -118,20 +106,10 @@ class AnimalApiView(APIView):
             )
         return Response(serializer.errors, status=status.HTTP_400_BAD_REQUEST)
 
+
 class RescueOrganizationApiView(APIView):
     permission_classes = [IsAuthenticated]
 
-    @extend_schema(
-        parameters=[
-            {
-                'name': 'id',
-                'required': False,
-                'description': 'ID of the rescue organization to retrieve.',
-                'type': int
-            },
-        ],
-        responses={200: RescueOrganizationSerializer(many=True)}
-    )
     def get(self, request, *args, **kwargs):
         """
         Get list of rescue organizations or a specific organization by ID.
@@ -148,32 +126,11 @@ class RescueOrganizationApiView(APIView):
         serializer = RescueOrganizationSerializer(organizations, many=True, context={"request": request})
         return Response(serializer.data, status=status.HTTP_200_OK)
 
-    @transaction.atomic
-    @extend_schema(
-        request=RescueOrgSerializer,  # Document the request body
-        responses={201: 'Rescue organization created/updated successfully!'}
-    )
-    def post(self, request, *args, **kwargs):
-        """
-        Create or update a rescue organization.
-        """
-        serializer = RescueOrgSerializer(data=request.data, context={"request": request})
-        if serializer.is_valid():
-            rescue_org = serializer.save(owner=request.user)
-            return Response(
-                {"message": "Rescue organization created/updated successfully!", "id": rescue_org.id},
-                status=status.HTTP_201_CREATED,
-            )
-        return Response(serializer.errors, status=status.HTTP_400_BAD_REQUEST)
 
 class AddImageApiView(APIView):
     permission_classes = [IsAuthenticated]
 
     @transaction.atomic
-    @extend_schema(
-        request=ImageCreateSerializer,
-        responses={201: 'Image added successfully!'}
-    )
     def post(self, request, *args, **kwargs):
         """
         Add an image to an animal or adoption notice.
@@ -182,7 +139,7 @@ class AddImageApiView(APIView):
         if serializer.is_valid():
             if serializer.validated_data["attach_to_type"] == "animal":
                 object_to_attach_to = Animal.objects.get(id=serializer.validated_data["attach_to"])
-            elif serializer.validated_data["attach_to_type"] == "adoption_notice":
+            elif serializer.fields["attach_to_type"] == "adoption_notice":
                 object_to_attach_to = AdoptionNotice.objects.get(id=serializer.validated_data["attach_to"])
             else:
                 raise ValueError("Unknown attach_to_type given, should not happen. Check serializer")
@@ -200,9 +157,6 @@ class AddImageApiView(APIView):
 class SpeciesApiView(APIView):
     permission_classes = [IsAuthenticated]
 
-    @extend_schema(
-        responses={200: SpeciesSerializer(many=True)}
-    )
     def get(self, request, *args, **kwargs):
         """
         Retrieve a list of species.

src/fellchensammlung/forms.py

@@ -1,13 +1,12 @@
 from django import forms
 
 from .models import AdoptionNotice, Animal, Image, ReportAdoptionNotice, ReportComment, ModerationAction, User, Species, \
-    Comment, SexChoicesWithAll, DistanceChoices
+    Comment, SexChoicesWithAll
 from django_registration.forms import RegistrationForm
 from crispy_forms.helper import FormHelper
 from crispy_forms.layout import Submit, Layout, Fieldset, HTML, Row, Column, Field, Hidden
 from django.utils.translation import gettext_lazy as _
 from notfellchen.settings import MEDIA_URL
-from crispy_forms.layout import Div
 
 
 def animal_validator(value: str):
@@ -125,21 +124,11 @@ class ImageForm(forms.ModelForm):
         self.helper.form_id = 'form-animal-photo'
         self.helper.form_class = 'card'
         self.helper.form_method = 'post'
-
         if in_flow:
-           submits= Div(Submit('submit', _('Speichern')),
+            self.helper.add_input(Submit('save-and-add-another', _('Speichern und weiteres Foto hinzufügen')))
-                      Submit('save-and-add-another', _('Speichern und weiteres Foto hinzufügen')), css_class="container-edit-buttons")
+            self.helper.add_input(Submit('submit', _('Speichern')))
         else:
-            submits = Fieldset(Submit('submit', _('Speichern')), css_class="container-edit-buttons")
+            self.helper.add_input(Submit('submit', _('Submit')))
-        self.helper.layout = Layout(
-            Div(
-                'image',
-                'alt_text',
-                css_class="spaced",
-            ),
-            submits
-        )
-
 
     class Meta:
         model = Image
@@ -192,8 +181,12 @@ class CustomRegistrationForm(RegistrationForm):
         self.helper.add_input(Submit('submit', _('Registrieren'), css_class="btn"))
 
 
+def _get_distances():
+    return {i: i for i in [20, 50, 100, 200, 500]}
+
+
 class AdoptionNoticeSearchForm(forms.Form):
+    location = forms.CharField(max_length=20, label=_("Stadt"), required=False)
+    max_distance = forms.ChoiceField(choices=_get_distances, label=_("Max. Distanz"))
     sex = forms.ChoiceField(choices=SexChoicesWithAll, label=_("Geschlecht"), required=False,
                             initial=SexChoicesWithAll.ALL)
-    max_distance = forms.ChoiceField(choices=DistanceChoices, initial=DistanceChoices.ONE_HUNDRED, label=_("Suchradius"))
-    location_string = forms.CharField(max_length=100, label=_("Stadt"), required=False)

src/fellchensammlung/migrations/0027_alter_animal_species_andoptionnoticenotification.py

@@ -1,27 +0,0 @@
-# Generated by Django 5.1.1 on 2024-12-14 07:57
-
-import django.db.models.deletion
-from django.db import migrations, models
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ('fellchensammlung', '0026_alter_animal_sex'),
-    ]
-
-    operations = [
-        migrations.AlterField(
-            model_name='animal',
-            name='species',
-            field=models.ForeignKey(on_delete=django.db.models.deletion.PROTECT, to='fellchensammlung.species', verbose_name='Tierart'),
-        ),
-        migrations.CreateModel(
-            name='AndoptionNoticeNotification',
-            fields=[
-                ('basenotification_ptr', models.OneToOneField(auto_created=True, on_delete=django.db.models.deletion.CASCADE, parent_link=True, primary_key=True, serialize=False, to='fellchensammlung.basenotification')),
-                ('adoption_notice', models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, to='fellchensammlung.adoptionnotice', verbose_name='Vermittlung')),
-            ],
-            bases=('fellchensammlung.basenotification',),
-        ),
-    ]

src/fellchensammlung/migrations/0028_searchsubscription.py

@@ -1,25 +0,0 @@
-# Generated by Django 5.1.1 on 2024-12-26 15:56
-
-import django.db.models.deletion
-from django.conf import settings
-from django.db import migrations, models
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ('fellchensammlung', '0027_alter_animal_species_andoptionnoticenotification'),
-    ]
-
-    operations = [
-        migrations.CreateModel(
-            name='SearchSubscription',
-            fields=[
-                ('id', models.BigAutoField(auto_created=True, primary_key=True, serialize=False, verbose_name='ID')),
-                ('sex', models.CharField(choices=[('F', 'Weiblich'), ('M', 'Männlich'), ('M_N', 'Männlich, kastriert'), ('F_N', 'Weiblich, kastriert'), ('I', 'Intergeschlechtlich')], max_length=20)),
-                ('radius', models.IntegerField(choices=[(20, '20 km'), (50, '50 km'), (100, '100 km'), (200, '200 km'), (500, '500 km')])),
-                ('location', models.ForeignKey(on_delete=django.db.models.deletion.PROTECT, to='fellchensammlung.location')),
-                ('owner', models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, to=settings.AUTH_USER_MODEL)),
-            ],
-        ),
-    ]

src/fellchensammlung/migrations/0029_rename_andoptionnoticenotification_adoptionnoticenotification_and_more.py

@@ -1,22 +0,0 @@
-# Generated by Django 5.1.4 on 2024-12-31 10:37
-
-from django.db import migrations, models
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ('fellchensammlung', '0028_searchsubscription'),
-    ]
-
-    operations = [
-        migrations.RenameModel(
-            old_name='AndoptionNoticeNotification',
-            new_name='AdoptionNoticeNotification',
-        ),
-        migrations.AlterField(
-            model_name='searchsubscription',
-            name='sex',
-            field=models.CharField(choices=[('F', 'Weiblich'), ('M', 'Männlich'), ('M_N', 'Männlich, kastriert'), ('F_N', 'Weiblich Kastriert'), ('I', 'Intergeschlechtlich'), ('A', 'Alle')], max_length=20),
-        ),
-    ]

src/fellchensammlung/migrations/0030_rename_radius_searchsubscription_max_distance.py

@@ -1,18 +0,0 @@
-# Generated by Django 5.1.4 on 2024-12-31 12:23
-
-from django.db import migrations
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ('fellchensammlung', '0029_rename_andoptionnoticenotification_adoptionnoticenotification_and_more'),
-    ]
-
-    operations = [
-        migrations.RenameField(
-            model_name='searchsubscription',
-            old_name='radius',
-            new_name='max_distance',
-        ),
-    ]

src/fellchensammlung/migrations/0031_alter_searchsubscription_location_and_more.py

@@ -1,24 +0,0 @@
-# Generated by Django 5.1.4 on 2024-12-31 12:42
-
-import django.db.models.deletion
-from django.db import migrations, models
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ('fellchensammlung', '0030_rename_radius_searchsubscription_max_distance'),
-    ]
-
-    operations = [
-        migrations.AlterField(
-            model_name='searchsubscription',
-            name='location',
-            field=models.ForeignKey(null=True, on_delete=django.db.models.deletion.PROTECT, to='fellchensammlung.location'),
-        ),
-        migrations.AlterField(
-            model_name='searchsubscription',
-            name='max_distance',
-            field=models.IntegerField(choices=[(20, '20 km'), (50, '50 km'), (100, '100 km'), (200, '200 km'), (500, '500 km')], null=True),
-        ),
-    ]

src/fellchensammlung/migrations/0032_searchsubscription_created_at_and_more.py

@@ -1,25 +0,0 @@
-# Generated by Django 5.1.4 on 2025-01-01 22:04
-
-import django.utils.timezone
-from django.db import migrations, models
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ('fellchensammlung', '0031_alter_searchsubscription_location_and_more'),
-    ]
-
-    operations = [
-        migrations.AddField(
-            model_name='searchsubscription',
-            name='created_at',
-            field=models.DateTimeField(auto_now_add=True, default=django.utils.timezone.now),
-            preserve_default=False,
-        ),
-        migrations.AddField(
-            model_name='searchsubscription',
-            name='updated_at',
-            field=models.DateTimeField(auto_now=True),
-        ),
-    ]

src/fellchensammlung/migrations/0033_rescueorganization_external_object_identifier_and_more.py

@@ -1,23 +0,0 @@
-# Generated by Django 5.1.4 on 2025-01-05 18:18
-
-from django.db import migrations, models
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ('fellchensammlung', '0032_searchsubscription_created_at_and_more'),
-    ]
-
-    operations = [
-        migrations.AddField(
-            model_name='rescueorganization',
-            name='external_object_identifier',
-            field=models.CharField(blank=True, max_length=200, null=True, verbose_name='External Object Identifier'),
-        ),
-        migrations.AddField(
-            model_name='rescueorganization',
-            name='external_source_identifier',
-            field=models.CharField(blank=True, choices=[('OSM', 'Open Street Map')], max_length=200, null=True, verbose_name='External Source Identifier'),
-        ),
-    ]

src/fellchensammlung/migrations/0034_speciesspecificurl.py

@@ -1,23 +0,0 @@
-# Generated by Django 5.1.4 on 2025-01-05 19:36
-
-import django.db.models.deletion
-from django.db import migrations, models
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ('fellchensammlung', '0033_rescueorganization_external_object_identifier_and_more'),
-    ]
-
-    operations = [
-        migrations.CreateModel(
-            name='SpeciesSpecificURL',
-            fields=[
-                ('id', models.BigAutoField(auto_created=True, primary_key=True, serialize=False, verbose_name='ID')),
-                ('url', models.URLField(verbose_name='Tierartspezifische URL')),
-                ('rescues_organization', models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, to='fellchensammlung.rescueorganization', verbose_name='Tierschutzorganisation')),
-                ('species', models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, to='fellchensammlung.species', verbose_name='Tierart')),
-            ],
-        ),
-    ]

src/fellchensammlung/migrations/0035_alter_image_alt_text_and_more.py

@@ -1,28 +0,0 @@
-# Generated by Django 5.1.4 on 2025-01-11 12:58
-
-from django.db import migrations, models
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ('fellchensammlung', '0034_speciesspecificurl'),
-    ]
-
-    operations = [
-        migrations.AlterField(
-            model_name='image',
-            name='alt_text',
-            field=models.TextField(max_length=2000, verbose_name='Alternativtext'),
-        ),
-        migrations.AlterField(
-            model_name='report',
-            name='reported_broken_rules',
-            field=models.ManyToManyField(to='fellchensammlung.rule', verbose_name='Regeln gegen die verstoßen wurde'),
-        ),
-        migrations.AlterField(
-            model_name='report',
-            name='user_comment',
-            field=models.TextField(blank=True, verbose_name='Kommentar/Zusätzliche Information'),
-        ),
-    ]

src/fellchensammlung/models.py

@@ -1,6 +1,4 @@
 import uuid
-from random import choices
-from tabnanny import verbose
 
 from django.db import models
 from django.urls import reverse
@@ -13,8 +11,6 @@ from django.contrib.auth.models import AbstractUser
 
 from .tools import misc, geo
 from notfellchen.settings import MEDIA_URL
-from .tools.geo import LocationProxy, Position
-from .tools.misc import age_as_hr_string, time_since_as_hr_string
 
 
 class Language(models.Model):
@@ -39,7 +35,7 @@ class Language(models.Model):
 
 
 class Location(models.Model):
-    place_id = models.IntegerField()  # OSM id
+    place_id = models.IntegerField()
     latitude = models.FloatField()
     longitude = models.FloatField()
     name = models.CharField(max_length=2000)
@@ -49,30 +45,26 @@ class Location(models.Model):
     def __str__(self):
         return f"{self.name} ({self.latitude:.5}, {self.longitude:.5})"
 
-    @property
-    def position(self):
-        return (self.latitude, self.longitude)
-
     @property
     def str_hr(self):
         return f"{self.name.split(',')[0]}"
 
     @staticmethod
     def get_location_from_string(location_string):
-        try:
+        geo_api = geo.GeoAPI()
-            proxy = LocationProxy(location_string)
+        geojson = geo_api.get_geojson_for_query(location_string)
-        except ValueError:
+        if geojson is None:
             return None
-        location = Location.get_location_from_proxy(proxy)
+        result = geojson[0]
-        return location
+        if "name" in result:
-
+            name = result["name"]
-    @staticmethod
+        else:
-    def get_location_from_proxy(proxy):
+            name = result["display_name"]
         location = Location.objects.create(
-            place_id=proxy.place_id,
+            place_id=result["place_id"],
-            latitude=proxy.latitude,
+            latitude=result["lat"],
-            longitude=proxy.longitude,
+            longitude=result["lon"],
-            name=proxy.name,
+            name=name,
         )
         return location
 
@@ -84,10 +76,6 @@ class Location(models.Model):
         instance.save()
 
 
-class ExternalSourceChoices(models.TextChoices):
-    OSM = "OSM", _("Open Street Map")
-
-
 class RescueOrganization(models.Model):
     def __str__(self):
         return f"{self.name}"
@@ -124,11 +112,6 @@ class RescueOrganization(models.Model):
     created_at = models.DateTimeField(auto_now_add=True)
     internal_comment = models.TextField(verbose_name=_("Interner Kommentar"), null=True, blank=True, )
     description = models.TextField(null=True, blank=True, verbose_name=_('Beschreibung'))  # Markdown allowed
-    external_object_identifier = models.CharField(max_length=200, null=True, blank=True,
-                                                  verbose_name=_('External Object Identifier'))
-    external_source_identifier = models.CharField(max_length=200, null=True, blank=True,
-                                                  choices=ExternalSourceChoices.choices,
-                                                  verbose_name=_('External Source Identifier'))
 
     def get_absolute_url(self):
         return reverse("rescue-organization-detail", args=[str(self.pk)])
@@ -137,20 +120,6 @@ class RescueOrganization(models.Model):
     def adoption_notices(self):
         return AdoptionNotice.objects.filter(organization=self)
 
-    @property
-    def position(self):
-        if self.location:
-            return Position(latitude=self.location.latitude, longitude=self.location.longitude)
-        else:
-            return None
-
-    @property
-    def description_short(self):
-        if self.description is None:
-            return ""
-        if len(self.description) > 200:
-            return self.description[:200] + f" ... [weiterlesen]({self.get_absolute_url()})"
-
 
 # Admins can perform all actions and have the highest trust associated with them
 # Moderators can make moderation decisions regarding the deletion of content
@@ -188,21 +157,12 @@ class User(AbstractUser):
         verbose_name = _('Nutzer*in')
         verbose_name_plural = _('Nutzer*innen')
 
-    def get_full_name(self):
-        if self.first_name and self.last_name:
-            return self.first_name + self.last_name
-        else:
-            return self.username
-
     def get_absolute_url(self):
         return reverse("user-detail", args=[str(self.pk)])
 
     def get_notifications_url(self):
         return self.get_absolute_url()
 
-    def get_unread_notifications(self):
-        return BaseNotification.objects.filter(user=self, read=False)
-
     def get_num_unread_notifications(self):
         return BaseNotification.objects.filter(user=self, read=False).count()
 
@@ -217,7 +177,7 @@ class User(AbstractUser):
 
 class Image(models.Model):
     image = models.ImageField(upload_to='images')
-    alt_text = models.TextField(max_length=2000, verbose_name=_('Alternativtext'))
+    alt_text = models.TextField(max_length=2000)
     owner = models.ForeignKey(User, on_delete=models.CASCADE)
     updated_at = models.DateTimeField(auto_now=True)
     created_at = models.DateTimeField(auto_now_add=True)
@@ -283,11 +243,6 @@ class AdoptionNotice(models.Model):
             sexes.add(animal.sex)
         return sexes
 
-    @property
-    def last_checked_hr(self):
-        time_since_last_checked = timezone.now() - self.last_checked
-        return time_since_as_hr_string(time_since_last_checked)
-
     def sex_code(self):
         # Treat Intersex as mixed in order to increase their visibility
         if len(self.sexes) > 1:
@@ -331,11 +286,6 @@ class AdoptionNotice(models.Model):
         # returns all subscriptions to that adoption notice
         return Subscriptions.objects.filter(adoption_notice=self)
 
-    @staticmethod
-    def get_active_ANs():
-        active_ans = [an for an in AdoptionNotice.objects.all() if an.is_active]
-        return active_ans
-
     def get_photos(self):
         """
         First trys to get group photos that are attached to the adoption notice if there is none it trys to fetch
@@ -418,8 +368,8 @@ class AdoptionNotice(models.Model):
         self.adoptionnoticestatus.set_unchecked()
 
         for subscription in self.get_subscriptions():
-            notification_title = _("Vermittlung deaktiviert:") + f" {self.name}"
+            notification_title = _("Vermittlung deaktiviert:") + f" {self}"
-            text = _("Die folgende Vermittlung wurde deaktiviert: ") + f"[{self.name}]({self.get_absolute_url()})"
+            text = _("Die folgende Vermittlung wurde deaktiviert: ") + f"{self.name}, {self.get_absolute_url()}"
             BaseNotification.objects.create(user=subscription.owner, text=text, title=notification_title)
 
 
@@ -539,7 +489,7 @@ class Animal(models.Model):
     date_of_birth = models.DateField(verbose_name=_('Geburtsdatum'))
     name = models.CharField(max_length=200)
     description = models.TextField(null=True, blank=True, verbose_name=_('Beschreibung'))
-    species = models.ForeignKey(Species, on_delete=models.PROTECT, verbose_name=_("Tierart"))
+    species = models.ForeignKey(Species, on_delete=models.PROTECT)
     photos = models.ManyToManyField(Image, blank=True)
     sex = models.CharField(
         max_length=20,
@@ -581,40 +531,6 @@ class Animal(models.Model):
         return reverse('animal-detail', args=[str(self.id)])
 
 
-class DistanceChoices(models.IntegerChoices):
-    TWENTY = 20, '20 km'
-    FIFTY = 50, '50 km'
-    ONE_HUNDRED = 100, '100 km'
-    TWO_HUNDRED = 200, '200 km'
-    FIVE_HUNDRED = 500, '500 km'
-
-
-class SearchSubscription(models.Model):
-    """
-    SearchSubscriptions allow a user to get a notification when a new AdoptionNotice is added that matches their Search
-    criteria. Search criteria are location, SexChoicesWithAll and distance
-
-    Process:
-    - User performs a normal search
-    - User clicks Button "Subscribe to this Search"
-    - SearchSubscription is added to database
-    - On new AdoptionNotice: Check all existing SearchSubscriptions for matches
-    - For matches: Send notification to user of the SearchSubscription
-    """
-    owner = models.ForeignKey(User, on_delete=models.CASCADE)
-    location = models.ForeignKey(Location, on_delete=models.PROTECT, null=True)
-    sex = models.CharField(max_length=20, choices=SexChoicesWithAll.choices)
-    max_distance = models.IntegerField(choices=DistanceChoices.choices, null=True)
-    updated_at = models.DateTimeField(auto_now=True)
-    created_at = models.DateTimeField(auto_now_add=True)
-
-    def __str__(self):
-        if self.location and self.max_distance:
-            return f"{self.owner}: [{SexChoicesWithAll(self.sex).label}] {self.max_distance}km - {self.location}"
-        else:
-            return f"{self.owner}: [{SexChoicesWithAll(self.sex).label}]"
-
-
 class Rule(models.Model):
     """
     Class to store rules
@@ -648,8 +564,8 @@ class Report(models.Model):
     id = models.UUIDField(primary_key=True, default=uuid.uuid4, help_text=_('ID dieses reports'),
                           verbose_name=_('ID'))
     status = models.CharField(max_length=30, choices=STATES)
-    reported_broken_rules = models.ManyToManyField(Rule, verbose_name=_("Regeln gegen die verstoßen wurde"))
+    reported_broken_rules = models.ManyToManyField(Rule)
-    user_comment = models.TextField(blank=True, verbose_name=_("Kommentar/Zusätzliche Information"))
+    user_comment = models.TextField(blank=True)
     created_at = models.DateTimeField(auto_now_add=True)
     updated_at = models.DateTimeField(auto_now=True)
 
@@ -708,12 +624,9 @@ class ModerationAction(models.Model):
         return f"[{self.action}]: {self.public_comment}"
 
 
-class TextTypeChoices(models.TextChoices):
+"""
-    DEDICATED = "dedicated", _("Fest zugeordnet")
+Membership
-    MALE = "M", _("Männlich")
+"""
-    MALE_NEUTERED = "M_N", _("Männlich, kastriert")
-    FEMALE_NEUTERED = "F_N", _("Weiblich, kastriert")
-    INTER = "I", _("Intergeschlechtlich")
 
 
 class Text(models.Model):
@@ -838,14 +751,6 @@ class CommentNotification(BaseNotification):
         return self.comment.get_absolute_url
 
 
-class AdoptionNoticeNotification(BaseNotification):
-    adoption_notice = models.ForeignKey(AdoptionNotice, on_delete=models.CASCADE, verbose_name=_('Vermittlung'))
-
-    @property
-    def url(self):
-        return self.adoption_notice.get_absolute_url
-
-
 class Subscriptions(models.Model):
     owner = models.ForeignKey(User, on_delete=models.CASCADE, verbose_name=_('Nutzer*in'))
     adoption_notice = models.ForeignKey(AdoptionNotice, on_delete=models.CASCADE, verbose_name=_('AdoptionNotice'))
@@ -880,13 +785,3 @@ class Timestamp(models.Model):
 
     def ___str__(self):
         return f"[{self.key}] - {self.timestamp.strftime('%H:%M:%S %d-%m-%Y ')} - {self.data}"
-
-
-class SpeciesSpecificURL(models.Model):
-    """
-    Model that allows to specify a URL for a rescue organization where a certain species can be found
-    """
-    species = models.ForeignKey(Species, on_delete=models.CASCADE, verbose_name=_("Tierart"))
-    rescues_organization = models.ForeignKey(RescueOrganization, on_delete=models.CASCADE,
-                                             verbose_name=_("Tierschutzorganisation"))
-    url = models.URLField(verbose_name=_("Tierartspezifische URL"))

src/fellchensammlung/static/fellchensammlung/css/styles.css

@@ -37,49 +37,13 @@ body {
 }
 
 
-a {
-    color: inherit;
-}
-
-h1, h2, h3 {
-    margin-bottom: 5px;
-    margin-top: 5px;
-}
-
 table {
     width: 100%;
 }
 
-@media screen and (max-width: 600px) {
+a {
-    .responsive thead {
+    text-decoration: none;
-        visibility: hidden;
+    color: inherit;
-        height: 0;
-        position: absolute;
-    }
-
-    .responsive tr {
-        display: block;
-    }
-
-    .responsive td {
-        border: 1px solid;
-        border-bottom: none;
-        display: block;
-        font-size: .8em;
-        text-align: right;
-        width: 100%;
-    }
-
-    .responsive td::before {
-        content: attr(data-label);
-        float: left;
-        font-weight: bold;
-        text-transform: uppercase;
-    }
-
-    .responsive td:last-child {
-        border-bottom: 1px solid;
-    }
 }
 
 table {
@@ -101,7 +65,7 @@ td {
     padding: 5px;
 }
 
-thead td {
+th {
     border: 3px solid black;
     border-collapse: collapse;
     padding: 8px;
@@ -135,16 +99,11 @@ textarea {
     width: 100%;
 }
 
-.container-cards h1,
-.container-cards h2 {
-    width: 100%; /* Make sure heading fills complete line */
-}
-
 .card {
     flex: 1 25%;
     margin: 10px;
     border-radius: 8px;
-    padding: 8px;
+    padding: 5px;
     background: var(--background-three);
     color: var(--text-two);
 }
@@ -158,8 +117,8 @@ textarea {
     }
 }
 
-.spaced > * {
+.spaced {
-    margin: 10px;
+    margin-bottom: 30px;
 }
 
 /*******************************/
@@ -242,11 +201,7 @@ select, .button {
     display: block;
 }
 
-a.btn, a.btn2, a.nav-link {
+.btn2 {
-    text-decoration: none;
-}
-
-.btn2, .btn3 {
     background-color: var(--secondary-light-one);
     color: var(--primary-dark-one);
     padding: 8px;
@@ -255,158 +210,6 @@ a.btn, a.btn2, a.nav-link {
     margin: 5px;
 }
 
-.btn3 {
-    border: 1px solid black;
-}
-
-.checkmark {
-    display: inline-block;
-    position: relative;
-    left: 0.2rem;
-    bottom: 0.075rem;
-    background-color: var(--primary-light-one);
-    color: var(--secondary-light-one);
-    border-radius: 0.5rem;
-    width: 1.5rem;
-    height: 1.5rem;
-    text-align: center;
-}
-
-.switch {
-    cursor: pointer;
-    display: inline-block;
-}
-
-.toggle-switch {
-    display: inline-block;
-    background: #ccc;
-    border-radius: 16px;
-    width: 58px;
-    height: 32px;
-    position: relative;
-    vertical-align: middle;
-    transition: background 0.25s;
-}
-
-.toggle-switch:before, .toggle-switch:after {
-    content: "";
-}
-
-.toggle-switch:before {
-    display: block;
-    background: linear-gradient(to bottom, #fff 0%, #eee 100%);
-    border-radius: 50%;
-    box-shadow: 0 0 0 1px rgba(0, 0, 0, 0.25);
-    width: 24px;
-    height: 24px;
-    position: absolute;
-    top: 4px;
-    left: 4px;
-    transition: left 0.25s;
-}
-
-.toggle:hover .toggle-switch:before {
-    background: linear-gradient(to bottom, #fff 0%, #fff 100%);
-    box-shadow: 0 0 0 1px rgba(0, 0, 0, 0.5);
-}
-
-.checked + .toggle-switch {
-    background: #56c080;
-}
-
-.checked + .toggle-switch:before {
-    left: 30px;
-}
-
-.toggle-checkbox {
-    position: absolute;
-    visibility: hidden;
-}
-
-.slider-label {
-    margin-left: 5px;
-    position: relative;
-    top: 2px;
-}
-
-/* Refactor tooltip based on https://luigicavalieri.com/blog/css-tooltip-appearing-from-any-direction/ to allow different directions */
-.tooltip {
-    display: inline-flex;
-    justify-content: center;
-    position: relative;
-}
-
-.tooltip:hover .tooltiptext {
-    display: flex;
-    opacity: 1;
-    visibility: visible;
-}
-
-.tooltip .tooltiptext {
-    border-radius: 4px;
-    bottom: calc(100% + 0.6em + 2px);
-    box-shadow: 0px 2px 4px #07172258;
-    background-color: var(--primary-dark-one);
-    color: var(--secondary-light-one);
-    font-size: 0.68rem;
-    justify-content: center;
-    line-height: 1.35em;
-    padding: 0.5em 0.7em;
-    position: absolute;
-    text-align: center;
-    width: 7rem;
-    z-index: 1;
-    display: flex;
-    opacity: 0;
-    transition: all 0.3s ease-in;
-    visibility: hidden;
-}
-
-.tooltip .tooltiptext::before {
-    border-width: 0.6em 0.8em 0;
-    border-color: transparent;
-    border-top-color: var(--primary-dark-one);
-    content: "";
-    display: block;
-    border-style: solid;
-    position: absolute;
-    top: 100%;
-}
-
-/* Makes the tooltip fly from above */
-.tooltip.top .tooltiptext {
-    margin-bottom: 8px;
-}
-
-.tooltip.top:hover .tooltiptext {
-    margin-bottom: 0;
-}
-
-/* Make adjustments for bottom */
-.tooltip.bottom .tooltiptext {
-    top: calc(100% + 0.6em + 2px);
-    margin-top: 8px;
-}
-
-.tooltip.bottom:hover .tooltiptext {
-    margin-top: 0;
-}
-
-.tooltip.bottom .tooltiptext::before {
-    transform: rotate(180deg);
-    /* 100% of the height of .tooltip */
-    bottom: 100%;
-}
-
-.tooltip:not(.top) .tooltiptext {
-    bottom: auto;
-
-}
-
-.tooltip:not(.top) .tooltiptext::before {
-    top: auto;
-}
-
 
 /*********************/
 /* UNIQUE COMPONENTS */
@@ -740,22 +543,12 @@ a.btn, a.btn2, a.nav-link {
 
 .header-card-adoption-notice {
     display: flex;
-    justify-content: space-between;
+    justify-content: space-evenly;
     align-items: center;
     flex-wrap: wrap;
 
 }
 
-.search-subscription-header {
-    display: flex;
-    justify-content: space-evenly;
-    flex-wrap: wrap;
-
-    h3 {
-        width: 80%;
-    }
-}
-
 .table-adoption-notice-info {
     margin-top: 10px;
 }
@@ -800,30 +593,15 @@ a.btn, a.btn2, a.nav-link {
 .adoption-card-report-link, .notification-card-mark-read {
     margin-left: auto;
     font-size: 2rem;
+    padding: 10px;
 }
 
-.adoption-card-report-link {
-    margin-right: 12px;
-}
-
-
 .notification-card-mark-read {
     display: inline;
 }
 
-.inline-container {
+.heading-card-adoption-notice {
-    display: inline-block;
-}
-
-.inline-container > * {
-    vertical-align: middle;
-}
-
-h2.heading-card-adoption-notice {
-    font-size: 2rem;
-    line-height: 2rem;
     word-wrap: anywhere;
-    width: 80%;
 }
 
 .tags {
@@ -838,15 +616,17 @@ h2.heading-card-adoption-notice {
 }
 
 .detail-adoption-notice-header h1 {
+    width: 50%;
     display: inline-block;
 }
 
 .detail-adoption-notice-header a {
+    display: inline-block;
     float: right;
 }
 
 @media (max-width: 920px) {
-    .detail-adoption-notice-header .inline-container {
+    .detail-adoption-notice-header h1 {
         width: 100%;
     }
 
@@ -864,7 +644,7 @@ h2.heading-card-adoption-notice {
     padding: 5px;
 }
 
-.comment, .notification, .search-subscription {
+.comment, .notification {
     flex: 1 100%;
     margin: 10px;
     border-radius: 8px;
@@ -880,16 +660,7 @@ h2.heading-card-adoption-notice {
     }
 }
 
-.announcement-header {
+.announcement {
-    font-size: 1.2rem;
-    margin: 0px;
-    padding: 0px;
-    color: var(--text-two);
-    text-shadow: none;
-    font-weight: bold;
-}
-
-div.announcement {
     flex: 1 100%;
     margin: 10px;
     border-radius: 8px;
@@ -897,6 +668,13 @@ div.announcement {
     background: var(--background-three);
     color: var(--text-two);
 
+    h1 {
+        font-size: 1.2rem;
+        margin: 0px;
+        padding: 0px;
+        color: var(--text-two);
+        text-shadow: none;
+    }
 }
 
 
@@ -910,43 +688,6 @@ div.announcement {
 
 }
 
-.half {
-    width: 49%;
-}
-
-
-#results {
-    margin-top: 10px;
-    list-style-type: none;
-    padding: 0;
-}
-
-.result-item {
-    padding: 8px;
-    margin: 4px 0;
-    background-color: #ddd1a5;
-    cursor: pointer;
-    border-radius: 8px;
-}
-
-.result-item:hover {
-    background-color: #ede1b5;
-}
-
-.label {
-    border-radius: 8px;
-    padding: 4px;
-    color: #fff;
-}
-
-.active-adoption {
-    background-color: #4a9455;
-}
-
-.inactive-adoption {
-    background-color: #000;
-}
-
 /************************/
 /* GENERAL HIGHLIGHTING */
 /************************/
@@ -963,14 +704,6 @@ div.announcement {
     border: rgba(17, 58, 224, 0.51) 4px solid;
 }
 
-.error {
-    color: #370707;
-    font-weight: bold;
-}
-
-.error::before {
-    content: "⚠️";
-}
 
 /*******/
 /* MAP */
@@ -984,11 +717,6 @@ div.announcement {
     cursor: pointer;
 }
 
-.animal-shelter-marker {
-    background-image: url('../img/animal_shelter.png');
-!important;
-}
-
 .maplibregl-popup {
     max-width: 600px !important;
 }

src/fellchensammlung/static/fellchensammlung/js/custom.js

@@ -1,21 +0,0 @@
-function ifdef(variable, prefix = "", suffix = "") {
-    if (variable !== undefined) {
-        return prefix + variable + suffix;
-    } else {
-        return "";
-    }
-}
-
-function geojson_to_summary(location) {
-    if (ifdef(location.properties.name) !== "") {
-        return location.properties.name + ifdef(location.properties.city, " (", ")");
-    } else {
-        return ifdef(location.properties.street, "", ifdef(location.properties.housenumber, " ",""))  + ifdef(location.properties.city, ", ", "") + ifdef(location.properties.countrycode, ", ", "")
-    }
-
-}
-
-function geojson_to_searchable_string(location) {
-    return ifdef(location.properties.name, "", ", ") + ifdef(location.properties.street, "", ifdef(location.properties.housenumber, " ",", ")) + ifdef(location.properties.city, "", ", ") + ifdef(location.properties.country, "", "")
-}
-

src/fellchensammlung/tasks.py

@@ -1,5 +1,3 @@
-import logging
-
 from celery.app import shared_task
 from django.utils import timezone
 from notfellchen.celery import app as celery_app
@@ -7,8 +5,6 @@ from .mail import send_notification_email
 from .tools.admin import clean_locations, deactivate_unchecked_adoption_notices, deactivate_404_adoption_notices
 from .tools.misc import healthcheck_ok
 from .models import Location, AdoptionNotice, Timestamp
-from .tools.notifications import notify_moderators_of_AN_to_be_checked
-from .tools.search import notify_search_subscribers
 
 
 def set_timestamp(key: str):
@@ -38,15 +34,12 @@ def task_deactivate_unchecked():
     set_timestamp("task_deactivate_404_adoption_notices")
 
 
-@celery_app.task(name="commit.post_an_save")
+@celery_app.task(name="commit.add_location")
-def post_adoption_notice_save(pk):
+def add_adoption_notice_location(pk):
     instance = AdoptionNotice.objects.get(pk=pk)
     Location.add_location_to_object(instance)
     set_timestamp("add_adoption_notice_location")
-    logging.info(f"Location was added to Adoption notice {pk}")
 
-    notify_search_subscribers(instance, only_if_active=True)
-    notify_moderators_of_AN_to_be_checked(instance)
 
 @celery_app.task(name="tools.healthcheck")
 def task_healthcheck():

src/fellchensammlung/templates/fellchensammlung/about.html

@@ -6,50 +6,25 @@
 
 {% block content %}
     {% if about_us %}
-        <div class="card">
+        <h1>{{ about_us.title }}</h1>
-            <h1>{{ about_us.title }}</h1>
+        {{ about_us.content | render_markdown }}
-            <p>
-                {{ about_us.content | render_markdown }}
-            </p>
-        </div>
     {% endif %}
 
-    <h2>{% translate "Regeln" %}</h2>
+    <h1>{% translate "Regeln" %}</h1>
     {% include "fellchensammlung/lists/list-rules.html" %}
 
-    {% if faq %}
-        <div class="card">
-            <h2>{{ faq.title }}</h2>
-            <p>
-                {{ faq.content | render_markdown }}
-            </p>
-        </div>
-    {% endif %}
-
     {% if privacy_statement %}
-        <div class="card">
+        <h1>{{ privacy_statement.title }}</h1>
-            <h2>{{ privacy_statement.title }}</h2>
+        {{ privacy_statement.content | render_markdown }}
-            <p>
-                {{ privacy_statement.content | render_markdown }}
-            </p>
-        </div>
     {% endif %}
 
     {% if terms_of_service %}
-        <div class="card">
+        <h1>{{ terms_of_service.title }}</h1>
-            <h2>{{ terms_of_service.title }}</h2>
+        {{ terms_of_service.content | render_markdown }}
-            <p>
-                {{ terms_of_service.content | render_markdown }}
-            </p>
-        </div>
     {% endif %}
 
     {% if imprint %}
-        <div class="card">
+        <h1>{{ imprint.title }}</h1>
-            <h2>{{ imprint.title }}</h2>
+        {{ imprint.content | render_markdown }}
-            <p>
-                {{ imprint.content | render_markdown }}
-            </p>
-        </div>
     {% endif %}
 {% endblock %}

src/fellchensammlung/templates/fellchensammlung/animal-shelters.html

@@ -1,13 +0,0 @@
-{% extends "fellchensammlung/base_generic.html" %}
-{% load i18n %}
-
-{% block title %}<title>{% translate "Tierschutzorganisationen" %}</title>{% endblock %}
-
-{% block content %}
-    <div class="container-cards">
-        <div class="card">
-            {% include "fellchensammlung/partials/partial-map.html" %}
-        </div>
-    </div>
-    {% include "fellchensammlung/lists/list-animal-shelters.html" %}
-{% endblock %}

src/fellchensammlung/templates/fellchensammlung/base_generic.html

@@ -14,8 +14,6 @@
     <link href="{% static 'fontawesomefree/css/brands.css' %}" rel="stylesheet" type="text/css">
     <link href="{% static 'fontawesomefree/css/solid.css' %}" rel="stylesheet" type="text/css">
 
-    <script src="{% static 'fellchensammlung/js/custom.js' %}"></script>
-
     <link rel="apple-touch-icon" sizes="180x180" href="{% static 'fellchensammlung/favicon/apple-touch-icon.png' %}">
     <link rel="icon" type="image/png" sizes="32x32" href="{% static 'fellchensammlung/favicon/favicon-32x32.png' %}">
     <link rel="icon" type="image/png" sizes="16x16" href="{% static 'fellchensammlung/favicon/favicon-16x16.png' %}">

src/fellchensammlung/templates/fellchensammlung/details/detail-rescue-organization.html

@@ -5,57 +5,17 @@
 {% block title %}<title>{{ org.name }}</title>{% endblock %}
 
 {% block content %}
-    <div class="container-cards">
+    <div class="card">
-        <div class="card half">
+        <h1>{{ org.name }}</h1>
-            <h1>{{ org.name }}</h1>
 
-            <b><i class="fa-solid fa-location-dot"></i></b>
+        <b><i class="fa-solid fa-location-dot"></i></b>
-            {% if org.location %}
+        {% if org.location %}
-                {{ org.location.str_hr }}
+            {{ org.location.str_hr }}
-            {% else %}
+        {% else %}
-                {{ org.location_string }}
+            {{ org.location_string }}
-            {% endif %}
+        {% endif %}
-            <p>{{ org.description | render_markdown }}</p>
+        <p>{{ org.description | render_markdown }}</p>
-            <table class="responsive">
-                <thead>
-                <tr>
-                    {% if org.website %}
-                        <td>{% translate "Website" %}</td>
-                    {% endif %}
-                    {% if org.phone_number %}
-                        <td>{% translate "Telefonnummer" %}</td>
-                    {% endif %}
-                    {% if org.email %}
-                        <td>{% translate "E-Mail" %}</td>
-                    {% endif %}
-                </tr>
-                </thead>
-                <tr>
-                    {% if org.website %}
-                        <td data-label="{% trans 'Website' %} ">
-                            {{ org.website }}
-                        </td>
-                    {% endif %}
-                    {% if org.phone_number %}
-                        <td data-label="{% trans 'Telefonnummer' %}">
-                            {{ org.phone_number }}
-                        </td>
-                    {% endif %}
-
-                    {% if org.email %}
-                        <td data-label="{% trans 'E-Mail' %}">
-                            {{ org.email }}
-                        </td>
-                    {% endif %}
-                </tr>
-            </table>
-        </div>
-        <div class="card half">
-            {% include "fellchensammlung/partials/partial-map.html" %}
-        </div>
     </div>
-
-
     <h2>{% translate 'Vermittlungen der Organisation' %}</h2>
     <div class="container-cards">
         {% if org.adoption_notices %}

src/fellchensammlung/templates/fellchensammlung/details/detail-user.html

@@ -1,83 +1,50 @@
 {% extends "fellchensammlung/base_generic.html" %}
 {% load i18n %}
 
-{% block title %}<title>{{ user.get_full_name }}</title>{% endblock %}
-
-
 {% block content %}
     <h1>{{ user.get_full_name }}</h1>
-    <div class="spaced">
 
-        <div class="container-cards">
+    <p><strong>{% translate "Username" %}:</strong> {{ user.username }}</p>
-            <h2>{% trans 'Daten' %}</h2>
+    <p><strong>{% translate "E-Mail" %}:</strong> {{ user.email }}</p>
-            <div class="card">
+
-                <p><strong>{% translate "Username" %}:</strong> {{ user.username }}</p>
+    {% if user.preferred_language %}
-                <p><strong>{% translate "E-Mail" %}:</strong> {{ user.email }}</p>
+        <p><strong>{% translate "Sprache" %}:</strong> {{ user.preferred_language }}</p>
+    {% else %}
+        <p>{% translate "Keine bevorzugte Sprache gesetzt." %}</p>
+    {% endif %}
+
+    <div class="container-cards">
+        {% if user.id is request.user.id %}
+                <div class="card">
+            {% if token %}
+                <form action="" method="POST">
+                    {% csrf_token %}
+                    <p class="text-muted"><strong>{% translate "API token:" %}</strong> {{ token }}</p>
+                    <input class="btn" type="submit" name="delete_token"
+                           value={% translate "Delete API token" %}>
+                </form>
+            {% else %}
+                <p>{% translate "Kein API-Token vorhanden." %}</p>
+                <form action="" method="POST">
+                    {% csrf_token %}
+                    <input class="btn" type="submit" name="create_token"
+                           value={% translate "Create API token" %}>
+                </form>
+            {% endif %}
             </div>
-        </div>
+            </div><p>
-
-
-        <div class="container-cards">
-            <h2>{% trans 'Profil verwalten' %}</h2>
             <div class="container-comment-form">
+                <h2>{% trans 'Profil verwalten' %}</h2>
                 <p>
                     <a class="btn2" href="{% url 'password_change' %}">{% translate "Change password" %}</a>
                     <a class="btn2" href="{% url 'user-me-export' %}">{% translate "Daten exportieren" %}</a>
                 </p>
             </div>
-        </div>
+            </p>
-
-        {% if user.id is request.user.id %}
-            <div class="detail-animal-header"><h2>{% trans 'Einstellungen' %}</h2></div>
-            <div class="container-cards">
-                <form class="card" action="" method="POST">
-                    {% csrf_token %}
-                    {% if user.email_notifications %}
-                        <label class="toggle">
-                            <input type="submit" class="toggle-checkbox checked" name="toggle_email_notifications">
-                            <div class="toggle-switch round "></div>
-                            <span class="slider-label">
-                        {% translate 'E-Mail Benachrichtigungen' %}
-                        </span>
-                        </label>
-                    {% else %}
-                        <label class="toggle">
-                            <input type="submit" class="toggle-checkbox" name="toggle_email_notifications">
-                            <div class="toggle-switch round"></div>
-                            <span class="slider-label">
-                        {% translate 'E-Mail Benachrichtigungen' %}
-                        </span>
-                        </label>
-                    {% endif %}
-                </form>
-                <div class="card">
-                    {% if token %}
-                        <form action="" method="POST">
-                            {% csrf_token %}
-                            <p class="text-muted"><strong>{% translate "API token:" %}</strong> {{ token }}</p>
-                            <input class="btn" type="submit" name="delete_token"
-                                   value={% translate "Delete API token" %}>
-                        </form>
-                    {% else %}
-                        <p>{% translate "Kein API-Token vorhanden." %}</p>
-                        <form action="" method="POST">
-                            {% csrf_token %}
-                            <input class="btn" type="submit" name="create_token"
-                                   value={% translate "Create API token" %}>
-                        </form>
-                    {% endif %}
-                </div>
-            </div>
-
             <h2>{% translate 'Benachrichtigungen' %}</h2>
             {% include "fellchensammlung/lists/list-notifications.html" %}
-
-            <h2>{% translate 'Abonnierte Suchen' %}</h2>
-            {% include "fellchensammlung/lists/list-search-subscriptions.html" %}
-
             <h2>{% translate 'Meine Vermittlungen' %}</h2>
             {% include "fellchensammlung/lists/list-adoption-notices.html" %}
 
         {% endif %}
-    </div>
 {% endblock %}

src/fellchensammlung/templates/fellchensammlung/details/detail_adoption_notice.html

@@ -6,39 +6,23 @@
 
 {% block content %}
     <div class="detail-adoption-notice-header">
-        <div class="inline-container">
+        <h1 class="detail-adoption-notice-header">{{ adoption_notice.name }}
-            <h1>{{ adoption_notice.name }}</h1>
             {% if not is_subscribed %}
-                <div class="tooltip bottom">
+                <form class="notification-card-mark-read" method="post">
-                    <form class="notification-card-mark-read" method="post">
+                    {% csrf_token %}
-                        {% csrf_token %}
+                    <input type="hidden" name="action" value="subscribe">
-                        <input type="hidden" name="action" value="subscribe">
+                    <input type="hidden" name="adoption_notice_id" value="{{ adoption_notice.pk }}">
-                        <input type="hidden" name="adoption_notice_id" value="{{ adoption_notice.pk }}">
+                    <button class="btn2" type="submit" id="submit"><i class="fa-solid fa-bell"></i></button>
-                        <button class="btn2" type="submit" id="submit"><i class="fa-solid fa-bell"></i></button>
+                </form>
-                    </form>
-                    <span class="tooltiptext">
-                        {% translate 'Abonniere diese Vermittlung um bei Kommentaren oder Statusänderungen benachrichtigt zu werden' %}
-                    </span>
-                </div>
             {% else %}
-                <div class="tooltip bottom">
+                <form class="notification-card-mark-read" method="post">
-                    <form class="notification-card-mark-read" method="post">
+                    {% csrf_token %}
-                        {% csrf_token %}
+                    <input type="hidden" name="action" value="unsubscribe">
-                        <input type="hidden" name="action" value="unsubscribe">
+                    <input type="hidden" name="adoption_notice_id" value="{{ adoption_notice.pk }}">
-                        <input type="hidden" name="adoption_notice_id" value="{{ adoption_notice.pk }}">
+                    <button class="btn2" type="submit" id="submit"><i class="fa-solid fa-bell-slash"></i></button>
-                        <button class="btn2" type="submit" id="submit"><i class="fa-solid fa-bell-slash"></i></button>
+                </form>
-                    </form>
-                    <span class="tooltiptext">
-                        {% translate 'Deabonnieren. Du bekommst keine Benachrichtigungen zu dieser Vermittlung mehr' %}
-                    </span>
-                </div>
             {% endif %}
-            {% if adoption_notice.is_active %}
+        </h1>
-                <span id="submit" class="label active-adoption" style=>{% trans 'Aktive Vermittlung' %}</span>
-            {% else %}
-                <span id="submit" class="label inactive-adoption" style=>{% trans 'Vermittlung inaktiv' %}</span>
-            {% endif %}
-        </div>
         {% if has_edit_permission %}
             <a class="btn2"
                href="{% url 'adoption-notice-add-photo' adoption_notice_id=adoption_notice.pk %}">{% translate 'Foto hinzufügen' %}</a>
@@ -47,20 +31,18 @@
         {% endif %}
     </div>
     <div class="table-adoption-notice-info">
-        <table class="responsive">
+        <table>
-            <thead>
             <tr>
-                <td>{% translate "Ort" %}</td>
+                <th>{% translate "Ort" %}</th>
                 {% if adoption_notice.organization %}
-                    <td>{% translate "Organisation" %}</td>
+                    <th>{% translate "Organisation" %}</th>
                 {% endif %}
-                <td>{% translate "Suchen seit" %}</td>
+                <th>{% translate "Suchen seit" %}</th>
-                <td>{% translate "Zuletzt aktualisiert" %}</td>
+                <th>{% translate "Zuletzt aktualisiert" %}</th>
-                <td>{% translate "Weitere Informationen" %}</td>
+                <th>{% translate "Weitere Informationen" %}</th>
             </tr>
-            </thead>
             <tr>
-                <td data-label="{% trans 'Ort' %} ">
+                <td>
                     {% if adoption_notice.location %}
                         {{ adoption_notice.location }}
                     {% else %}
@@ -68,29 +50,13 @@
                     {% endif %}
                 </td>
                 {% if adoption_notice.organization %}
-                    <td data-label="{% trans 'Organisation' %}">
+                    <td><a href="{{adoption_notice.organization.get_absolute_url }}">{{ adoption_notice.organization }}</a></td>
-                        <div>
-                            <a href="{{ adoption_notice.organization.get_absolute_url }}">{{ adoption_notice.organization }}</a>
-                            {% if adoption_notice.organization.trusted %}
-                                <div class="tooltip top">
-                                    <div class="checkmark"><i class="fa-solid fa-check"></i></div>
-                                    <span class="tooltiptext">
-                                    {% translate 'Diese Organisation kennt sich mit Ratten aus und achtet auf gute Abgabebedingungen' %}
-                                    </span>
-                                </div>
-
-                            {% endif %}
-                        </div>
-                    </td>
-
                 {% endif %}
 
-                <td data-label="{% trans 'Suchen seit' %}">{{ adoption_notice.searching_since }}</td>
+                <td>{{ adoption_notice.searching_since }}</td>
-                <td data-label="{% trans 'Zuletzt aktualisiert' %}">
+                <td>{{ adoption_notice.last_checked | date:'d. F Y' }}</td>
-                    {{ adoption_notice.last_checked_hr }}
+                {% if adoption_notice.further_information %}
-                </td>
+                    <td>
-                <td data-label="{% trans 'Weitere Informationen' %}">
-                    {% if adoption_notice.further_information %}
                         <form method="get" action="{% url 'external-site' %}">
                             <input type="hidden" name="url" value="{{ adoption_notice.further_information }}">
                             <button class="btn" type="submit" id="submit">
@@ -98,10 +64,10 @@
                                     class="fa-solid fa-arrow-up-right-from-square"></i>
                             </button>
                         </form>
-                    {% else %}
+                    </td>
-                        -
+                {% else %}
-                    {% endif %}
+                    <td>-</td>
-                </td>
+                {% endif %}
             </tr>
         </table>
     </div>

src/fellchensammlung/templates/fellchensammlung/forms/form-image.html

@@ -9,7 +9,7 @@
             Lade hier ein Foto hoch - wähle den Titel wie du willst und mach bitte eine Bildbeschreibung,
             damit die Fotos auch für blinde und sehbehinderte Personen zugänglich sind.
         {% endblocktranslate %}
-        <p><a class="btn2" href="https://www.dbsv.org/bildbeschreibung-4-regeln.html">{% translate 'Anleitung für Bildbeschreibungen' %}</a></p>
+        <p><a class="btn" href="https://www.dbsv.org/bildbeschreibung-4-regeln.html">{% translate 'Anleitung für Bildbeschreibungen' %}</a></p>
     </p>
     <div class="container-form">
         {% crispy form %}

src/fellchensammlung/templates/fellchensammlung/forms/form-report.html

@@ -7,6 +7,6 @@
     <form method = "post" enctype="multipart/form-data">
         {% csrf_token %}
         {{ form.as_p }}
-        <button class="btn2" type="submit">{% translate "Melden" %}</button>
+        <button class="button-report" type="submit">{% translate "Melden" %}</button>
     </form>
 {% endblock %}

src/fellchensammlung/templates/fellchensammlung/header.html

@@ -37,8 +37,8 @@
         <nav id="main-menu">
             <ul class="menu">
                 <li>
-                    <a class="nav-link " href="{% url "search" %}">
+                    <a class="nav-link " href="{% url "search" %}"><i
-                        <i class="fas fa-search"></i> {% translate 'Suchen' %}
+                            class="fas fa-search"></i> {% translate 'Suchen' %}
                     </a>
                 </li>
                 <li><a class="nav-link " href="{% url "add-adoption" %}"><i

src/fellchensammlung/templates/fellchensammlung/instance-health-check.html

@@ -106,15 +106,7 @@
             {% csrf_token %}
             <input type="hidden" name="action" value="deactivate_unchecked_adoption_notices">
             <button class="btn" type="submit" id="submit">
-                <i class="fa-solid fa-broom"></i> {% translate "Deaktiviere ungeprüfte Vermittlungen" %}
+                <i class="fa-solid fa-broom"></i> {% translate "Deaktivire ungeprüfte Vermittlungen" %}
-            </button>
-        </form>
-
-        <form class="notification-card-mark-read" method="post">
-            {% csrf_token %}
-            <input type="hidden" name="action" value="deactivate_404">
-            <button class="btn" type="submit" id="submit">
-                <i class="fa-solid fa-broom"></i> {% translate "Deaktiviere 404 Vermittlungen" %}
             </button>
         </form>
     </div>

src/fellchensammlung/templates/fellchensammlung/lists/list-animal-shelters.html

@@ -1,10 +0,0 @@
-{% load i18n %}
-<div class="container-cards spaced">
-    {% if rescue_organizations %}
-        {% for rescue_organization in rescue_organizations %}
-            {% include "fellchensammlung/partials/partial-rescue-organization.html" %}
-        {% endfor %}
-    {% else %}
-        <p>{% translate "Keine Tierschutzorganisationen gefunden." %}</p>
-    {% endif %}
-</div>

src/fellchensammlung/templates/fellchensammlung/lists/list-notifications.html

@@ -1,10 +1,5 @@
-{% load i18n %}
 <div class="container-cards">
-    {% if notifications %}
+    {% for notification in notifications %}
-        {% for notification in notifications %}
+        {% include "fellchensammlung/partials/partial-notification.html" %}
-            {% include "fellchensammlung/partials/partial-notification.html" %}
+    {% endfor %}
-        {% endfor %}
-    {% else %}
-        <p>{% translate 'Keine ungelesenen Benachrichtigungen' %}</p>
-    {% endif %}
 </div>

src/fellchensammlung/templates/fellchensammlung/lists/list-search-subscriptions.html

@@ -1,10 +0,0 @@
-{% load i18n %}
-<div class="container-cards">
-    {% if search_subscriptions %}
-        {% for search_subscription in search_subscriptions %}
-            {% include "fellchensammlung/partials/partial-search-subscription.html" %}
-        {% endfor %}
-    {% else %}
-        <p>{% translate 'Keine abonnierten Suchen' %}</p>
-    {% endif %}
-</div>

src/fellchensammlung/templates/fellchensammlung/partials/partial-adoption-notice-minimal.html

@@ -2,34 +2,31 @@
 {% load i18n %}
 
 <div class="card">
-    <div class="header-card-adoption-notice">
+    <div>
-        <h2 class="heading-card-adoption-notice"><a
+        <div class="header-card-adoption-notice">
-                href="{{ adoption_notice.get_absolute_url }}"> {{ adoption_notice.name }}</a></h2>
+            <h1><a class="heading-card-adoption-notice"
-        <div class="tooltip bottom">
+                   href="{{ adoption_notice.get_absolute_url }}"> {{ adoption_notice.name }}</a></h1>
             <a class="adoption-card-report-link" href="{{ adoption_notice.get_report_url }}"><i
                     class="fa-solid fa-flag"></i></a>
-            <span class="tooltiptext">
-                        {% translate 'Melde diese Vermittlung an Moderator*innen' %}
-                    </span>
         </div>
+        <p>
+            <b><i class="fa-solid fa-location-dot"></i></b>
+            {% if adoption_notice.location %}
+                {{ adoption_notice.location.str_hr }}
+            {% else %}
+                {{ adoption_notice.location_string }}
+            {% endif %}
+        </p>
+        <p>
+            {% if adoption_notice.description_short %}
+                {{ adoption_notice.description_short | render_markdown }}
+            {% endif %}
+        </p>
+        {% if adoption_notice.get_photo %}
+            <div class="adoption-notice-img img-small">
+                <img src="{{ MEDIA_URL }}/{{ adoption_notice.get_photo.image }}"
+                     alt="{{ adoption_notice.get_photo.alt_text }}">
+            </div>
+        {% endif %}
     </div>
-    <p>
+</div>
-        <b><i class="fa-solid fa-location-dot"></i></b>
-        {% if adoption_notice.location %}
-            {{ adoption_notice.location.str_hr }}
-        {% else %}
-            {{ adoption_notice.location_string }}
-        {% endif %}
-    </p>
-    <p>
-        {% if adoption_notice.description_short %}
-            {{ adoption_notice.description_short | render_markdown }}
-        {% endif %}
-    </p>
-    {% if adoption_notice.get_photo %}
-        <div class="adoption-notice-img img-small">
-            <img src="{{ MEDIA_URL }}/{{ adoption_notice.get_photo.image }}"
-                 alt="{{ adoption_notice.get_photo.alt_text }}">
-        </div>
-    {% endif %}
-</div>

src/fellchensammlung/templates/fellchensammlung/partials/partial-animal-card.html

@@ -1,5 +1,4 @@
 {% load i18n %}
-{% load custom_tags %}
 <div class="card">
     <div class="detail-animal-header">
         <h1><a href="{% url 'animal-detail' animal_id=animal.pk %}">{{ animal.name }}</a></h1>
@@ -20,7 +19,7 @@
     </div>
 
     {% if animal.description %}
-        <p>{{ animal.description | render_markdown }}</p>
+        <p>{{ animal.description }}</p>
     {% endif %}
     {% for photo in  animal.get_photos %}
         <img src="{{ MEDIA_URL }}/{{ photo.image }}" alt="{{ photo.alt_text }}">

src/fellchensammlung/templates/fellchensammlung/partials/partial-announcement.html

@@ -1,11 +1,10 @@
 {% load i18n %}
 {% load custom_tags %}
 <div class="announcement {{ announcement.type }}">
-    <details class="announcement" open>
+    <div class="announcement-header">
-        <summary class="announcement-header">{{ announcement.title }}</summary>
+        <h1 class="announcement">{{ announcement.title }}</h1>
-        <p>
+    </div>
-            {{ announcement.content | render_markdown }}
+    <p>
-        </p>
+        {{ announcement.content | render_markdown }}
-    </details>
+    </p>
-
 </div>

src/fellchensammlung/templates/fellchensammlung/partials/partial-check-adoption-notice.html

@@ -4,11 +4,10 @@
     <h1>
         <a href="{{ adoption_notice.get_absolute_url }}">{{ adoption_notice.name }}</a>
     </h1>
-    <i>{% translate 'Zuletzt geprüft:' %} {{ adoption_notice.last_checked_hr }}</i>
     {% if adoption_notice.further_information %}
         <p>{% translate "Externe Quelle" %}: {{ adoption_notice.link_to_more_information | safe }}</p>
     {% endif %}
-    <div class="container-edit-buttons">
+    <div>
         <form method="post">
             {% csrf_token %}
             <input type="hidden"

src/fellchensammlung/templates/fellchensammlung/partials/partial-comment-section.html

@@ -1,7 +1,7 @@
 {% load i18n %}
 
 <div class="container-comments">
-    <h2>{% translate 'Kommentare' %}</h2>
+    <h2>{% translate 'Comments' %}</h2>
     {% if adoption_notice.comments %}
         {% for comment in adoption_notice.comments %}
             {% include "fellchensammlung/partials/partial-comment.html" %}

src/fellchensammlung/templates/fellchensammlung/partials/partial-map.html

@@ -6,141 +6,35 @@
 <script src="{% settings_value "MAP_TILE_SERVER" %}/assets/maplibre-gl/maplibre-gl.js"></script>
 <link href="{% settings_value "MAP_TILE_SERVER" %}/assets/maplibre-gl/maplibre-gl.css" rel="stylesheet"/>
 
-<!-- add Turf see https://maplibre.org/maplibre-gl-js/docs/examples/draw-a-radius/ -->
-<script src="{% static 'fellchensammlung/js/turf.min.js' %}"></script>
-
 <!-- add container for the map -->
 <div id="map" style="width:100%;aspect-ratio:16/9"></div>
 
 <!-- start map -->
 <script>
-    {% if zoom_level %}
-        var zoom_level = {{ zoom_level }};
-    {% else %}
-        var zoom_level = 4;
-    {% endif %}
-
-    {% if map_center %}
-        var map_center = [{{ map_center.longitude | pointdecimal }}, {{ map_center.latitude | pointdecimal }}];
-    {% else %}
-        var map_center = [10.49, 50.68]; <!-- Point middle of Germany -->
-        zoom_level = 4;  //Overwrite zoom level when no place is found
-    {% endif %}
 
     let map = new maplibregl.Map({
         container: 'map',
         style: '{% static "fellchensammlung/map/styles/colorful.json" %}',
-        center: map_center,
+        center: [10.49, 50.68],
-        zoom: zoom_level
+        zoom: 5
     }).addControl(new maplibregl.NavigationControl());
 
     {% for adoption_notice in adoption_notices_map %}
         {% if adoption_notice.location %}
-            // create the popup
+        // create the popup
-            const popup_{{ forloop.counter }} = new maplibregl.Popup({offset: 25}).setHTML(`{% include "fellchensammlung/partials/partial-adoption-notice-minimal.html" %}`);
+        const popup_{{ forloop.counter }} = new maplibregl.Popup({offset: 25}).setHTML(`{% include "fellchensammlung/partials/partial-adoption-notice-minimal.html" %}`);
 
-            // create DOM element for the marker
+        // create DOM element for the marker
-            const el_{{ forloop.counter }} = document.createElement('div');
+        const el_{{ forloop.counter }} = document.createElement('div');
-            el_{{ forloop.counter }}.id = 'marker_{{ forloop.counter }}';
+        el_{{ forloop.counter }}.id = 'marker_{{ forloop.counter }}';
-            el_{{ forloop.counter }}.classList.add('marker');
+        el_{{ forloop.counter }}.classList.add('marker');
 
-            const location_popup_{{ forloop.counter }} = [{{ adoption_notice.location.longitude | pointdecimal }}, {{ adoption_notice.location.latitude | pointdecimal }}];
+        const location_popup_{{ forloop.counter }} = [{{ adoption_notice.location.longitude | pointdecimal }}, {{ adoption_notice.location.latitude | pointdecimal }}];
-            // create the marker
+        // create the marker
-            new maplibregl.Marker({element: el_{{ forloop.counter }}})
+        new maplibregl.Marker({element: el_{{ forloop.counter }}})
-                .setLngLat(location_popup_{{ forloop.counter }})
+            .setLngLat(location_popup_{{ forloop.counter }})
-                .setPopup(popup_{{ forloop.counter }}) // sets a popup on this marker
+            .setPopup(popup_{{ forloop.counter }}) // sets a popup on this marker
-                .addTo(map);
+            .addTo(map);
         {% endif %}
     {% endfor %}
-
-    {% for rescue_organization in rescue_organizations %}
-        {% if rescue_organization.location %}
-            // create the popup
-            const popup_{{ forloop.counter }} = new maplibregl.Popup({offset: 25}).setHTML(`{% include "fellchensammlung/partials/partial-rescue-organization.html" %}`);
-
-            // create DOM element for the marker
-            const el_{{ forloop.counter }} = document.createElement('div');
-            el_{{ forloop.counter }}.id = 'marker_{{ forloop.counter }}';
-            el_{{ forloop.counter }}.classList.add('animal-shelter-marker', 'marker');
-
-            const location_popup_{{ forloop.counter }} = [{{ rescue_organization.location.longitude | pointdecimal }}, {{ rescue_organization.location.latitude | pointdecimal }}];
-            // create the marker
-            new maplibregl.Marker({element: el_{{ forloop.counter }}})
-                .setLngLat(location_popup_{{ forloop.counter }})
-                .setPopup(popup_{{ forloop.counter }}) // sets a popup on this marker
-                .addTo(map);
-        {% endif %}
-    {% endfor %}
-
-    map.on('load', async () => {
-        image = await map.loadImage('{% static "fellchensammlung/img/pin.png" %}');
-        map.addImage('pin', image.data);
-        {% for map_pin in map_pins %}
-            map.addSource('point', {
-                'type': 'geojson',
-                'data': {
-                    'type': 'FeatureCollection',
-                    'features': [
-                        {
-                            'type': 'Feature',
-                            'geometry': {
-                                'type': 'Point',
-                                'coordinates': [{{ map_pin.location.longitude | pointdecimal }}, {{ map_pin.location.latitude | pointdecimal }}]
-                            }
-                        }
-                    ]
-                }
-            });
-        {% endfor %}
-        map.addLayer({
-            'id': 'pints',
-            'type': 'symbol',
-            'source': 'point',
-            'layout': {
-                'icon-image': 'pin',
-                'icon-size': 0.1
-            }
-        });
-    });
-
-    {% if search_center  %}
-        var search_center = [{{ search_center.longitude | pointdecimal }}, {{ search_center.latitude | pointdecimal }}];
-        map.on('load', () => {
-            const radius = {{ search_radius }}; // kilometer
-            const options = {
-                steps: 64,
-                units: 'kilometers'
-            };
-            const circle = turf.circle(search_center, radius, options);
-
-            // Add the circle as a GeoJSON source
-            map.addSource('location-radius', {
-                type: 'geojson',
-                data: circle
-            });
-
-            // Add a fill layer with some transparency
-            map.addLayer({
-                id: 'location-radius',
-                type: 'fill',
-                source: 'location-radius',
-                paint: {
-                    'fill-color': 'rgba(140,207,255,0.3)',
-                    'fill-opacity': 0.5
-                }
-            });
-
-            // Add a line layer to draw the circle outline
-            map.addLayer({
-                id: 'location-radius-outline',
-                type: 'line',
-                source: 'location-radius',
-                paint: {
-                    'line-color': '#0094ff',
-                    'line-width': 3
-                }
-            });
-        });
-    {% endif %}
-
 </script>

src/fellchensammlung/templates/fellchensammlung/partials/partial-report.html

@@ -16,7 +16,7 @@
     <p><b>{% translate "Kommentar zur Meldung" %}:</b>
         {{ report.user_comment }}
     </p>
-    <div class="container-edit-buttons">
+    <div>
         <form action="allow" class="">
             {% csrf_token %}
             <input type="hidden" name="report_id" value="{{ report.pk }}">

src/fellchensammlung/templates/fellchensammlung/partials/partial-rescue-organization.html

@@ -1,22 +0,0 @@
-{% load custom_tags %}
-{% load i18n %}
-
-<div class="card">
-    <div>
-        <h2 class="heading-card-adoption-notice"><a
-                href="{{ rescue_organization.get_absolute_url }}"> {{ rescue_organization.name }}</a></h2>
-        <p>
-            <b><i class="fa-solid fa-location-dot"></i></b>
-            {% if rescue_organization.location %}
-                {{ rescue_organization.location.str_hr }}
-            {% else %}
-                {{ rescue_organization.location_string }}
-            {% endif %}
-        </p>
-        <p>
-            {% if rescue_organization.description_short %}
-                {{ rescue_organization.description_short | render_markdown }}
-            {% endif %}
-        </p>
-    </div>
-</div>

src/fellchensammlung/templates/fellchensammlung/partials/partial-search-subscription.html

@@ -1,25 +0,0 @@
-{% load i18n %}
-{% load custom_tags %}
-<div class="search-subscription">
-    <div class="search-subscription-header">
-        <h3>{{ search_subscription }}</h3>
-        <form class="" method="POST">
-            {% csrf_token %}
-            <input type="hidden" name="action" value="search_subscription_delete">
-            <input type="hidden" name="search_subscription_id" value="{{ search_subscription.pk }}">
-            <button class="btn3" type="submit" id="submit"><i class="fa-solid fa-close"></i></button>
-        </form>
-    </div>
-    <table>
-        <tr>
-            <th>{% trans 'Geschlecht' %}</th>
-            <th>{% trans 'Suchort' %}</th>
-            <th>{% trans 'Suchradius' %}</th>
-        </tr>
-        <tr>
-            <td>{{ search_subscription.sex }}</td>
-            <td>{{ search_subscription.location }}</td>
-            <td>{{ search_subscription.max_distance }}km</td>
-        </tr>
-    </table>
-</div>

src/fellchensammlung/templates/fellchensammlung/search.html

@@ -4,85 +4,15 @@
 {% block title %}<title>{% translate "Suche" %}</title>{% endblock %}
 
 {% block content %}
-    {% get_current_language as LANGUAGE_CODE_CURRENT %}
+    <form class="form-search card" method="post">
-    <div class="container-cards">
+        {% csrf_token %}
-        <form class="form-search card half" method="post">
+        <input type="hidden" name="longitude" maxlength="200" id="longitude">
-            {% csrf_token %}
+        <input type="hidden" name="latitude" maxlength="200" id="latitude">
-            <input type="hidden" name="longitude" maxlength="200" id="longitude">
+        {{ search_form.as_p }}
-            <input type="hidden" name="latitude" maxlength="200" id="latitude">
+        <input class="btn" type="submit" value="Search" name="search">
-            <input type="hidden" id="place_id" name="place_id">
+    </form>
-            {{ search_form.as_p }}
+    {% if place_not_found %}
-            <ul id="results"></ul>
+        <p class="error">{% translate "Ort nicht gefunden" %}</p>
-            <div class="container-edit-buttons">
+    {% endif %}
-                <button class="btn" type="submit" value="search" name="search">
-                    <i class="fas fa-search"></i> {% trans 'Suchen' %}
-                </button>
-                {% if searched %}
-                    {% if subscribed_search %}
-                        <button class="btn" type="submit" value="{{ subscribed_search.pk }}"
-                                name="unsubscribe_to_search">
-                            <i class="fas fa-bell-slash"></i> {% trans 'Suche nicht mehr abonnieren' %}
-                        </button>
-                    {% else %}
-                        <button class="btn" type="submit" name="subscribe_to_search">
-                            <i class="fas fa-bell"></i> {% trans 'Suche abonnieren' %}
-                        </button>
-                    {% endif %}
-                {% endif %}
-            </div>
-        {% if place_not_found %}
-            <p class="error">
-                {% trans 'Ort nicht gefunden' %}
-            </p>
-        {% endif %}
-        </form>
-        <div class="card half">
-            {% include "fellchensammlung/partials/partial-map.html" %}
-        </div>
-    </div>
     {% include "fellchensammlung/lists/list-adoption-notices.html" %}
-
-    <script>
-        const locationInput = document.getElementById('id_location_string');
-        const resultsList = document.getElementById('results');
-        const placeIdInput = document.getElementById('place_id');
-
-        locationInput.addEventListener('input', async function () {
-            const query = locationInput.value.trim();
-
-            if (query.length < 3) {
-                resultsList.innerHTML = ''; // Don't search for or show results if input is less than 3 characters
-                return;
-            }
-
-            try {
-                const response = await fetch(`{{ geocoding_api_url }}/?q=${encodeURIComponent(query)}&limit=5&lang={{ LANGUAGE_CODE_CURRENT }}`);
-                const data = await response.json();
-
-                if (data && data.features) {
-                    resultsList.innerHTML = ''; // Clear previous results
-
-                    const locations = data.features.slice(0, 5); // Show only the first 5 results
-
-                    locations.forEach(location => {
-                        const listItem = document.createElement('li');
-                        listItem.classList.add('result-item');
-                        listItem.textContent = geojson_to_summary(location);
-
-                        // Add event when user clicks on a result location
-                        listItem.addEventListener('click', () => {
-
-                            locationInput.value = geojson_to_searchable_string(location); // Set input field to selected location
-                            resultsList.innerHTML = ''; // Clear the results after selecting a location
-                        });
-
-                        resultsList.appendChild(listItem);
-                    });
-                }
-            } catch (error) {
-                console.error('Error fetching location data:', error);
-                resultsList.innerHTML = '<li class="result-item">Error fetching data. Please try again.</li>';
-            }
-        });
-    </script>
 {% endblock %}

src/fellchensammlung/tools/admin.py

@@ -3,8 +3,7 @@ import logging
 from django.utils import timezone
 from datetime import timedelta
 
-from fellchensammlung.models import AdoptionNotice, Location, RescueOrganization, AdoptionNoticeStatus, Log, \
+from fellchensammlung.models import AdoptionNotice, Location, RescueOrganization, AdoptionNoticeStatus, Log
-    AdoptionNoticeNotification
 from fellchensammlung.tools.misc import is_404
 
 
@@ -83,10 +82,3 @@ def deactivate_404_adoption_notices():
                 logging_msg = f"Automatically set Adoption Notice {adoption_notice.id} closed as link to more information returened 404"
                 logging.info(logging_msg)
                 Log.objects.create(action="automated", text=logging_msg)
-
-                deactivation_message = f'Die Vermittlung  [{adoption_notice.name}]({adoption_notice.get_absolute_url()}) wurde automatisch deaktiviert, da die Website unter "Mehr Informationen" nicht mehr online ist.'
-                for subscription in adoption_notice.get_subscriptions():
-                    AdoptionNoticeNotification.objects.create(user=subscription.owner,
-                                                               title="Vermittlung deaktiviert",
-                                                               adoption_notice=adoption_notice,
-                                                               text=deactivation_message)

src/fellchensammlung/tools/geo.py

@@ -1,6 +1,4 @@
 import logging
-from collections import namedtuple
-
 import requests
 import json
 from math import radians, sqrt, sin, cos, atan2
@@ -8,23 +6,6 @@ from math import radians, sqrt, sin, cos, atan2
 from notfellchen import __version__ as nf_version
 from notfellchen import settings
 
-Position = namedtuple('Position', ['latitude', 'longitude'])
-
-
-def zoom_level_for_radius(radius) -> int:
-    if radius is None:
-        return 4
-    if radius <= 20:
-        return 8
-    if radius <= 50:
-        return 7
-    if radius <= 150:
-        return 6
-    if radius <= 300:
-        return 5
-    else:
-        return 4
-
 
 def calculate_distance_between_coordinates(position1, position2):
     """
@@ -48,8 +29,6 @@ def calculate_distance_between_coordinates(position1, position2):
 
     distance_in_km = earth_radius_km * c
 
-    logging.debug(f"Calculated Distance: {distance_in_km:.5}km")
-
     return distance_in_km
 
 
@@ -67,44 +46,8 @@ class RequestMock:
         return ResponseMock()
 
 
-class GeoFeature:
-
-    @staticmethod
-    def geofeatures_from_photon_result(result):
-        geofeatures = []
-        for feature in result["features"]:
-            geojson = {}
-            try:
-                geojson['name'] = feature["properties"]["name"]
-            except KeyError:
-                geojson['name'] = feature["properties"]["street"]
-            geojson['place_id'] = feature["properties"]["osm_id"]
-            geojson['lat'] = feature["geometry"]["coordinates"][1]
-            geojson['lon'] = feature["geometry"]["coordinates"][0]
-            geofeatures.append(geojson)
-        return geofeatures
-
-    @staticmethod
-    def geofeatures_from_nominatim_result(result):
-        geofeatures = []
-        for feature in result:
-            geojson = {}
-            if "name" in feature:
-                geojson['name'] = feature["name"]
-            else:
-                geojson['name'] = feature["display_name"]
-            geojson['place_id'] = feature["place_id"]
-            geojson['lat'] = feature["lat"]
-            geojson['lon'] = feature["lon"]
-            geofeatures.append(geojson)
-        return geofeatures
-
-
 class GeoAPI:
     api_url = settings.GEOCODING_API_URL
-    api_format = settings.GEOCODING_API_FORMAT
-    assert api_format in ['nominatim', 'photon']
-
     # Set User-Agent headers as required by most usage policies (and it's the nice thing to do)
     headers = {
         'User-Agent': f"Notfellchen {nf_version}",
@@ -119,68 +62,34 @@ class GeoAPI:
         else:
             self.requests = requests
 
+    def get_coordinates_from_query(self, location_string):
+        try:
+            result = \
+            self.requests.get(self.api_url, {"q": location_string, "format": "jsonv2"}, headers=self.headers).json()[0]
+        except IndexError:
+            return None
+        return result["lat"], result["lon"]
+
     def _get_raw_response(self, location_string):
         result = self.requests.get(self.api_url, {"q": location_string, "format": "jsonv2"}, headers=self.headers)
         return result.content
 
-    def get_geojson_for_query(self, location_string, language="de"):
+    def get_geojson_for_query(self, location_string):
         try:
-            if self.api_format == 'nominatim':
+            result = self.requests.get(self.api_url,
-                logging.info(f"Querying nominatim instance for: {location_string} ({self.api_url})")
+                                       {"q": location_string,
-                result = self.requests.get(self.api_url,
+                                        "format": "jsonv2"},
-                                           {"q": location_string,
+                                       headers=self.headers).json()
-                                            "format": "jsonv2"},
-                                           headers=self.headers).json()
-                geofeatures = GeoFeature.geofeatures_from_nominatim_result(result)
-            elif self.api_format == 'photon':
-                logging.info(f"Querying photon instance for: {location_string} ({self.api_url})")
-                result = self.requests.get(self.api_url,
-                                           {"q": location_string, "lang": language},
-                                           headers=self.headers).json()
-                logging.warning(result)
-                geofeatures = GeoFeature.geofeatures_from_photon_result(result)
-            else:
-                raise NotImplementedError
-
         except Exception as e:
-            logging.warning(f"Exception {e} when querying geocoding server")
+            logging.warning(f"Exception {e} when querying Nominatim")
             return None
-        if len(geofeatures) == 0:
+        if len(result) == 0:
-            logging.warning(f"Couldn't find a result for {location_string} when querying geocoding server")
+            logging.warning(f"Couldn't find a result for {location_string} when querying Nominatim")
             return None
-        return geofeatures
+        return result
-
-
-class LocationProxy:
-    """
-    Location proxy is used as a precursor to the location model without the need to create unnecessary database objects
-    """
-
-    def __init__(self, location_string):
-        """
-        Creates the location proxy from the location string
-        """
-        self.geo_api = GeoAPI()
-        geofeatures = self.geo_api.get_geojson_for_query(location_string)
-        if geofeatures is None:
-            raise ValueError
-        result = geofeatures[0]
-        self.name = result["name"]
-        self.place_id = result["place_id"]
-        self.latitude = result["lat"]
-        self.longitude = result["lon"]
-
-    def __eq__(self, other):
-        return self.place_id == other.place_id
-
-    def __str__(self):
-        return self.name
-
-    @property
-    def position(self):
-        return (self.latitude, self.longitude)
 
 
 if __name__ == "__main__":
     geo = GeoAPI(debug=False)
+    print(geo.get_coordinates_from_query("12101"))
     print(calculate_distance_between_coordinates(('48.4949904', '9.040330235970146'), ("48.648333", "9.451111")))

src/fellchensammlung/tools/misc.py

@@ -1,9 +1,6 @@
 import datetime as datetime
 import logging
 
-from django.utils.translation import ngettext
-from django.utils.translation import gettext as _
-
 from notfellchen import settings
 import requests
 
@@ -32,31 +29,6 @@ def age_as_hr_string(age: datetime.timedelta) -> str:
         return f'{days:.0f} Tag{pluralize(days)}'
 
 
-def time_since_as_hr_string(age: datetime.timedelta) -> str:
-    days = age.days
-    weeks = age.days / 7
-    months = age.days / 30
-    years = age.days / 365
-    if years >= 1:
-        text = ngettext(
-            "vor einem Jahr",
-            "vor %(years)d Tagen",
-            years,
-        ) % {
-                   "years": years,
-               }
-    elif months >= 3:
-        text = _("vor %(month)d Monaten") % {"month": months}
-    elif weeks >= 3:
-        text = _("vor %(weeks)d Wochen") % {"weeks": weeks}
-    else:
-        if days == 0:
-            text = _("Heute")
-        else:
-            text = ngettext("vor einem Tag","vor %(count)d Tagen", days,) % {"count": days,}
-    return text
-
-
 def healthcheck_ok():
     try:
         requests.get(settings.HEALTHCHECKS_URL, timeout=10)

src/fellchensammlung/tools/notifications.py

@@ -1,11 +0,0 @@
-from fellchensammlung.models import User, AdoptionNoticeNotification, TrustLevel
-
-
-def notify_moderators_of_AN_to_be_checked(adoption_notice):
-    if adoption_notice.is_disabled_unchecked:
-        for moderator in User.objects.filter(trust_level__gt=TrustLevel.MODERATOR):
-            AdoptionNoticeNotification.objects.create(adoption_notice=adoption_notice,
-                                                      user=moderator,
-                                                      title=f" Prüfe Vermittlung {adoption_notice}",
-                                                      text=f"{adoption_notice} muss geprüft werden bevor sie veröffentlicht wird.",
-                                                      )

src/fellchensammlung/tools/search.py

@@ -1,151 +0,0 @@
-import logging
-from django.utils.translation import gettext_lazy as _
-
-from .geo import LocationProxy, Position
-from ..forms import AdoptionNoticeSearchForm
-from ..models import SearchSubscription, AdoptionNotice, AdoptionNoticeNotification, SexChoicesWithAll, Location
-
-
-def notify_search_subscribers(adoption_notice: AdoptionNotice, only_if_active : bool = True):
-    """
-    This functions checks for all search subscriptions if the new adoption notice fits the search.
-    If the new adoption notice fits the search subscription, it sends a notification to the user that created the search.
-    """
-    logging.debug(f"Notifying {adoption_notice}.")
-    if only_if_active and not adoption_notice.is_active:
-        logging.debug(f"No notifications triggered for adoption notice {adoption_notice} because it's not active.")
-        return
-    for search_subscription in SearchSubscription.objects.all():
-        logging.debug(f"Search subscription {search_subscription} found.")
-        search = Search(search_subscription=search_subscription)
-        if search.adoption_notice_fits_search(adoption_notice):
-            notification_text = f"{_('Zu deiner Suche')} {search_subscription} wurde eine neue Vermittlung gefunden"
-            AdoptionNoticeNotification.objects.create(user=search_subscription.owner,
-                                                      title=f"{_('Neue Vermittlung')}: {adoption_notice}",
-                                                      adoption_notice=adoption_notice,
-                                                      text=notification_text)
-            logging.debug(f"Notification for search subscription {search_subscription} was sent.")
-        else:
-            logging.debug(f"Adoption notice {adoption_notice} was not fitting the search subscription.")
-
-    logging.info(f"Subscribers for AN {adoption_notice.pk} have been notified\n")
-
-
-class Search:
-    def __init__(self, request=None, search_subscription=None):
-        self.sex = None
-        self.area_search = None
-        self.max_distance = None
-        self.location = None # Can either be Location (DjangoModel) or LocationProxy
-        self.place_not_found = False  # Indicates that a location was given but could not be geocoded
-        self.search_form = None
-        # Either place_id or location string must be set for area search
-        self.location_string = None
-
-        if request:
-            self.search_from_request(request)
-        elif search_subscription:
-            self.search_from_search_subscription(search_subscription)
-
-
-    def __str__(self):
-        return f"Search: {self.sex=}, {self.location=}, {self.area_search=}, {self.max_distance=}"
-
-    def __eq__(self, other):
-        """
-        Custom equals that also supports SearchSubscriptions
-
-        Only allowed to be called for located subscriptions
-        """
-        # If both locations are empty check only for sex
-        if self.location is None and other.location is None:
-            return self.sex == other.sex
-        # If one location is empty and the other is not, they are not equal
-        elif self.location is not None and other.location is None or self.location is None and other.location is not None:
-            return False
-        return self.location == other.location and self.sex == other.sex and self.max_distance == other.max_distance
-
-    def _locate(self):
-        try:
-            self.location = LocationProxy(self.location_string)
-        except ValueError:
-            self.place_not_found = True
-
-    @property
-    def position(self):
-        if self.area_search and not self.place_not_found:
-            return Position(latitude=self.location.latitude, longitude=self.location.longitude)
-        else:
-            return None
-
-    def adoption_notice_fits_search(self, adoption_notice: AdoptionNotice):
-        # Make sure sex is set and sex is not set to all (then it can be disregarded)
-        if self.sex is not None and self.sex != SexChoicesWithAll.ALL:
-            # AN does not fit search if search sex is not in available sexes of this AN
-            if not self.sex in adoption_notice.sexes:
-                logging.debug("Sex mismatch")
-                return False
-        # make sure it's an area search and the place is found to check location
-        if self.area_search and not self.place_not_found:
-            # If adoption notice is in not in search distance, return false
-            if not adoption_notice.in_distance(self.location.position, self.max_distance):
-                logging.debug("Area mismatch")
-                return False
-        return True
-
-
-    def get_adoption_notices(self):
-        adoptions = AdoptionNotice.objects.order_by("-created_at")
-        # Filter for active adoption notices
-        adoptions = [adoption for adoption in adoptions if adoption.is_active]
-        # Check if adoption notice fits search.
-        adoptions = [adoption for adoption in adoptions if self.adoption_notice_fits_search(adoption)]
-
-        return adoptions
-
-    def search_from_request(self, request):
-        if request.method == 'POST':
-            self.search_form = AdoptionNoticeSearchForm(request.POST)
-            self.search_form.is_valid()
-            self.sex = self.search_form.cleaned_data["sex"]
-
-            if self.search_form.cleaned_data["location_string"] != "" and self.search_form.cleaned_data[
-                "max_distance"] != "":
-                self.area_search = True
-                self.location_string = self.search_form.cleaned_data["location_string"]
-                self.max_distance = int(self.search_form.cleaned_data["max_distance"])
-                self._locate()
-        else:
-            self.search_form = AdoptionNoticeSearchForm()
-
-    def search_from_search_subscription(self, search_subscription: SearchSubscription):
-        self.sex = search_subscription.sex
-        self.location = search_subscription.location
-        self.area_search = True
-        self.max_distance = search_subscription.max_distance
-
-
-    def subscribe(self, user):
-        logging.info(f"{user} subscribed to search")
-        if isinstance(self.location, LocationProxy):
-            self.location = Location.get_location_from_proxy(self.location)
-        SearchSubscription.objects.create(owner=user,
-                                          location=self.location,
-                                          sex=self.sex,
-                                          max_distance=self.max_distance)
-
-    def get_subscription_or_none(self, user):
-        user_subscriptions = SearchSubscription.objects.filter(owner=user)
-        for subscription in user_subscriptions:
-            if self == subscription:
-                return subscription
-
-    def is_subscribed(self, user):
-        """
-        Returns true if a user is already subscribed to a search with these parameters
-        """
-        subscription = self.get_subscription_or_none()
-        if subscription is None:
-            return False
-        else:
-            return True

src/fellchensammlung/urls.py

@@ -5,7 +5,6 @@ from .forms import CustomRegistrationForm
 from .feeds import LatestAdoptionNoticesFeed
 
 from . import views
-from drf_spectacular.views import SpectacularAPIView, SpectacularRedocView, SpectacularSwaggerView
 
 urlpatterns = [
     path("", views.index, name="index"),
@@ -25,8 +24,6 @@ urlpatterns = [
     path("vermittlung/<int:adoption_notice_id>/add-photo", views.add_photo_to_adoption_notice, name="adoption-notice-add-photo"),
     # ex: /adoption_notice/2/add-animal
     path("vermittlung/<int:adoption_notice_id>/add-animal", views.adoption_notice_add_animal, name="adoption-notice-add-animal"),
-
-    path("tierschutzorganisationen/", views.list_rescue_organizations, name="rescue-organizations"),
     path("organisation/<int:rescue_organization_id>/", views.detail_view_rescue_organization,
          name="rescue-organization-detail"),
 
@@ -85,10 +82,6 @@ urlpatterns = [
     ## API ##
     #########
     path('api/', include('fellchensammlung.api.urls')),
-    path('api/schema/', SpectacularAPIView.as_view(), name='schema'),
-    # Optional UI:
-    path('api/schema/swagger-ui/', SpectacularSwaggerView.as_view(url_name='schema'), name='swagger-ui'),
-    path('api/schema/redoc/', SpectacularRedocView.as_view(url_name='schema'), name='redoc'),
 
     ###################
     ## External Site ##

src/fellchensammlung/views.py

@@ -1,6 +1,5 @@
 import logging
 
-from django.contrib.auth.views import redirect_to_login
 from django.http import HttpResponseRedirect, JsonResponse, HttpResponse
 from django.shortcuts import render, redirect
 from django.urls import reverse
@@ -17,20 +16,17 @@ from notfellchen import settings
 from fellchensammlung import logger
 from .models import AdoptionNotice, Text, Animal, Rule, Image, Report, ModerationAction, \
     User, Location, AdoptionNoticeStatus, Subscriptions, CommentNotification, BaseNotification, RescueOrganization, \
-    Species, Log, Timestamp, TrustLevel, SexChoicesWithAll, SearchSubscription, AdoptionNoticeNotification
+    Species, Log, Timestamp, TrustLevel, SexChoicesWithAll
 from .forms import AdoptionNoticeForm, AdoptionNoticeFormWithDateWidget, ImageForm, ReportAdoptionNoticeForm, \
     CommentForm, ReportCommentForm, AnimalForm, \
     AdoptionNoticeSearchForm, AnimalFormWithDateWidget, AdoptionNoticeFormWithDateWidgetAutoAnimal
 from .models import Language, Announcement
-from .tools.geo import GeoAPI, zoom_level_for_radius
+from .tools.geo import GeoAPI
 from .tools.metrics import gather_metrics_data
-from .tools.admin import clean_locations, get_unchecked_adoption_notices, deactivate_unchecked_adoption_notices, \
+from .tools.admin import clean_locations, get_unchecked_adoption_notices, deactivate_unchecked_adoption_notices
-    deactivate_404_adoption_notices
+from .tasks import add_adoption_notice_location
-from .tasks import post_adoption_notice_save
 from rest_framework.authtoken.models import Token
 
-from .tools.search import Search
-
 
 def user_is_trust_level_or_above(user, trust_level=TrustLevel.MODERATOR):
     return user.is_authenticated and user.trust_level >= trust_level
@@ -172,44 +168,37 @@ def animal_detail(request, animal_id):
 
 
 def search(request):
-    # A user just visiting the search site did not search, only upon completing the search form a user has really
+    place_not_found = None
-    # searched. This will toggle the "subscribe" button
+    latest_adoption_list = AdoptionNotice.objects.order_by("-created_at")
-    searched = False
+    active_adoptions = [adoption for adoption in latest_adoption_list if adoption.is_active]
-    search = Search()
-    search.search_from_request(request)
     if request.method == 'POST':
-        searched = True
-        if "subscribe_to_search" in request.POST:
-            # Make sure user is logged in
-            if not request.user.is_authenticated:
-                return redirect_to_login(next=request.path)
-            search.subscribe(request.user)
-        if "unsubscribe_to_search" in request.POST:
-            if not request.user.is_authenticated:
-                return redirect_to_login(next=request.path)
-            search_subscription = SearchSubscription.objects.get(pk=request.POST["unsubscribe_to_search"])
-            if search_subscription.owner == request.user:
-                search_subscription.delete()
-            else:
-                raise PermissionDenied
-    if request.user.is_authenticated:
-        subscribed_search = search.get_subscription_or_none(request.user)
-    else:
-        subscribed_search = None
 
-    context = {"adoption_notices": search.get_adoption_notices(),
+        sex = request.POST.get("sex")
-               "search_form": search.search_form,
+        if sex != SexChoicesWithAll.ALL:
-               "place_not_found": search.place_not_found,
+            active_adoptions = [adoption for adoption in active_adoptions if sex in adoption.sexes]
-               "subscribed_search": subscribed_search,
+
-               "searched": searched,
+        search_form = AdoptionNoticeSearchForm(request.POST)
-               "adoption_notices_map": AdoptionNotice.get_active_ANs(),
+        search_form.is_valid()
-               "map_center": search.position,
+        if search_form.cleaned_data["location"] == "":
-               "search_center": search.position,
+            adoption_notices_in_distance = active_adoptions
-               "map_pins": [search],
+            place_not_found = False
-               "location": search.location,
+        else:
-               "search_radius": search.max_distance,
+            max_distance = int(request.POST.get('max_distance'))
-               "zoom_level": zoom_level_for_radius(search.max_distance),
+            if max_distance == "":
-               "geocoding_api_url": settings.GEOCODING_API_URL, }
+                max_distance = None
+            geo_api = GeoAPI()
+            search_position = geo_api.get_coordinates_from_query(request.POST['location'])
+            if search_position is None:
+                place_not_found = True
+                adoption_notices_in_distance = active_adoptions
+            else:
+                adoption_notices_in_distance = [a for a in active_adoptions if a.in_distance(search_position, max_distance)]
+
+        context = {"adoption_notices": adoption_notices_in_distance, "search_form": search_form,
+                   "place_not_found": place_not_found}
+    else:
+        search_form = AdoptionNoticeSearchForm()
+        context = {"adoption_notices": active_adoptions, "search_form": search_form}
     return render(request, 'fellchensammlung/search.html', context=context)
 
 
@@ -220,13 +209,18 @@ def add_adoption_notice(request):
                                                           in_adoption_notice_creation_flow=True)
 
         if form.is_valid():
-            an_instance = form.save(commit=False)
+            instance = form.save(commit=False)
-            an_instance.owner = request.user
+            instance.owner = request.user
+            instance.save()
 
+            """Spin up a task that adds the location"""
+            add_adoption_notice_location.delay_on_commit(instance.pk)
+
+            # Set correct status
             if request.user.trust_level >= TrustLevel.MODERATOR:
-                an_instance.set_active()
+                instance.set_active()
             else:
-                an_instance.set_unchecked()
+                instance.set_unchecked()
 
             # Get the species and number of animals from the form
             species = form.cleaned_data["species"]
@@ -235,21 +229,13 @@ def add_adoption_notice(request):
             date_of_birth = form.cleaned_data["date_of_birth"]
             for i in range(0, num_animals):
                 Animal.objects.create(owner=request.user,
-                                      name=f"{species} {i + 1}", adoption_notice=an_instance, species=species, sex=sex,
+                                      name=f"{species} {i + 1}", adoption_notice=instance, species=species, sex=sex,
                                       date_of_birth=date_of_birth)
 
             """Log"""
             Log.objects.create(user=request.user, action="add_adoption_notice",
-                               text=f"{request.user} hat Vermittlung {an_instance.pk} hinzugefügt")
+                               text=f"{request.user} hat Vermittlung {instance.pk} hinzugefügt")
-
+            return redirect(reverse("adoption-notice-detail", args=[instance.pk]))
-            """Spin up a task that adds the location and notifies search subscribers"""
-            post_adoption_notice_save.delay(an_instance.id)
-
-            """Subscriptions"""
-            # Automatically subscribe user that created AN to AN
-            Subscriptions.objects.create(owner=request.user, adoption_notice=an_instance)
-
-            return redirect(reverse("adoption-notice-detail", args=[an_instance.pk]))
     else:
         form = AdoptionNoticeFormWithDateWidgetAutoAnimal(in_adoption_notice_creation_flow=True)
     return render(request, 'fellchensammlung/forms/form_add_adoption.html', {'form': form})
@@ -302,7 +288,7 @@ def add_photo_to_animal(request, animal_id):
                 form = ImageForm(in_flow=True)
                 return render(request, 'fellchensammlung/forms/form-image.html', {'form': form})
             else:
-                return redirect(reverse("adoption-notice-detail", args=[animal.adoption_notice.pk], ))
+                return redirect(reverse("animal-detail", args=[animal_id]))
     else:
         form = ImageForm(in_flow=True)
         return render(request, 'fellchensammlung/forms/form-image.html', {'form': form})
@@ -354,7 +340,7 @@ def animal_edit(request, animal_id):
             """Log"""
             Log.objects.create(user=request.user, action="add_photo_to_animal",
                                text=f"{request.user} hat Tier {animal.pk} zum Tier geändert")
-            return redirect(reverse("adoption-notice-detail", args=[animal.adoption_notice.pk], ))
+            return redirect(reverse("animal-detail", args=[animal.pk], ))
     else:
         form = AnimalForm(instance=animal)
     return render(request, 'fellchensammlung/forms/form-adoption-notice.html', context={"form": form})
@@ -367,7 +353,7 @@ def about(request):
     lang = Language.objects.get(languagecode=language_code)
 
     legal = {}
-    for text_code in ["terms_of_service", "privacy_statement", "imprint", "about_us", "faq"]:
+    for text_code in ["terms_of_service", "privacy_statement", "imprint", "about_us"]:
         try:
             legal[text_code] = Text.objects.get(text_code=text_code, language=lang, )
         except Text.DoesNotExist:
@@ -444,8 +430,7 @@ def report_detail_success(request, report_id):
 def user_detail(request, user, token=None):
     context = {"user": user,
                "adoption_notices": AdoptionNotice.objects.filter(owner=user),
-               "notifications": BaseNotification.objects.filter(user=user, read=False),
+               "notifications": BaseNotification.objects.filter(user=user, read=False)}
-               "search_subscriptions": SearchSubscription.objects.filter(owner=user), }
     if token is not None:
         context["token"] = token
     return render(request, 'fellchensammlung/details/detail-user.html', context=context)
@@ -469,10 +454,6 @@ def my_profile(request):
             Token.objects.create(user=request.user)
         elif "delete_token" in request.POST:
             Token.objects.get(user=request.user).delete()
-        elif "toggle_email_notifications" in request.POST:
-            user = request.user
-            user.email_notifications = not user.email_notifications
-            user.save()
 
         action = request.POST.get("action")
         if action == "notification_mark_read":
@@ -488,11 +469,6 @@ def my_profile(request):
             for notification in notifications:
                 notification.read = True
                 notification.save()
-        elif action == "search_subscription_delete":
-            search_subscription_id = request.POST.get("search_subscription_id")
-            SearchSubscription.objects.get(pk=search_subscription_id).delete()
-            logging.info(f"Deleted subscription {search_subscription_id}")
-
     try:
         token = Token.objects.get(user=request.user)
     except Token.DoesNotExist:
@@ -533,7 +509,7 @@ def updatequeue(request):
 
 
 def map(request):
-    adoption_notices = AdoptionNotice.get_active_ANs()
+    adoption_notices = AdoptionNotice.objects.all()  #TODO: Filter to active
     context = {"adoption_notices_map": adoption_notices}
     return render(request, 'fellchensammlung/map.html', context=context)
 
@@ -554,8 +530,6 @@ def instance_health_check(request):
             clean_locations(quiet=False)
         elif action == "deactivate_unchecked_adoption_notices":
             deactivate_unchecked_adoption_notices()
-        elif action == "deactivate_404":
-            deactivate_404_adoption_notices()
 
     number_of_adoption_notices = AdoptionNotice.objects.all().count()
     none_geocoded_adoption_notices = AdoptionNotice.objects.filter(location__isnull=True)
@@ -610,16 +584,9 @@ def external_site_warning(request):
     return render(request, 'fellchensammlung/external_site_warning.html', context=context)
 
 
-def list_rescue_organizations(request):
-    rescue_organizations = RescueOrganization.objects.all()
-    context = {"rescue_organizations": rescue_organizations}
-    return render(request, 'fellchensammlung/animal-shelters.html', context=context)
-
-
 def detail_view_rescue_organization(request, rescue_organization_id):
     org = RescueOrganization.objects.get(pk=rescue_organization_id)
-    return render(request, 'fellchensammlung/details/detail-rescue-organization.html',
+    return render(request, 'fellchensammlung/details/detail-rescue-organization.html', context={"org": org})
-                  context={"org": org, "map_center": org.position, "zoom_level": 6, "rescue_organizations": [org]})
 
 
 def export_own_profile(request):

src/notfellchen/__init__.py

@@ -1,4 +1,4 @@
-__version__ = "0.4.0"
+__version__ = "0.3.1"
 
 # This will make sure the app is always imported when
 # Django starts so that shared_task will use this app.

src/notfellchen/settings.py

@@ -90,9 +90,6 @@ HEALTHCHECKS_URL = config.get("monitoring", "healthchecks_url", fallback=None)
 
 """ GEOCODING """
 GEOCODING_API_URL = config.get("geocoding", "api_url", fallback="https://nominatim.hyteck.de/search")
-# GEOCODING_API_FORMAT is allowed to be one of ['nominatim', 'photon']
-GEOCODING_API_FORMAT = config.get("geocoding", "api_format", fallback="nominatim")
-
 """ Tile Server """
 MAP_TILE_SERVER = config.get("map", "tile_server", fallback="https://tiles.hyteck.de")
 
@@ -172,9 +169,7 @@ INSTALLED_APPS = [
     'crispy_forms',
     "crispy_bootstrap4",
     "rest_framework",
-    'rest_framework.authtoken',
+    'rest_framework.authtoken'
-    'drf_spectacular',
-    'drf_spectacular_sidecar',  # required for Django collectstatic discovery
 ]
 
 MIDDLEWARE = [
@@ -288,16 +283,5 @@ REST_FRAMEWORK = {
     'DEFAULT_AUTHENTICATION_CLASSES': [
         'rest_framework.authentication.TokenAuthentication',
         'rest_framework.authentication.SessionAuthentication',
-    ],
+    ]
-    'DEFAULT_SCHEMA_CLASS': 'drf_spectacular.openapi.AutoSchema',
-}
-
-SPECTACULAR_SETTINGS = {
-    'SWAGGER_UI_DIST': 'SIDECAR',  # shorthand to use the sidecar instead
-    'SWAGGER_UI_FAVICON_HREF': 'SIDECAR',
-    'REDOC_DIST': 'SIDECAR',
-    'TITLE': 'Notfellchen API',
-    'DESCRIPTION': 'Adopt a animal in need',
-    'VERSION': '1.0.0',
-    'SERVE_INCLUDE_SCHEMA': False,
 }

src/templates/django_registration/activation_complete.html

@@ -2,7 +2,5 @@
 {% load i18n %}
 
 {% block content %}
-    <div class="card">
+<p>{% translate "Dein Account ist nun aktiviert. Viel Spaß!" %}</p>
-        <p>{% translate "Dein Account ist nun aktiviert. Viel Spaß!" %}</p>
-    </div>
 {% endblock %}

src/templates/django_registration/activation_form.html

@@ -3,15 +3,13 @@
 {% load crispy_forms_tags %}
 
 {% block content %}
-    <div class="card">
     {% if not user.is_authenticated %}
         <form action="" method="post">
             {% csrf_token %}
             {{ form.as_p }}
-            <input type="submit" class="btn" value={% translate 'Absenden' %}>
+            <input type="submit" class="btn2" value={% translate 'Absenden' %}>
         </form>
     {% else %}
         <p>{% translate "Du bist bereits eingeloggt." %}</p>
     {% endif %}
-    </div>
 {% endblock %}

src/templates/django_registration/registration_complete.html

@@ -2,10 +2,5 @@
 {% load i18n %}
 
 {% block content %}
-    <p class="card">
+<p>{% translate "Du bist nun registriert. Du hast eine E-Mail mit einem Link zur Aktivierung deines Kontos bekommen." %}</p>
-        {% blocktranslate %}
-            Du bist nun registriert und hast eine E-Mail mit einem Link zur Aktivierung deines Kontos bekommen. Solltest
-            du die E-Mail nicht erhalten haben oder andere Fragen haben, schreib uns an info@notfellchen.org
-        {% endblocktranslate %}
-    </p>
 {% endblock %}

src/tests/test_geo.py

@@ -1,4 +1,4 @@
-from fellchensammlung.tools.geo import calculate_distance_between_coordinates, LocationProxy
+from fellchensammlung.tools.geo import calculate_distance_between_coordinates
 from django.test import TestCase
 
 
@@ -22,22 +22,3 @@ class DistanceTest(TestCase):
             except AssertionError as e:
                 print(f"Distance calculation failed. Expected {distance}, got {result}")
                 raise e
-
-    def test_e2e_distance(self):
-        l_stuttgart = LocationProxy("Stuttgart")
-        l_tue = LocationProxy("Tübingen")
-        # Should be 30km
-        print(f"{l_stuttgart.position} -> {l_tue.position}")
-        distance_tue_stuttgart = calculate_distance_between_coordinates(l_stuttgart.position, l_tue.position)
-        self.assertLess(distance_tue_stuttgart, 50)
-        self.assertGreater(distance_tue_stuttgart, 20)
-
-
-        l_ueberlingen = LocationProxy("Überlingen")
-        l_pfullendorf = LocationProxy("Pfullendorf")
-        # Should be 18km
-        distance_ueberlingen_pfullendorf = calculate_distance_between_coordinates(l_ueberlingen.position, l_pfullendorf.position)
-        self.assertLess(distance_ueberlingen_pfullendorf, 21)
-        self.assertGreater(distance_ueberlingen_pfullendorf, 15)
-
-

src/tests/test_search.py

@@ -1,104 +0,0 @@
-from time import sleep
-
-from django.test import TestCase
-from django.urls import reverse
-from fellchensammlung.models import SearchSubscription, User, TrustLevel, AdoptionNotice, Location, SexChoicesWithAll, \
-    Animal, Species, AdoptionNoticeNotification, SexChoices
-from model_bakery import baker
-
-from fellchensammlung.tools.geo import LocationProxy
-from fellchensammlung.tools.search import Search, notify_search_subscribers
-
-
-class TestSearch(TestCase):
-    @classmethod
-    def setUpTestData(cls):
-        cls.test_user0 = User.objects.create_user(username='testuser0',
-                                                  first_name="Admin",
-                                                  last_name="BOFH",
-                                                  password='12345')
-
-        cls.test_user1 = User.objects.create_user(username='testuser1',
-                                                  first_name="Max",
-                                                  last_name="Müller",
-                                                  password='12345')
-        cls.test_user2 = User.objects.create_user(username='testuser2',
-                                                  first_name="Miriam",
-                                                  last_name="Müller",
-                                                  password='12345')
-        cls.test_user0.trust_level = TrustLevel.ADMIN
-        cls.test_user0.save()
-
-        rat = baker.make(Species, name="Farbratte")
-
-        cls.location_stuttgart = Location.get_location_from_string("Stuttgart")
-
-        cls.location_tue = Location.get_location_from_string("Kirchentellinsfurt")
-        cls.location_berlin = Location.get_location_from_string("Berlin")
-
-        cls.adoption1 = baker.make(AdoptionNotice, name="TestAdoption1", owner=cls.test_user0,
-                                   location=cls.location_stuttgart)
-        rat1 = baker.make(Animal,
-                          name="Rat1",
-                          adoption_notice=cls.adoption1,
-                          species=rat,
-                          sex=SexChoices.MALE,
-                          description="Eine unglaublich süße Ratte")
-        cls.adoption1.set_active()
-
-        cls.adoption2 = baker.make(AdoptionNotice, name="TestAdoption2", owner=cls.test_user0, location=cls.location_berlin)
-        rat2 = baker.make(Animal,
-                          name="Rat2",
-                          adoption_notice=cls.adoption2,
-                          species=rat,
-                          sex=SexChoices.FEMALE,
-                          description="Eine unglaublich süße Ratte")
-        cls.adoption2.set_active()
-
-        cls.subscription1 = SearchSubscription.objects.create(owner=cls.test_user1,
-                                                              location=cls.location_tue,
-                                                              max_distance=50,
-                                                              sex=SexChoicesWithAll.MALE)
-
-        cls.subscription2 = SearchSubscription.objects.create(owner=cls.test_user2,
-                                                              location=cls.location_berlin,
-                                                              max_distance=50,
-                                                              sex=SexChoicesWithAll.ALL)
-
-    def test_equals(self):
-        search_subscription1 = SearchSubscription.objects.create(owner=self.test_user0,
-                                                                 location=self.location_stuttgart,
-                                                                 sex=SexChoicesWithAll.ALL,
-                                                                 max_distance=100
-                                                                 )
-        search1 = Search()
-        search1.search_position = LocationProxy("Stuttgart").position
-        search1.max_distance = 100
-        search1.area_search = True
-        search1.sex = SexChoicesWithAll.ALL
-        search1.location_string = "Stuttgart"
-        search1._locate()
-
-        self.assertEqual(search_subscription1, search1)
-
-    def test_adoption_notice_fits_search(self):
-        search1 = Search(search_subscription=self.subscription1)
-        self.assertTrue(search1.adoption_notice_fits_search(self.adoption1))
-        self.assertFalse(search1.adoption_notice_fits_search(self.adoption2))
-
-        search2 = Search(search_subscription=self.subscription2)
-        self.assertFalse(search2.adoption_notice_fits_search(self.adoption1))
-        self.assertTrue(search2.adoption_notice_fits_search(self.adoption2))
-
-    def test_notification(self):
-        """
-        Simulates as if a new adoption notice is added and checks if Notifications are triggered.
-
-        Must simulate adding a new adoption notice, the actual celery task can not be tested as celery can not
-        be tested as it can't access the in-memory test database.
-        https://stackoverflow.com/questions/46530784/make-django-test-case-database-visible-to-celery/46564964#46564964
-        """
-        notify_search_subscribers(self.adoption1)
-
-        self.assertTrue(AdoptionNoticeNotification.objects.filter(user=self.test_user1, adoption_notice=self.adoption1).exists())
-        self.assertFalse(AdoptionNoticeNotification.objects.filter(user=self.test_user2).exists())

src/tests/test_views.py

@@ -23,7 +23,7 @@ class AnimalAndAdoptionTest(TestCase):
         test_user0.trust_level = TrustLevel.ADMIN
         test_user0.save()
 
-        adoption1 = baker.make(AdoptionNotice, name="TestAdoption1", owner=test_user0)
+        adoption1 = baker.make(AdoptionNotice, name="TestAdoption1")
         rat = baker.make(Species, name="Farbratte")
 
         rat1 = baker.make(Animal,
@@ -67,6 +67,7 @@ class AnimalAndAdoptionTest(TestCase):
                      "save-and-add-another-animal": "Speichern"}
 
         response = self.client.post(reverse('add-adoption'), data=form_data)
+        print(response.content)
 
         self.assertTrue(response.status_code < 400)
         self.assertTrue(AdoptionNotice.objects.get(name="TestAdoption4").is_active)
@@ -91,8 +92,7 @@ class SearchTest(TestCase):
         berlin = Location.get_location_from_string("Berlin")
         adoption1.location = berlin
         adoption1.save()
-
+        stuttgart = Location.get_location_from_string("Tübingen")
-        stuttgart = Location.get_location_from_string("Stuttgart")
         adoption3.location = stuttgart
         adoption3.save()
 
@@ -119,14 +119,11 @@ class SearchTest(TestCase):
         self.assertContains(response, "TestAdoption3")
         self.assertNotContains(response, "TestAdoption2")
 
-    def test_location_search(self):
+    def test_plz_search(self):
-        response = self.client.post(reverse('search'), {"max_distance": 50, "location_string": "Berlin", "sex": "A"})
+        response = self.client.post(reverse('search'), {"max_distance": 100, "location": "Berlin", "sex": "A"})
         self.assertEqual(response.status_code, 200)
-        # We can't use assertContains because TestAdoption3 will always be in response at is included in map
+        self.assertContains(response, "TestAdoption1")
-        # In order to test properly, we need to only care for the context that influences the list display
+        self.assertNotContains(response, "TestAdoption3")
-        an_names = [a.name for a in response.context["adoption_notices"]]
-        self.assertTrue("TestAdoption1" in an_names) # Adoption in Berlin
-        self.assertFalse("TestAdoption3" in an_names) # Adoption in Stuttgart
 
 
 class UpdateQueueTest(TestCase):
@@ -152,7 +149,7 @@ class UpdateQueueTest(TestCase):
     def test_login_required(self):
         response = self.client.get(reverse('updatequeue'))
         self.assertEqual(response.status_code, 302)
-        self.assertEqual(response.url, "/accounts/login/?next=/updatequeue/")
+        self.assertEquals(response.url, "/accounts/login/?next=/updatequeue/")
 
     def test_set_updated(self):
         self.client.login(username='testuser0', password='12345')