diff --git a/core/modules/views_ui/src/ViewEditForm.php b/core/modules/views_ui/src/ViewEditForm.php index 131a224ce77..bc577283047 100644 --- a/core/modules/views_ui/src/ViewEditForm.php +++ b/core/modules/views_ui/src/ViewEditForm.php @@ -16,6 +16,7 @@ use Drupal\Core\Theme\ThemeManagerInterface; 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 @@ class ViewEditForm extends ViewFormBase { 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 @@ class ViewEditForm extends ViewFormBase { $url = Url::fromUri("base:$path"); } } - catch (NotAcceptableHttpException) { + catch (BadRequestException | NotAcceptableHttpException) { $url = '/' . $path; } diff --git a/core/modules/views_ui/src/ViewListBuilder.php b/core/modules/views_ui/src/ViewListBuilder.php index 06c422e2a18..3ad105a8ce5 100644 --- a/core/modules/views_ui/src/ViewListBuilder.php +++ b/core/modules/views_ui/src/ViewListBuilder.php @@ -10,6 +10,7 @@ use Drupal\Core\Entity\EntityTypeInterface; 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 @@ class ViewListBuilder extends ConfigEntityListBuilder { $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) { + catch (BadRequestException | NotAcceptableHttpException) { $rendered_path = '/' . $path; } } diff --git a/core/modules/views_ui/tests/modules/views_ui_test/config/install/views.view.sa_contrib_2013_035.yml b/core/modules/views_ui/tests/modules/views_ui_test/config/install/views.view.sa_contrib_2013_035.yml index 80029fb944a..d6280799790 100644 --- a/core/modules/views_ui/tests/modules/views_ui_test/config/install/views.view.sa_contrib_2013_035.yml +++ b/core/modules/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 index 2c3ca64b5f7..887bdfae72c 100644 --- a/core/modules/views_ui/tests/src/Functional/XssTest.php +++ b/core/modules/views_ui/tests/src/Functional/XssTest.php @@ -37,6 +37,8 @@ class XssTest extends UITestBase { */ public function testNoDoubleEscaping(): void { $this->drupalGet('admin/structure/views'); + $this->assertSession()->pageTextContains('sa_contrib_2013_035'); + $this->assertSession()->pageTextContains('Page (/foobar )'); $this->assertSession()->assertNoEscaped('<'); $this->drupalGet('admin/structure/views/view/sa_contrib_2013_035');