"EP" == Evan Prodromou evan@wikitravel.org writes:
EP> Attached is a patch to escape the descriptions in log EP> entries. It's pretty much self-contained, and I think EP> qualifies as a Righteous Patch. B-)
Well, I was wrong, as Brion pointed out: links in the $action param were getting escaped. I went through and did all the escaping at the caller end, per his suggestion, but it was tedious, and I had to do the combining of the two args to addEntry ($action and $comment) manually in each space, too. It was a hassle, so I just went ahead and changed the contract of addEntry(). Now, the $action param is left alone, so it can contain wiki text, but the $comment param is escaped. It's used unescaped for the change comment.
Attached is a patch. I'll be less cocky this time around.