Needs work
Project:
Drupal core
Version:
main
Component:
views.module
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
15 Jan 2015 at 07:44 UTC
Updated:
2 May 2023 at 07:30 UTC
Jump to comment: Most recent, Most recent file

Comments
Comment #1
lendudeNice.
Moving to right queue.
Comment #2
dawehnerSo yeah we could patch
\Drupal\views\Plugin\views\area\Viewto do so. I think we could totally do that.Comment #3
lmeurs commentedI attached patches for D8 and D7.
Comment #4
lmeurs commentedComment #6
lmeurs commentedSame patches, different file names.
Comment #7
lendudeI think this needs a better explanation, the description is not clear.
Also I'm not a fan of using nothing to inherit the parent arguments. I think it would be much clearer to use a placeholder like %. So 40/%/10 in the example.
"If 'Inherit contextual filters' is checked, inherited values can be merged with custom values by using % where you want to use the parent value. For example, %example2.", '%example2' => '40/%/10'
Something along those lines?
Manually tested this and looks good, but this also needs tests.
Comment #8
lendudeComment #9
aerozeppelin commentedAdded tests and updated description as per#7.
Comment #10
dawehnerIdeally we would support tokens here.
Comment #11
lmeurs commented@aerozeppelin: Thanks for the patch, great to have tests! I reviewed your code and don't really get what is happening at the following snippet.
If an argument is left blank by defining
%, the first element of$inherit_argumentswill always be used since$inherit_argumentsdoes not change in the loop. Also the order of arguments changes by not copying inherited arguments from their specific positions.Comment #12
aerozeppelin commented@lmeurs your welcome. Thank you for your feedback.
You are correct, the above snippet is wrong, it performs an incorrect operation. The tests were returning false positives. I've amended the above snippet and added more tests. Let's see.
Comment #13
aerozeppelin commentedComment #20
el1_1el commentedrerolling for 8.6
Comment #24
lubwn commentedPatch works nicely, but maybe if we can push tokens to contexual filters, that would be very nice! I am looking for such a functionality.
Comment #28
ranjith_kumar_k_u commentedComment #31
needs-review-queue-bot commentedThe Needs Review Queue Bot tested this issue. It either no longer applies to Drupal core, or fails the Drupal core commit checks. Therefore, this issue status is now "Needs work".
Apart from a re-roll or rebase, this issue may need more work to address feedback in the issue or MR comments. To progress an issue, incorporate this feedback as part of the process of updating the issue. This helps other contributors to know what is outstanding.
Consult the Drupal Contributor Guide to find step-by-step guides for working with issues.
Comment #32
nikhil_110 commentedAttached patch against Drupal 10.1.x
Patch #28 is not applied for Drupal 10.1.x so Inter-diff file is not added.