learning-credentials 0.4.1rc1__tar.gz → 0.4.1rc2__tar.gz

{learning_credentials-0.4.1rc1/learning_credentials.egg-info → learning_credentials-0.4.1rc2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: learning-credentials
-Version: 0.4.1rc1
+Version: 0.4.1rc2
 Summary: A pluggable service for preparing Open edX credentials.
 Author-email: OpenCraft <help@opencraft.com>
 License-Expression: AGPL-3.0-or-later

{learning_credentials-0.4.1rc1 → learning_credentials-0.4.1rc2}/learning_credentials/generators.py

@@ -375,11 +375,14 @@ def generate_pdf_credential(
     log.info("Starting credential generation for user %s", user.id)
 
     username = _get_user_name(user)
-    context_name = get_learning_context_name(learning_context_key)
-    template_path = options.get('template')
 
     # Handle multiline context name (semicolon separator for backward compatibility).
-    context_name = context_name.replace(';', '\n').replace(r'\n', '\n')
+    context_name = get_learning_context_name(learning_context_key).replace(';', '\n')
+    custom_context_text_element = options.get('text_elements', {}).get('context', {})
+    if isinstance(custom_context_text_element, dict):
+        context_name = custom_context_text_element.get('text', '').replace(';', '\n') or context_name
+
+    template_path = options.get('template')
     if '\n' in context_name:
         template_path = options.get('template_multiline', template_path)
 

{learning_credentials-0.4.1rc1 → learning_credentials-0.4.1rc2/learning_credentials.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: learning-credentials
-Version: 0.4.1rc1
+Version: 0.4.1rc2
 Summary: A pluggable service for preparing Open edX credentials.
 Author-email: OpenCraft <help@opencraft.com>
 License-Expression: AGPL-3.0-or-later

{learning_credentials-0.4.1rc1 → learning_credentials-0.4.1rc2}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "learning-credentials"
-version = "0.4.1rc1"
+version = "0.4.1rc2"
 description = "A pluggable service for preparing Open edX credentials."
 dynamic = ["readme"]
 requires-python = ">=3.11"

{learning_credentials-0.4.1rc1 → learning_credentials-0.4.1rc2}/tests/test_generators.py

@@ -474,16 +474,34 @@ def test_save_credential(mock_contentfile: Mock, mock_token_hex: Mock, storage:
     ("context_name", "options", "expected_template_slug", "expected_context_name"),
     [
         # Default.
-        ('Test Course', {'template': 'default'}, 'default', 'Test Course'),
+        ('Test Course', {'template': 'default', 'template_multiline': 'multiline'}, 'default', 'Test Course'),
         # Specify a different template for multiline course names and replace \n with newline.
         ('Test\nCourse', {'template': 'default', 'template_multiline': 'multiline'}, 'multiline', 'Test\nCourse'),
-        ('Test\\nCourse', {'template_multiline': 'multiline'}, 'multiline', 'Test\nCourse'),
         # Backward compatibility with semicolon separator.
-        ('Test;Course', {'template_multiline': 'multiline'}, 'multiline', 'Test\nCourse'),
-        # Mixed semicolon and newline separators.
-        ('Te\nst\\nCourse;', {'template_multiline': 'multiline'}, 'multiline', 'Te\nst\nCourse\n'),
+        ('Test\nCourse;', {'template_multiline': 'multiline'}, 'multiline', 'Test\nCourse\n'),
         # Ensure that the default template is used when no multiline template is specified.
-        ('Test\\nCourse', {'template': 'default'}, 'default', 'Test\nCourse'),
+        ('Test\nCourse', {'template': 'default'}, 'default', 'Test\nCourse'),
+        # Custom context text with newlines uses multiline template.
+        (
+            'Single Line',
+            {'template_multiline': 'multiline', 'text_elements': {'context': {'text': 'Line 1\nLine 2'}}},
+            'multiline',
+            'Line 1\nLine 2',
+        ),
+        # Custom context text with semicolon uses multiline template.
+        (
+            'Single Line',
+            {'template_multiline': 'multiline', 'text_elements': {'context': {'text': 'Line 1;Line 2'}}},
+            'multiline',
+            'Line 1\nLine 2',
+        ),
+        # Disabled context element falls back to learning context name for template selection.
+        (
+            'Multi\nLine',
+            {'template_multiline': 'multiline', 'text_elements': {'context': False}},
+            'multiline',
+            'Multi\nLine',
+        ),
     ],
 )
 @patch(