> > It might be better if mgd_delete_article removed all reply articles by
> > default, with an optional parameter that could be set to leave them
> > in place.
>
> Several equally good defaults exist. You could refuse to delete an
> article when it has reply articles unless forced by a similar flag.
> In either case you could choose to either move the reply articles up
> or indeed issue a recursive delete.
>
> The same discussion goes for any container/containee[0] relationship.
> Article/reply article, topic/article, page/subpage, style/element,
> page/page element, article/attachment, and I'm probably missing a few.
>
> I wouldn't want a delete to be synonimous to rm -rf, personally.

You're right, the default should be to remove only the article specified.
In any case, the current remedy is to wrap safeguards around the existing
delete function to guard against removal of the article if reply articles
exist.

The current (1.2.5) admin site doesn't do this, however.

Paul N.



--
This is The Midgard Project's mailing list. For more information,
please visit the project's web site at http://www.midgard-project.org

To unsubscribe the list, send an empty email message to address
[EMAIL PROTECTED]

Reply via email to