Fix/additive custom claims arrays (#170)

* feat: add preset claims while keeping existing claim contents

* feat: add more custom claims

* chore: update CHANGELOG

* Update CHANGELOG.md

Co-authored-by: Sergei Mikhailov <[email protected]>
Signed-off-by: Martin Nicholson <[email protected]>

* chore: update application_version

* fix: set claim JSON correctly for non-objects

* chore: version bump and CHANGELOG update

---------

Signed-off-by: Martin Nicholson <[email protected]>
Co-authored-by: Sergei Mikhailov <[email protected]>

Author: oatymart

CHANGELOG.md

@@ -1,6 +1,11 @@
 CHANGELOG
 =========
 
+2.11.1
+------
+
+* Fixed claim selection for simple arrays
+
 2.11.0
 ------
 
@@ -118,7 +123,7 @@ Fixed handling of concurrent AGS requests
 -----
 
 * Updated LTI 1.3 bundle and libraries dependencies
-* Updated Symfony packages to version 5.3.x  
+* Updated Symfony packages to version 5.3.x
 * Updated devkit registration to be the default one
 
 2.0.1

config/services.yaml

@@ -8,7 +8,7 @@ parameters:
     application_env: '%env(resolve:APP_ENV)%'
     application_api_key: '%env(resolve:APP_API_KEY)%'
     application_vendors: '%kernel.project_dir%/vendor/composer/installed.php'
-    application_version: '2.11.0'
+    application_version: '2.11.1'
     container.dumper.inline_factories: true
     cache.redis.namespace: '%env(default:cache.redis.namespace.default:REDIS_CACHE_NAMESPACE)%'
     cache.redis.namespace.default: 'devkit'

templates/platform/message/ltiResourceLinkLaunch.html.twig

@@ -225,10 +225,17 @@
         $(".editorClaim").click(function(event){
             try {
                 var claims = JSON.parse($("#lti_resource_link_launch_claims").val() || "{}");
-                claims[$(this).data('name')] = {
-                    ...$(this).data('value'),
-                    ...claims[$(this).data('name')]
-                };
+                var name = $(this).data('name');
+                var value = $(this).data('value');
+
+                if (typeof value === 'object' && !Array.isArray(value)) {
+                    claims[name] = {
+                        ...value,
+                        ...claims[name]
+                    };
+                } else {
+                    claims[name] = value;
+                }
                 $("#lti_resource_link_launch_claims").val(JSON.stringify(claims, undefined, 4));
                 $("#lti_resource_link_launch_claims").removeClass('is-invalid');
                 $("#lti_resource_link_launch_claims").addClass('is-valid');