Updated drupal core.

dmundra · Nov 19, 2024 · e79de22 · e79de22
1 parent af52a41
commit e79de22
Show file tree

Hide file tree

Showing 7 changed files with 116 additions and 112 deletions.
diff --git a/composer.lock b/composer.lock
diff --git a/core/composer.json b/core/composer.json
@@ -34,7 +34,7 @@
         "symfony/process": "^7.1",
         "symfony/polyfill-iconv": "^1.26",
         "symfony/yaml": "^7.1",
-        "twig/twig": "^3.14.0",
+        "twig/twig": "^3.14.2",
         "doctrine/annotations": "^2.0",
         "doctrine/lexer": "^2.0",
         "guzzlehttp/guzzle": "^7.5",

diff --git a/core/lib/Drupal.php b/core/lib/Drupal.php
@@ -75,7 +75,7 @@ class Drupal {
   /**
    * The current system version.
    */
-  const VERSION = '11.0.6';
+  const VERSION = '11.0.7';
 
   /**
    * Core API compatibility.

diff --git a/core/modules/views_ui/src/ViewEditForm.php b/core/modules/views_ui/src/ViewEditForm.php
@@ -16,6 +16,7 @@
 use Drupal\Core\Url;
 use Drupal\views\Views;
 use Symfony\Component\DependencyInjection\ContainerInterface;
+use Symfony\Component\HttpFoundation\Exception\BadRequestException;
 use Symfony\Component\HttpFoundation\RequestStack;
 use Symfony\Component\HttpKernel\Exception\NotAcceptableHttpException;
 use Drupal\Core\Extension\ModuleHandlerInterface;
@@ -464,7 +465,7 @@ public function getDisplayDetails($view, $display) {
 
           if ($path && (!str_contains($path, '%'))) {
             // Wrap this in a try/catch as trying to generate links to some
-            // routes may throw a NotAcceptableHttpException if they do not
+            // routes may throw an exception, for example if they do not
             // respond to HTML, such as RESTExports.
             try {
               if (!parse_url($path, PHP_URL_SCHEME)) {
@@ -476,7 +477,7 @@ public function getDisplayDetails($view, $display) {
                 $url = Url::fromUri("base:$path");
               }
             }
-            catch (NotAcceptableHttpException $e) {
+            catch (BadRequestException | NotAcceptableHttpException $e) {
               $url = '/' . $path;
             }
 

diff --git a/core/modules/views_ui/src/ViewListBuilder.php b/core/modules/views_ui/src/ViewListBuilder.php
@@ -10,6 +10,7 @@
 use Drupal\Core\Link;
 use Drupal\Core\Url;
 use Symfony\Component\DependencyInjection\ContainerInterface;
+use Symfony\Component\HttpFoundation\Exception\BadRequestException;
 use Symfony\Component\HttpKernel\Exception\NotAcceptableHttpException;
 
 /**
@@ -261,14 +262,14 @@ protected function getDisplaysList(EntityInterface $view) {
           $path = $display->getPath();
           if ($view->status() && !str_contains($path, '%')) {
             // Wrap this in a try/catch as trying to generate links to some
-            // routes may throw a NotAcceptableHttpException if they do not
+            // routes may throw an exception, for example if they do not
             // respond to HTML, such as RESTExports.
             try {
               // @todo Views should expect and store a leading /. See:
               //   https://www.drupal.org/node/2423913
               $rendered_path = Link::fromTextAndUrl('/' . $path, Url::fromUserInput('/' . $path))->toString();
             }
-            catch (NotAcceptableHttpException $e) {
+            catch (BadRequestException | NotAcceptableHttpException $e) {
               $rendered_path = '/' . $path;
             }
           }

diff --git a/...es/views_ui/tests/modules/views_ui_test/config/install/views.view.sa_contrib_2013_035.yml b/...es/views_ui/tests/modules/views_ui_test/config/install/views.view.sa_contrib_2013_035.yml
@@ -183,4 +183,4 @@ display:
     display_plugin: page
     position: 2
     display_options:
-      path: foobar
+      path: 'foobar '
diff --git a/core/modules/views_ui/tests/src/Functional/XssTest.php b/core/modules/views_ui/tests/src/Functional/XssTest.php
@@ -37,6 +37,8 @@ public function testViewsUi(): void {
    */
   public function testNoDoubleEscaping(): void {
     $this->drupalGet('admin/structure/views');
+    $this->assertSession()->pageTextContains('sa_contrib_2013_035');
+    $this->assertSession()->pageTextContains('Page (/foobar )');
     $this->assertSession()->assertNoEscaped('&lt;');
 
     $this->drupalGet('admin/structure/views/view/sa_contrib_2013_035');