- ๐ณ๐ฌNigeria chike Nigeria
I can confirm #8 as I have two views block displays, one block added on the page and the other used for sorting. Patch #5 didn't work while patch #9 worked.
- ๐บ๐ธUnited States z3cka
Patch from #9 works like a charm. Thanks for the patch!
- Status changed to RTBC
over 1 year ago 9:28pm 14 March 2023 - Status changed to Needs work
over 1 year ago 11:12pm 16 March 2023 - ๐บ๐ธUnited States loze Los Angeles
After further testing, i've realized this is not working correctly.
The route parameters are not passed along and I am getting this error.Symfony\Component\Routing\Exception\MissingMandatoryParametersException: Some mandatory parameters are missing ("node") to generate a URL for route "view.gallery_groups.page_1". in Drupal\Core\Routing\UrlGenerator->doGenerate() (line 181 of ...../core/lib/Drupal/Core/Routing/UrlGenerator.php).
- Status changed to Needs review
over 1 year ago 12:54am 17 March 2023 - ๐บ๐ธUnited States loze Los Angeles
I think this got it working.
I added a hook_views_pre_render() implementation where Im adding the params of the current route to the draggableviews contextual link render array.
This allows it to work when your draggableview requires a contextual filter.
- ๐จ๐ฆCanada lily.yan
I can confirm that applying patch https://www.drupal.org/files/issues/2023-03-16/draggableviews-route-erro... โ to draggableviews 2.1.2 can fix the issue.
- ๐บ๐ธUnited States adrianm6254
I applied patch #15 and it has cleared up all the RouteNotFoundExceptions I was getting.
- last update
about 1 year ago 1 pass - last update
about 1 year ago 1 pass - ๐บ๐ธUnited States joshuasosa
Patch #15 works for me. No more errors about this issue in log. Thanks!
- Status changed to RTBC
11 months ago 7:48pm 14 December 2023 - ๐บ๐ธUnited States rclemings
Another vote for #15. That makes seven so I'll set it to RTBC.
- ๐บ๐ธUnited States loze Los Angeles
While this does work, the "order view" contextual link is always shown regardless of the users permission to access the sort page.
- Status changed to Needs work
10 months ago 9:14pm 21 January 2024 - Status changed to Needs review
10 months ago 6:49am 22 January 2024 - ๐บ๐ธUnited States loze Los Angeles
This one checks the route access and unsets the contextual link if false.
- last update
10 months ago 1 pass - last update
10 months ago 1 pass - Merge request !40Issue #3316975 - Contextual Link throws RouteNotFoundException: Route "view.this" does not exist โ (Open) created by loze
- ๐บ๐ธUnited States loze Los Angeles
#25 was no longer applying with the latest 2.1.x updates. This MR40 is the patch from #25 with the merge errors resolved.
david.muffley โ made their first commit to this issueโs fork.
- Status changed to RTBC
4 months ago 12:58am 20 July 2024 - ๐ญ๐ทCroatia devad
Patch #32 fixed the issue for me.
DraggableViews 2.1.4
D10.3.1
PHP 8.1 Patch #32 fixes the immediate problem but throws a sitewide error for me:
[Violation] Added non-passive event listener to a scroll-blocking event. Consider marking event handler as 'passive' to make the page more responsive. See
[Violation] Added non-passive event listener to a scroll-blocking 'touchstart' event. Consider marking event handler as 'passive' to make the page more responsive. See https://www.chromestatus.com/feature/5745543795965952DraggableViews 2.1.4
D 10.3.2
PHP 8.2.21- Status changed to Needs work
2 months ago 3:13pm 10 September 2024 - ๐จ๐ฆCanada iStryker
I can confirm this removes the error. This fix removes the "Order View" contextual link.
How to reproduce the removal
- Install 2.1.x draggableviews
- Install the draggableviews_demo module (with it having the default Draggable Views Data of "Order Page")
- GoTo the draggableviews display page (/draggableviews-demo)
- Check the contextual links....you will see Order ViewNow
- Git Checkout 3316975-contextual-link-error
- Clear cache
- GoTo the draggableviews display page (/draggableviews-demo)
- Check the contextual links....you will NOT see Order View - ๐ญ๐บHungary fox mulder
patch added in #35 causes error:
Uncaught PHP Exception TypeError: "draggableviews_views_pre_render(): Argument #1 ($view) must be of type ViewExecutable, Drupal\\views\\ViewExecutable given" at .../web/modules/contrib/draggableviews/draggableviews.module line 276 - ๐จ๐ฟCzech Republic Bohus Ulrych Pilsen (Czechia)
Yes, patch #35 breaks e.g. /admin/content/files View. Reason is missing
use Drupal\views\ViewExecutable;
because is used
function draggableviews_views_pre_render(ViewExecutable $view) - Status changed to Needs review
2 months ago 12:58am 17 September 2024 - ๐บ๐ธUnited States loze Los Angeles
Patch #35 does not work as pointed out in #38
Patch #32 does work which is a diff for MR40. Please test against those.
- ๐จ๐ฟCzech Republic Bohus Ulrych Pilsen (Czechia)
Patch #32 seems to be working without any problems or errors.
- ๐ง๐ชBelgium stijnd Belgium
Patch #32 works perfect, using 2.1.4 with Drupal 10.3.5
-
istryker โ
committed d1a4a588 on 2.1.x
Issue #3316975 by loze, david.muffley, bohus ulrych: Contextual Link...
-
istryker โ
committed d1a4a588 on 2.1.x
Automatically closed - issue fixed for 2 weeks with no activity.