Fix JS error in Drupal.behaviors.activeLinks (escape queryString in selector).
2 open threads
Closes #3464340
Merge request reports
Activity
added 1 commit
- 7d31b84f - Add test (reveal JS error from Drupal.behaviors.activeLinks).
added 1 commit
- f69796cc - Fix JS error in Drupal.behaviors.activeLinks (escape queryString in selector).
added 1 commit
- a085cfa3 - Add test (reveal JS error from Drupal.behaviors.activeLinks).
added 1 commit
- b83a5673 - Fix JS error in Drupal.behaviors.activeLinks (escape queryString in selector).
added 25 commits
-
b83a5673...6b50d024 - 23 commits from branch
project:11.x
- 5ce43357 - Add test (reveal JS error from Drupal.behaviors.activeLinks).
- 412be1fa - Fix JS error in Drupal.behaviors.activeLinks (escape queryString in selector).
-
b83a5673...6b50d024 - 23 commits from branch
22 22 attach(context) { 23 23 // Start by finding all potentially active links. 24 24 const path = drupalSettings.path; 25 const queryString = JSON.stringify(path.currentQuery); 26 const querySelector = path.currentQuery 27 ? `[data-drupal-link-query='${queryString}']` 25 const queryString = path.currentQuery 26 ? JSON.stringify(path.currentQuery) 27 : null; changed this line in version 9 of the diff
added 88 commits
-
99507f73...07abb808 - 84 commits from branch
project:11.x
- 76890b83 - Add test (reveal JS error from Drupal.behaviors.activeLinks).
- 47fcc5c5 - Fix JS error in Drupal.behaviors.activeLinks (escape queryString in selector).
- fe3a8112 - Use CSS.escape.
- 58c169f8 - Combine statements.
Toggle commit list-
99507f73...07abb808 - 84 commits from branch
22 22 attach(context) { 23 23 // Start by finding all potentially active links. 24 24 const path = drupalSettings.path; 25 const queryString = JSON.stringify(path.currentQuery); 26 25 const querySelector = path.currentQuery 27 ? `[data-drupal-link-query='${queryString}']` 26 ? `[data-drupal-link-query="${CSS.escape(JSON.stringify(path.currentQuery))}"]` changed this line in version 10 of the diff
Please register or sign in to reply