Allow to use multiple arguments with spaces

Oh if i apply this patch i get

patch: **** Only garbage was found in the patch input.

Could this be ported to version 7?
thanks!

The current patch is against 6.x, so please keep it there.

If you want to help please test the patch there, i'm pretty sure then it's not that hard to port to 7.x-3.x

When testing this patch i saw that '-' are converted to '/' if you don't have checked "replace spaces with -' this sounds like a bug.
Shouldn't be to hard to fix.

One additional thing i saw is that

+++ b/includes/handlers.incundefined
@@ -1049,15 +1049,21 @@ function views_break_phrase_string($str, &$handler = NULL) {
+  if (strpos($str, ' ') !== FALSE) {

this doesn't allow to use "+" anymore

+++ b/handlers/views_handler_field.incundefined
@@ -1197,7 +1197,8 @@ If you would like to have the characters %5B and %5D please use the html entity
+        $path = str_replace(' ', '- ', $path);

I totally get this change, you want to have a way to distinct between "-" and "--", though i'm wondering whether people actually expect to have " " to "-". Wouldn't it be possible to use -- to "\", then "-" to "/" and then "\" to "-", so for the actual output nothing changes.

I'm trying to write a simpletest without taxonomy terms, but with plain title strings.

Triggering the testbot.

Would it be a crazy time to suggest that we replace the OR operator from a + to something like a | or something that isn't a somewhat special character in urls?

+1 for comment #14!

Looks like the D7 patch in #10 no longer applies (not too much surprise there, but just to save others some time trying) - the issues are with includes/handlers.inc.

In general we certainly needs tests here, especially if we want to get this into D8 as well.

Would it be a crazy time to suggest that we replace the OR operator from a + to something like a | or something that isn't a somewhat special character in urls?

The problem, even it might work, is that we aren't allowed to change existing behavior. This means we have to support still both usecases.

1. +++ b/handlers/views_handler_argument_string.inc
   @@ -181,6 +178,13 @@ class views_handler_argument_string extends views_handler_argument {
   +        $this->value[$item] = strtr(strtr($value, '/', ' '), '--', '-');
   
   @@ -233,7 +237,7 @@ class views_handler_argument_string extends views_handler_argument {
   +      $value =  strtr(strtr($value, '/', ' '), '--', '-');
   
   @@ -243,19 +247,23 @@ class views_handler_argument_string extends views_handler_argument {
   +        $this->value[$item] = strtr(strtr($value, '/', ' '), '--', '-');
   

   Given that we do the same in several places, we should move that to a function.

2. +++ b/handlers/views_handler_field.inc
   @@ -1197,7 +1197,8 @@ If you would like to have the characters %5B and %5D please use the html entity
          if (!empty($alter['replace_spaces'])) {
   -        $path = str_replace(' ', '-', $path);
   +        $path = str_replace('-', '--', $path);
   +        $path = str_replace(' ', '- ', $path);
          }
   

   In general I do prefer to have both patches in one place, tbh.

Coming in late here, but could this simply be an option instead? Here's an example for string contextual filters which provides an option to preserve spaces. Default is the current behaviour.

Fix typo, whitespace and some code.

There's been movement on this issue on a related post: https://www.drupal.org/node/1534720

The 4 year old patch in #18 to views_handler_argument_string.inc still applies cleanly to the latest views 7.x-3.x-dev and if still relevant needs to be reviewed.

Stumbled upon this issue today, patch in #18 works perfectly

A minor adjustment, to remove the trinary operator.

#22 Work for me