Because contextual filter argument is a text field with a limited value of 128 as maxlength so one can pass max 3-4 fields as arguments. So changing field to textarea is working good and can pass any number of arguments.

CommentFileSizeAuthor
#7 larger_contectual_filter-2802161-3.patch666 bytesreszli
#2 make_contextual_filter-2802161-2.patch821 bytessumitmadan

Comments

sumitmadan created an issue. See original summary.

sumitmadan’s picture

sumitmadan commented
Status: Active » Needs review
StatusFileSize
new make_contextual_filter-2802161-2.patch821 bytes
vyasamit2007’s picture

vyasamit2007 commented
Status: Needs review » Reviewed & tested by the community

Works as expected. I'll leave it to module maintainer if it fits in the context of the module.

jibran’s picture

jibran
he/him
Primary language Urdu
Location Toronto, Canada
commented
Status: Reviewed & tested by the community » Needs work 
+++ b/src/Plugin/views/field/View.php
@@ -190,7 +190,7 @@ class View extends FieldPluginBase {
-        '#type' => 'textfield',
+        '#type' => 'textarea',

This means we have to remove the extra white spaces in submit. I'd rather increase the character limit.

sumitmadan’s picture

sumitmadan commented

Hi @jibran,

there can be N number of argument. How we will handle that case then?

jibran’s picture

jibran
he/him
Primary language Urdu
Location Toronto, Canada
commented
Assigned: Unassigned » damiankloip

@damianklip what do you think?

reszli’s picture

reszli
he/him
Primary language Hungarian
Location Târgu Mureș
commented
Status: Needs work » Reviewed & tested by the community
StatusFileSize
new larger_contectual_filter-2802161-3.patch666 bytes

here's a patch to increase the maxlength of the textfield - worked for me

joel_osc’s picture

joel_osc commented

I like the patch in #7... works great, thx!

ConradFlashback’s picture

ConradFlashback commented

Please commit this.

vladimiraus’s picture

vladimiraus
Primary language English
Location Brisbane 🇦🇺
commented
Assigned: damiankloip » Unassigned
Status: Reviewed & tested by the community » Fixed

Thank you for your contributions.
Merged. 🍻

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

  • vladimiraus committed 7ff7abdf on 8.x-1.x 
    fix: #2802161 Make contextual filter argument field as textarea

By:...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.