[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