Views incorrectly adding "all" parameter to contextual filter summary links

Also having similar bad URLs on summaries: if the summaries are on a block, then the URLs are to the main page display on the view.

When there was a feed, the summary was grabbing the feed URL. when i removed the feed, it went to the first page url. going to make my summary blocks in a separate view to see if that works better.

Problem may be around something like: display.block_1.display_options.arguments.created_year.summary_options.base_path (which is empty in my problematic view)

Sounds related to #2853661: Views contextual + global null + exposed filter introduces extra url param

@Daniel Schaefer sure I read it, just have nothing to add right now

If we can determine which classes or plugins are having the bad information here, that would be a step forward. Sometimes these things are hard as heck to dig out. Once we know that, we can dump the vars out. I don't know the innards of d8 views that well, tho.

views/src/Plugin/views/style/DefaultSummary.php is the fallback for summaries.

views/src/Plugin/views/style/UnformattedSummary.php is the another formatter.

views/src/Plugin/views/argument/ArgumentPluginBase.php has the default form for summaries. A lot of stuff in here.

views/src/Plugin/views/argument/StringArgument.php has summaryQuery() builds the summary query based on a string.

Also several yml files have settings for summaries in the tests. Speculating here I would bet the bug lies somewhere in one of these. If you put dump statements all over the functions in here the bug might turn up as a bad handling of an array or a default option somewhere.

Closed #2853661: Views contextual + global null + exposed filter introduces extra url param as a duplicate of this.

Here is a patch that works locally with the View provided in #2853661: Views contextual + global null + exposed filter introduces extra url param. You would have to remove the Null argument and re-add it to make sure it has the updated options.

I'm sure this will (should) fail other scenarios, since the 'fix' in \Drupal\views\ViewExecutable is hardly well thought out. But it should provide an indication where we need to look for a fix.

This also happens with display links "Global: Link to display", which is used to create the Grid and Table links used by the Media Library widget now in core, via the Media Library view.

Those Grid and Table links will ignore contextual filters, adding the "all" whether it's wanted or not. Actually, it adds whatever has been given in the contextual filter's "Exception Value" field. If you blank that field, the link is rendered with an asterisk, which also breaks the contextual filter.

Thus, anyone wishing to add contextual filters to the Media Library view will see their filters ignored when the user clicks the Grid or Table links to change the display.

Issue 2853661 is closed as dupe but has, IMO, a better description of the problem.

I've noticed that default_argument_skip_url is not being respected.

I came to this issue via Bugsmash daily traige.

I think we need to do some work on the solution.
The option that is used in the patch #23 is removed now.
Issue that removed the option: https://www.drupal.org/node/981870

I am able to reproduce the issue. However slightly confused whether I took right steps to reproduce the issue.
I'm going to upload the issue along with the test-only patch and a fix.
Here https://pastebin.com/cJweDtHX is the view that I used for testing purposes.

Further investigation:
IMO, `all` option is appearing because one might have kept exception value as `all`
So even if we remove that value default value will still be * i.e. %2A in the URL
Reference https://git.drupalcode.org/project/drupal/-/blob/11.x/core/modules/views...

So one way to fix this could be something like empty fallback value instead of "*"

          // Provide some fallback in case no exception value could be found.
          $parameters[$variable_name] = '';

Works fine without the exposed filter, only breaks for me using the exposed filter first.

Uh use facets module? ;)