[development] Deleted records [Have you ever laughed fate in the face?]

sime info at urbits.com
Fri Jul 21 02:40:10 UTC 2006

Larry Garfield wrote:

>Adding a "Deleted" column to node, comment, and various other tables means 
>that every query against those tables would have to be modified to add "WHERE 
>deleted=0".  Between core and contrib, that's thousands of queries that would 
>need to be updated.  
Exactly, no normal module should need to concern itself with something 
that is "deleted". But my original suggestion was for the abstraction 
layer to add the "deleted=0" condition by default. So what you are 
suggesting shouldn't be a problem.

And to repeat. There could also be a way to call db_query() that allows 
special code (eg. upgrade, trashbin, deep search) to over-ride the 
default behaviour and include the deleted rows in the result set.

I can't see any downside except execution speed in the abstraction layer.

More information about the development mailing list