Skip remaining arguments if an empty exception found

The last submitted patch, 3: 3388941-skip-arguments.patch, failed testing. View results →

fix unit test

Found a problem: https://www.drupal.org/node/2452659 →
The real problem is try to render taxonomy/term/%/feed link when argument is not present. The argument is necessary because % in the base url. The real solution will be that deny preview in views_ui when (one of ) necessary arguments are missing.

This is a working but not perfect solution.

This patch deny preview if necessary arguments are missing.

@zeeshan_khan I m confused: your patch seem not to add/correct relevant parts to the issue and adds grammar and Uppercase and sentence period comment corrections on other spots. It was surely by accident? So let me remove/hide it from the issue summary. Feel free to re-add it if I am mistaken.

Not sure this can just be removed. Can see from earlier patches it failed some tests, so shows it was setup by design. Think it will need a way to maintain existing functionality + add the new.

I did not enough time for create environment, where I can run tests. The first patch is for run all test only.

Can this be turned to an MR please.

I have tested this patch and its working.
when i only pass the tid then the URL is skip the empty argument . so i have added a MR for this.
before patch url : /test/17/%2A/%2A
Afterpatch url: /test/17
Output is same for both.

Used git blame all the way back to contrib day https://git.drupalcode.org/project/drupal/-/blob/5259e3b9eddb50f5b0cb604...

Original code seemed to be a placeholder

$pieces[] = '*'; // gotta put something if there just isn't one.

So think updating makes sense

This seems like it could add test coverage for the use-case in the issue summary.