[dev] [patch] IMP & Kronolith to new Horde::img() API

Chuck Hagenbuch chuck at horde.org
Thu Dec 2 11:42:15 PST 2004


Quoting "Derek P. Moore" <derekm at hackunix.org>:

> Here are the new patches for updating IMP & Kronolith to the new 
> img() API where applicable.
>
> These are corrected for formatting and coding standards per Chuck's request.

Looks good - thanks! They're committed.

> About the IMP patch:
>
> In imp/templates/mailbox/header.inc, in a Horde::link() and Horde::img() was
>   sprintf("%s %s", _("Apply Filters on"), $title)
> which seems completely pointless to me.  I changed it to
>   sprintf(_("Apply Filters to %s"), $title)
> which I'm sure is what was meant in the first place.  I note this because it
> breaks any translations done to _("Apply Filters on").

Makes sense.

-chuck

-- 
"But she goes not abroad in search of monsters to destroy." - John 
Quincy Adams


More information about the dev mailing list