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

Jan Schneider jan at horde.org
Mon Sep 10 08:14:44 UTC 2007


Zitat von Michael Rubinsky <mike at theupstairsroom.com>:

> 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 it's not a necessity, it's a matter of programming style.

Jan.

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



More information about the dev mailing list