Jewett, Jim J wrote:
> Laurens:
>
>> # Start of fix for spaces
>> r1 =3D re.compile(r' ', re.IGNORECASE)
>> url =3D r1.sub('%20', str(url), 0)
>> # End of fix for spaces
>> url =3D string.strip (str (url))
>
> Yes, that will replace spaces (including trailing spaces)
> with a %20.
Well I see now that stripping whitespace and replacing spaces should occur
in that order(currently it's the other way around). Otherwise leading and
trailing spaces will be replaced by %20 as well. I leave this as an
excercise to the reader. I was only trying to prove that adding a fix was
quite easy.
> The problem is in getting the right thing
> passed to CleanUrl in the first place.
As its name implies, CleanUrl is meant to clean URLs. It makes sense to put
it there, no? It's called for other reference attributes, like <img
src="abc">
Regards
-Laurens
_______________________________________________
plucker-list mailing list
[EMAIL PROTECTED]
http://lists.rubberchicken.org/mailman/listinfo/plucker-list
