[dev] [cvs] commit: incubator/oscar/templates/videos index.php

Michael Rubinsky mike at theupstairsroom.com
Mon Sep 10 03:56:10 UTC 2007


Quoting Jan Schneider <jan at horde.org>:

> Zitat von Michael Rubinsky <mike at theupstairsroom.com>:
>
>> mrubinsk    2007-09-09 16:32:26 EDT
>>
>>   Modified files:
>>     oscar/templates/videos index.php
>>   Log:
>>   prefix fixes
>>
>>   Revision  Changes    Path
>>   1.3       +12 -12    incubator/oscar/templates/videos/index.php
>>
>
> We should consider doing the opposite, i.e. instead of prefixing the
> keys all over the code, we should translate the video hash in the
> backend driver.

Yea, I thought about that, but at least until we have (or decide we  
want) a full fledged abstracted driver, it seemed like it would be  
uneccesary code/function calls/mapping arrays etc... and we could  
always do a search/replace later if needed.

Of course, just my opinion :)


Thanks,
mike

--
The Horde Project (www.horde.org)
mrubinsk at horde.org

"Time just hates me. That's why it made me an adult." - Josh Joplin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: PGP Digital Signature
Url : http://lists.horde.org/archives/dev/attachments/20070909/6f18cb27/attachment.bin 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-keys
Size: 2013 bytes
Desc: PGP Public Key
Url : http://lists.horde.org/archives/dev/attachments/20070909/6f18cb27/attachment-0001.bin 


More information about the dev mailing list