<HTML><BODY style="word-wrap: break-word; -khtml-nbsp-mode: space; -khtml-line-break: after-white-space; "><BR><DIV><DIV>On Jul 17, 2006, at 2:40 PM, Neil Drumm wrote:</DIV><BR class="Apple-interchange-newline"><BLOCKQUOTE type="cite"><P style="margin: 0.0px 0.0px 0.0px 0.0px"><FONT face="Helvetica" size="3" style="font: 12.0px Helvetica">I think this is a good change. Forgiveness is a good thing to have. But</FONT></P> <P style="margin: 0.0px 0.0px 0.0px 0.0px"><FONT face="Helvetica" size="3" style="font: 12.0px Helvetica">it simply can't be added without people looking at it. Same goes for</FONT></P> <P style="margin: 0.0px 0.0px 0.0px 0.0px"><FONT face="Helvetica" size="3" style="font: 12.0px Helvetica">every other feature.</FONT></P> </BLOCKQUOTE><BR></DIV><DIV>just to put everybody in the loop here: chx and i have been discussing the idea of a deletion API for drupal core, which would address killes' modular concerns as regards this functionality, as well as allow us to break out all the confirm code into a seperate module. what this means is that site admins would have the following options for a deletion workflow:</DIV><DIV><BR class="khtml-block-placeholder"></DIV><DIV>1. the current scheme: no trashbin, with confirm screens</DIV><DIV>2. trashbin with confirm screens</DIV><DIV>3. trashbin with no confirm screens</DIV><DIV>4. no trashbin, no confirm screens (for those who like to live dangerously!)</DIV><DIV><BR class="khtml-block-placeholder"></DIV><DIV>simply enable or disable either of the confirm.module or trashbin.module to get the desired combination.</DIV><DIV><BR class="khtml-block-placeholder"></DIV><DIV>in addition, the API would provide non-core modules with greater access to the core's deletion cycle (which is currently fairly closed)</DIV><DIV><BR class="khtml-block-placeholder"></DIV><DIV>i've already begun work on this, and my goal is to have this up for testing within one week, if all goes well.</DIV><DIV><BR class="khtml-block-placeholder"></DIV><DIV>in the meantime, i would still love to get reviews of the original trashbin patch, as most of that code will be used in this new API schema.</DIV><DIV><BR class="khtml-block-placeholder"></DIV><DIV>patch is located here: <A href="http://drupal.org/node/35422#comment-115495">http://drupal.org/node/35422#comment-115495</A></DIV><DIV><BR class="khtml-block-placeholder"></DIV><DIV>test site is located here: <A href="http://undo.xcarnated.com">http://undo.xcarnated.com</A>/</DIV><BR></BODY></HTML>