Add migration

fsbraun · fsbraun · commit 7e64671153db · 2024-07-02T16:32:03.000+02:00
diff --git a/djangocms_text/cms_plugins.py b/djangocms_text/cms_plugins.py
@@ -14,7 +14,7 @@
     HttpResponse,
     HttpResponseBadRequest,
     HttpResponseForbidden,
-    HttpResponseRedirect,
+    HttpResponseRedirect, JsonResponse,
 )
 from django.shortcuts import get_object_or_404
 from django.template import RequestContext
@@ -545,14 +545,14 @@ def get_available_urls(self, request):
                             "text": " " * (0 if search else len(page_content.page.node.path) // 4 - 1)
                                     + page_content.title,
                             "url": page_content.get_absolute_url(),
-                            "value": f"cms.page:{page_content.page.pk}",
+                            "id": f"cms.page:{page_content.page.pk}",
                             "verbose": page_content.title,
                         } for page_content in qs
                     ]
                 }
             ]
         }
-        return HttpResponse(json.dumps(urls))
+        return JsonResponse(urls)
 
     @classmethod
     def get_child_plugin_candidates(cls, slot, page):
diff --git a/djangocms_text/html.py b/djangocms_text/html.py
@@ -90,7 +90,7 @@ def __call__(self) -> dict[str, Union[dict[str, set[str]], set[str], None]]:
 
 def clean_html(data: str, full: bool = False, cleaner: NH3Parser = cms_parser) -> str:
     """
-    Cleans HTML from XSS vulnerabilities using lxml
+    Cleans HTML from XSS vulnerabilities using nh3
     If full is False, only the contents inside <body> will be returned (without
     the <body> tags).
     """
@@ -185,9 +185,10 @@ def dynamic_href(elem: Element, obj: models.Model, attr: str) -> None:
 
 def dynamic_src(elem: Element, obj: models.Model, attr: str) -> None:
     """
-    This method modifies the provided element by setting the value of the specified attribute based on the provided object.
-    If the object has a "get_absolute_url" method and it returns a non-empty value, the attribute of the element will be set to the URL returned by the method.
-    Otherwise, the "data-cms-error" attribute of the XML element will be set to "ref-not-found".
+    This method modifies the provided element by setting the value of the specified attribute based on the provided
+    object. If the object has a "get_absolute_url" method, and it returns a non-empty value, the attribute of the
+    element will be set to the URL returned by the method. Otherwise, the "data-cms-error" attribute of the XML
+    element will be set to "ref-not-found".
 
     :param elem: The XML element to modify.
     :type elem: Element
@@ -217,7 +218,8 @@ def render_dynamic_attributes(
     Parameters:
     - dyn_html (str): The HTML content with dynamic attributes
     - admin_objects (bool) (optional): Flag to indicate whether to fetch data from admin objects (default: False)
-    - remove_attr (bool) (optional): Flag to indicate whether to remove dynamic attributes from the final HTML (default: True)
+    - remove_attr (bool) (optional): Flag to indicate whether to remove dynamic attributes from the final HTML
+      (default: True)
 
     Returns:
     - str: The updated HTML content with dynamic attributes
diff --git a/djangocms_text/migrations/0003_auto_20240702_1409.py b/djangocms_text/migrations/0003_auto_20240702_1409.py
@@ -0,0 +1,44 @@
+# Generated by Django 3.2.25 on 2024-07-02 14:09
+
+from django.db import migrations, models, ProgrammingError
+
+
+def migrate_text_ckeditor_fields(apps, schema_editor):
+    class CKEditorText(models.Model):
+        class Meta:
+            managed = False
+            db_table = "djangocms_text_ckeditor_text"
+        cmsplugin_ptr_id = models.PositiveIntegerField(primary_key=True)
+        body = models.TextField()
+
+    class Text_Text(models.Model):  # Name must not be used elsewhere as model
+        class Meta:
+            managed = False
+            db_table = "djangocms_text_text"
+        cmsplugin_ptr_id = models.PositiveIntegerField(primary_key=True)
+        body = models.TextField()
+        json = models.JSONField(blank=True, null=True)
+        rte = models.CharField(max_length=16, blank=True)
+
+    try:
+        existing_texts = Text_Text.objects.all().values_list("cmsplugin_ptr_id", flat=True)
+        qs = CKEditorText.objects.using(schema_editor.connection.alias).exclude(cmsplugin_ptr_id__in=existing_texts)
+    except ProgrammingError:
+        return
+    Text_Text.objects.using(schema_editor.connection.alias).bulk_create(
+        Text_Text(body=ckeditor_text.body, rte="text_ckeditor4", cmsplugin_ptr_id=ckeditor_text.cmsplugin_ptr_id)
+        for ckeditor_text in qs
+    )
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('djangocms_text', '0002_text_json_text_rte'),
+    ]
+
+    operations = [
+        migrations.RunPython(
+            code=migrate_text_ckeditor_fields,
+        )
+    ]
diff --git a/djangocms_text/widgets.py b/djangocms_text/widgets.py
@@ -72,7 +72,7 @@ def __init__(
 
         super().__init__(attrs)
 
-        self.installed_plugins = installed_plugins  # general
+        self.installed_plugins = installed_plugins or [] # general
         self.pk = pk  # specific
         self.placeholder = (
             placeholder.pk if isinstance(placeholder, models.Model) else placeholder
diff --git a/private/js/cms.linkfield.js b/private/js/cms.linkfield.js
@@ -86,9 +86,9 @@ class LinkField {
     _addResult(result) {
         const item = document.createElement('div');
         item.textContent = result.text;
-        if (result.value) {
+        if (result.id) {
             item.classList.add('cms-linkfield-option');
-            item.setAttribute('data-value', result.value);
+            item.setAttribute('data-value', result.id);
             item.setAttribute('data-href', result.url);
             item.setAttribute('data-text', result.verbose);
             item.addEventListener('click', this.handleSelection.bind(this));

Original file line number	Diff line number	Diff line change
`@@ -14,7 +14,7 @@`
`14`	`14`	`HttpResponse,`
`15`	`15`	`HttpResponseBadRequest,`
`16`	`16`	`HttpResponseForbidden,`
`17`		`- HttpResponseRedirect,`
	`17`	`+ HttpResponseRedirect, JsonResponse,`
`18`	`18`	`)`
`19`	`19`	`from django.shortcuts import get_object_or_404`
`20`	`20`	`from django.template import RequestContext`
`@@ -545,14 +545,14 @@ def get_available_urls(self, request):`
`545`	`545`	`"text": " " * (0 if search else len(page_content.page.node.path) // 4 - 1)`
`546`	`546`	`+ page_content.title,`
`547`	`547`	`"url": page_content.get_absolute_url(),`
`548`		`- "value": f"cms.page:{page_content.page.pk}",`
	`548`	`+ "id": f"cms.page:{page_content.page.pk}",`
`549`	`549`	`"verbose": page_content.title,`
`550`	`550`	`} for page_content in qs`
`551`	`551`	`]`
`552`	`552`	`}`
`553`	`553`	`]`
`554`	`554`	`}`
`555`		`- return HttpResponse(json.dumps(urls))`
	`555`	`+ return JsonResponse(urls)`
`556`	`556`
`557`	`557`	`@classmethod`
`558`	`558`	`def get_child_plugin_candidates(cls, slot, page):`