Limit snippet characters

app/blog/migrations/0042_alter_post_snippet.py

@@ -0,0 +1,21 @@
+# Generated by Django 4.2.5 on 2023-11-08 16:00
+
+import blog.utils
+from django.db import migrations
+import django_ckeditor_5.fields
+
+
+class Migration(migrations.Migration):
+    dependencies = [
+        ("blog", "0041_disallow_null_values_date_updated"),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name="post",
+            name="snippet",
+            field=django_ckeditor_5.fields.CKEditor5Field(
+                blank=True, null=True, validators=[blog.utils.snippet_validator]
+            ),
+        ),
+    ]

app/blog/models.py

@@ -4,6 +4,7 @@
 from django.utils.text import slugify
 from django.contrib.auth.models import User
 from django_ckeditor_5.fields import CKEditor5Field
+from .utils import snippet_validator
 
 # import logging
 from django_resized import ResizedImageField
@@ -70,7 +71,9 @@ class Post(models.Model):
     metaimg_alt_txt = models.CharField(max_length=500, default="John Solly Headshot")
     metaimg_attribution = models.CharField(max_length=500, blank=True, null=True)
     content = CKEditor5Field(blank=True, null=True, config_name="extends")
-    snippet = CKEditor5Field(blank=True, null=True, config_name="extends")
+    snippet = CKEditor5Field(
+        blank=True, null=True, config_name="extends", validators=[snippet_validator]
+    )
     date_posted = models.DateTimeField(default=timezone.now)
     date_updated = models.DateTimeField(auto_now=True)
     author = models.ForeignKey(User, on_delete=models.CASCADE)

app/blog/utils.py

@@ -7,6 +7,8 @@
 import numpy as np
 from .models import Post, Similarity
 from django.db import models
+from django.core.exceptions import ValidationError
+import re
 
 
 def load_pickle_file():
@@ -138,3 +140,17 @@ def compute_similarity(post_id: int) -> None:
         )
 
     cleanup_similarities(post)
+
+
+link_media_pattern = re.compile(
+    r"<a.*?/a>|<img.*?/img>|<video.*?/video>|<audio.*?/audio>", flags=re.IGNORECASE
+)
+max_length = 400
+
+
+def snippet_validator(value):
+    value_without_links_media = link_media_pattern.sub("", value)
+    if len(value_without_links_media) > max_length:
+        raise ValidationError(
+            f"The snippet cannot have more than {max_length} characters (excluding links and media)."
+        )

app/staticfiles/css/main.css

@@ -755,8 +755,9 @@ a[href^="http"]::after {
   margin: auto;
 }
 
-.errorlist {
-  color: var(--a11-red);
+.errorlist li {
+  color: red;
+  list-style-type: none;
 }
 
 a.hidden-link {
@@ -779,7 +780,6 @@ a.hidden-link {
     svg,
     table,
     p,
-    input,
     blockquote,
     li,
     legend,

app/tests/test_utils.py

@@ -1,9 +1,16 @@
 from django.utils.deprecation import MiddlewareMixin
-from blog.utils import create_context, answer_question, load_pickle_file
+from blog.utils import (
+    create_context,
+    answer_question,
+    load_pickle_file,
+    snippet_validator,
+    max_length,
+)
 import pandas as pd
 from unittest.mock import patch
 import numpy as np
 from unittest import TestCase
+from django.core.exceptions import ValidationError
 
 
 class TestUtils(TestCase, MiddlewareMixin):
@@ -76,3 +83,15 @@ def test_answer_question(
             question="What is the capital of the United States of America?", df=df
         )
         self.assertEqual(answer, "The mocked answer")
+
+    def test_snippet_validation(self):
+        valid_value_with_links = " ".join(
+            [f'<a href="http://example{i}.com">Link{i}</a>' for i in range(100)]
+        )
+        invalid_value = f"""A {'<a href="http://example.com">Link</a> ' * 10}{'B' * (max_length - 10)}"""
+
+        with self.assertRaises(ValidationError):
+            snippet_validator(invalid_value)
+
+        snippet_validator(valid_value_with_links)
+        self.assertTrue(True)