The regex in views.inc+33 appears to try to remove remaining %n arguments after all the substitutions are done. However, it has two problems.
First, I believe that \\/ should just be \/ as it's looking for a / and the single backslash is all that's needed to escape it. In the attached patch, I changed the substitutable character from slash to a comma, to fix the regex and make it easer to read.
Second, I think that you really want to find /%n and %n, so the / should be optional.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | 165354.patch | 626 bytes | douggreen |
| viewsargs.patch | 606 bytes | douggreen |
Comments
Comment #1
douggreen commentedHere's the same patch, but one that applies properly (^M on end-of-lines messed me up).
Comment #2
merlinofchaos commentedCommitted. Also the file format too.
Comment #3
(not verified) commented