Error when added to 404 pages

Hi @adrianopulz,
I have fixed this issue and also created MR for this issue . please replace The 'internal:/' URL is a reference to the internal path '/', which typically represents the homepage of your Drupal site. In other words, it's setting the fallback URL to the site's homepage.

Thank you!

Thanks for your fork @shreya_th!

The problem is for URLs created from the Request, URLs from the route will not be a problem. You added try/catch to the URL created from the route when the display['display_options']['path'] is set. We need the fallback only in the createFromRequest() call.

I've added a patch with the change example. I don't think we need to change the block of code using the UrlFromRoute.

What about other system paths?

That snippet of code has been removed in another fix. Can you confirm this is still an issue.

Hate to be reopening this issue, but the issue is still there using the latest 7.x release.

We worked around the issue for now by modifying the code to be as follows:

use Symfony\Component\Routing\Exception\ResourceNotFoundException;

protected function getExposedFormActionUrl(FormStateInterface $form_state): Url {
    $request = \Drupal::request();
    try {
      $url = Url::createFromRequest(clone $request);
      $url->setAbsolute();
      return $url;
    }
    catch (ResourceNotFoundException $e) {
      return Url::fromRoute('<front>')->setAbsolute();
    }
  }

A patch will follow, I expect this code needs some better attention.

Alex Bukach → made their first commit to this issue’s fork.

Created MR!86, here's the respective patch.

MR should be against 7.0.x but let’s add test coverage also

Could the steps be flushed out.

I added a view block that's used better exposed filters to 404 page and not getting any error.

On 10.3

I've got the same issue. I've defined a global "search" exposed form in block, which is added to the header of the website. Then on all 404 pages it shows the same error.

I've tried the patch in #12 and it went nicely, even if I'm still using the 6.x version. The patch fixed the issue, at least there is now no more errors.

smustgrave → closed merge request !49