[dev] [cvs] commit: kronolith/templates/perms perms.inc

[Jan Schneider]

Zitat von Matt Selsky <selsky at columbia.edu>:

> On Jul 16, 2006, at 2:25 PM, Ben Klang wrote:
>
>> On Jul 16, 2006, at 1:54 PM, Matt Selsky wrote:
>>
>>> selsky      2006-07-16 10:54:02 PDT
>>>
>>>  Modified files:
>>>    templates/perms      perms.inc
>>>  Log:
>>>  Group names are unlikely to contain @ characters.
>>>
>>>  Revision  Changes    Path
>>>  1.3       +10 -10    kronolith/templates/perms/perms.inc
>>>
>>>  Chora Links:
>>>    
>>> http://cvs.horde.org/diff.php/kronolith/templates/perms/perms.inc?r1=1.2&r2=1.3&ty=u
>>>
>>> -- 
>>> To unsubscribe, mail: cvs-unsubscribe at lists.horde.org
>> But what if they do contain '@' characters?  Will it break?
>
> Only the labels will break.  I think we need a generic function to   
> convert strings to characters safe for "id" atttributes.  How do we   
> do that in a BC way?

We don't. Repeat it everywhere we need it and mark it with @todo tags.

Jan.

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