[geeklog-devel] Deleting comments

Dirk Haun dirk at haun-online.de
Thu Jan 20 02:00:55 EST 2005


Tony,

>Only issue with this sort of stuff is it will clearly break 
>compatiblity. I say you would leave the stubbed out functions in 
>lib-common.php, call the new library equivalent (i.e. COM_deleteComment 
>would call CMT_deleteComment) and then log a warning to error.log that 
>the function is deprecated and will be removed in a future version.

That would still require lib-common.php to include the lib-comment.php
then. Since comments are only used by a few components (and plugins) I
was actually thinking about getting rid of that code entirely so that
those components that actually need it would have to include this.

if (!function_exists ('COM_comment')) {
    require_once ($_CONF['path_system'] . 'lib-comment.php');
}

Yeah, it would break compatibility. But then again, there are more flaws
in the plugin API regarding comments (I'll post something about them
later) and this would be a good opportunity to fix them all at once.

Vinny, I'm not opposed to having a comment.class.php instead of the lib-
comment.php if you think that makes sense.

bye, Dirk


-- 
http://www.haun-online.de/
http://mypod.de/




More information about the geeklog-devel mailing list