[development] module_exist() / module_exists()

Rob Barreca rob at electronicinsight.com
Thu Aug 31 20:49:07 UTC 2006


I'd say, forget the wrapper function. So much changes between each 
Drupal release cycle, as long as this is well documented in the 
upgrading handbook, a simple find and replace will do it.

Rob Roy Barreca
Founder and COO
Electronic Insight Corporation
http://www.electronicinsight.com
rob at electronicinsight.com



Greg Knaddison - GVS wrote:
> On 8/31/06, Konstantin Käfer <kkaefer at gmail.com> wrote:
>> Well, the bad thing about wrapper functions is that people won't
>> change them if it is not necessary. If we take the wrapper function
>> out after the next release, we will face the same problem.
>
> I don't have a strong feeling one way or another, but it's common for
> a deprecated wrapper to log the fact that it is deprecated to motivate
> people to fix the problems.
>
> A module_exist() that does a watchdog and then proceeds would be a
> little slower and annoying, so it would motivate people.
>
> Regards,
> Greg
>


More information about the development mailing list