[dev] Re: [cvs] commit: mimp mailbox.php

[Jan Schneider]

Zitat von Chuck Hagenbuch <chuck at horde.org>:

> Quoting Michael M Slusarz <slusarz at mail.curecanti.org>:
>
>> Maybe something is completely jacked up with my installation but before
>> I start digging into this, can someone tell me that I am not
>> completely crazy?  PHP 4.3.8 and Apache 2.0.50 BTW.
>
> You're not crazy; I just figured out what's going on. Horde_Mobile::
> does &amp
> encoding because WML is a strict-ish XML dialect. So it's getting done
twice,
> except for your fixes. MIMP is the only app that uses Horde_Mobile much,
thus
> all the work.

Ah, so you only see this in MIMP? Sounded like it was happening elsewhere
too.

> I see two options:
>
> - continue with the fixes the way you've been doing
> - remove the &amp; encoding in Horde_Mobile's link generation and instead
> require that it be passed fully encoded links.
>
> I think I like the 2nd one better, but neither is an obvious win to me.
> Thoughts?

I think the 2nd is the only option and absolutely makes sense because the
whole purpose of the link generation change was to create XHTML compliant
code, that is well formed XML.

Jan.

--
Do you need professional PHP or Horde consulting?
http://horde.org/consulting/