Re: [PATCH] Make gnus-summary-delete-article respect group parameters

[Simon Josefsson <jas@extundo.com>]

Steve Youngs <youngs@xemacs.org> writes:

> |--==> "SJ" == Simon Josefsson <jas@extundo.com> writes:
>
>   SJ> Yup.  I also like B DEL to really delete the article, but I don't
>   SJ> think it is important enough to modify existing behaviour.  Let's wait
>   SJ> for someone to report it as a bug, and then we can discuss it. :-)
>
> Well, consider this a bug report.  IMO, B DEL should ignore all group
> parameters especially expiry settings.  The only thing that B DEL
> should do is physically delete the article, completely, forever, do
> not pass go, do not collect 200 euro, the judges decision is final, no
> correspondence will be entered into.

Does this patch do what you want?

--- gnus-sum.el.~6.216.~	2002-07-09 15:15:09.000000000 +0200
+++ gnus-sum.el	2002-07-09 21:26:35.000000000 +0200
@@ -8826,9 +8826,7 @@
     (error "Couldn't open server"))
   ;; Compute the list of articles to delete.
   (let ((articles (sort (copy-sequence (gnus-summary-work-articles n)) '<))
-	(nnmail-expiry-target
-	 (or (gnus-group-find-parameter gnus-newsgroup-name 'expiry-target)
-	     nnmail-expiry-target))
+	(nnmail-expiry-target 'delete)
 	not-deleted)
     (if (and gnus-novice-user
 	     (not (gnus-yes-or-no-p

> Besides, just look at the doc for it...
>
> ,----[ C-h k B DEL ]
> | B DEL runs `gnus-summary-delete-article'
> | 
> | `gnus-summary-delete-article' is an interactive compiled Lisp function
> |   -- loaded from "gnus-sum"
> | (gnus-summary-delete-article &optional N)
> | 
> | Documentation:
> | Delete the N next (mail) articles.
> | This command actually deletes articles.	 This is not a marking
> | command.  The article will disappear forever from your life, never to
> | return.
> | If N is negative, delete backwards.
> | If N is nil and articles have been marked with the process mark,
> | delete these instead.
> `----
>
> Now, I've read that a dozen or so times and I just can't make it mean
> "when we say 'delete' we don't actually mean delete, we mean run the
> article through the normal expiry functions and if you use
> expiry-targets to archive your mail the article will go into the
> archive and won't get deleted at all".

Not the same issue, but there is another issue with B DEL in nnimap,
just FYI:

`nnimap-expunge-on-close'
     Unlike Parmenides, the IMAP designers decided that things that
     don't exist actually do exist.  More specifically, IMAP has the
     concept of marking articles `Deleted' which doesn't actually
     delete them, and this (marking them `Deleted', that is) is what
     nnimap does when you delete a article in Gnus (with `G <DEL>' or
     similar).

     Since the articles aren't really removed when we mark them with the
     `Deleted' flag we'll need a way to actually delete them.  Feel like
     running in circles yet?

     Traditionally, nnimap has removed all articles marked as `Deleted'
     when closing a mailbox but this is now configurable by this server
     variable.

     The possible options are:

    `always'
          The default behavior, delete all articles marked as "Deleted"
          when closing a mailbox.

    `never'
          Never actually delete articles.  Currently there is no way of
          showing the articles marked for deletion in nnimap, but other
          IMAP clients may allow you to do this.  If you ever want to
          run the EXPUNGE command manually, *Note Expunging mailboxes::.

    `ask'
          When closing mailboxes, nnimap will ask if you wish to
          expunge deleted articles or not.