I think view argument should not contain space.

Comments

pasqualle’s picture

StatusFileSize
new1.5 KB
pasqualle’s picture

Status: Active » Needs review
katbailey’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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

ineation’s picture

Status: Closed (fixed) » Active

Sorry, to reopen that, but I have a use case where I need to put views arguments (taxonomy terms name) that include a space in it.

I could use taxonomy term id instead but this will be less user friendly for the site admin later.
Right now I removed the validation lines that remove the space and my quicktabs works fine.

But again, I may be missing sthg (security?).

pasqualle’s picture

@ineation: I would suggest to use the "Transform spaces to dashes in URL" option in view for the taxonomy term argument as it result in cleaner urls.
But you are right, the argument may contain a space, we should change it to simple trim.

glennpratt’s picture

Wrong queue

afeijo’s picture

Pasqualle,

I use arguments with spaces, to filter by City Name! A field that came from Profile, where the user when signing in chose all available cities in a object

with that patch of yours, I cannot do that... any hint?

infiniteluke’s picture

Version: 6.x-2.x-dev » 6.x-2.0-rc4
Status: Active » Needs review
StatusFileSize
new812 bytes

Not sure if this patch will roll against other versions, but on 6.x-2.0-rc4, this patch replaces the line that removes whitespace all together with a trim.

avpaderno’s picture

Version: 6.x-2.0-rc4 » 6.x-2.x-dev
Issue summary: View changes
Status: Needs review » Closed (outdated)

I am closing this issue, since it's for a Drupal version no longer supported.