"internal:" filter doesn't work with inline CSS

Currently, Path Filter intentionally only handles Drupal paths inside double-quotes. This keeps the regex simple and covers the vast majority of uses. So technically this is probably a (quite valid) feature request.

The issue is finding a well-defined way to detect the end of the path. By translating only paths in double-quotes that was simple ... the path ends when you hit the closing double quote. If we remove that restriction, we need to decide which characters mark the end of the path. Some obvious ones would include whitespace, double or single quote, closing paren, bracket or brace. Anything else?

I'm just concerned about having a robust criterion. I don't want it to accidentally grab only part of, or gobble up more than the intended path.

Thoughts?

i think url(internal:themes/mytheme/img/test.gif) is very save from CSS point of view. An inline CSS style can only inside a style="" attribute and the content should contain a URL surrounded by url( and ). But you cannot say for sure there will be no spaces inside - for e.g. url ( or url ( internal: and so on is absolutely possible.

Additional, if you don't test for a path or path and filename you are unsafe... but there are tons of regexp examples on the net and inside drupal for such tasks. shouldn't be a major problem.

For what it's worth, adding filtering on single quotation marks would seem to be the most sure and least resource intensive solution. That would be legal within inline CSS and avoid needing crazy new regular expression processing.

^{benjamin, Agaric Design Collective}

Both double and single quotes are now handled in the D6 branch, which should fix this issue. Changes will be backported soon.

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