Gnus development mailing list
 help / color / mirror / Atom feed
From: Ulrich Pfeifer <pfeifer@buster.informatik.uni-dortmund.de>
Subject: sgnus 0.27, gnus-summary-move-article (nnml)
Date: Wed, 17 Jan 96 15:00:05 +0100	[thread overview]
Message-ID: <9601171400.AA02473@buster.informatik.uni-dortmund.de> (raw)

To me, gnus-summary-move-article seems broken.

First,   
   (cond ((and (eq action 'move)
	      (gnus-check-backend-function
                    'request-move-article gnus-newsgroup-name))
	 (error "The current group does not support article moving"))
	((and (eq action 'crosspost)
	      (gnus-check-backend-function
                    'request-replace-article gnus-newsgroup-name))
	 (error "The current group does not support article editing")))

probably should be:

   (cond ((and (eq action 'move)
	      (not (gnus-check-backend-function
                    'request-move-article gnus-newsgroup-name)))
	 (error "The current group does not support article moving"))
	((and (eq action 'crosspost)
	      (not (gnus-check-backend-function
                    'request-replace-article gnus-newsgroup-name)))
	 (error "The current group does not support article editing")))

Since gnus-check-backend-function returns t on success.

This seems only the first problem:

Signalling: (error "format specifier doesn't match argument type")
  message("Can't find article %d" "nil")
  apply(message ("Can't find article %d" nil))
  gnus-message(3 "Can't find article %d" nil)
  gnus-summary-goto-subject(nil)
  gnus-summary-remove-process-mark(nil)
  (while articles (setq article (pop articles)) (setq art-group (cond ... ... ...)) (if (not art-group) (gnus-message 1 "Couldn't %s article %s" ... article) (let* ... ... ...) (gnus-summary-goto-subject article) (gnus-summary-mark-article article gnus-canceled-mark)) (gnus-summary-remove-process-mark (car articles)))
  (let ((articles ...) (prefix ...) (names ...) (copy-buf ...) art-group to-method new-xref article) (unless (assq action names) (error "Unknown action %s" action)) (when (and ... ...) (setq to-newsgroup ...) (set ... to-newsgroup)) (setq to-method (if select-method ... ...)) (or (gnus-check-backend-function ... ...) (error "%s does not support article copying" ...)) (or (gnus-check-server to-method) (error "Can't open server %s" ...)) (gnus-message 6 "%s to %s: %s..." (caddr ...) (or select-method to-newsgroup) articles) (while articles (setq article ...) (setq art-group ...) (if ... ... ... ... ...) (gnus-summary-remove-process-mark ...)) (gnus-kill-buffer copy-buf) (gnus-set-mode-line (quote summary)))
  gnus-summary-move-article(nil nil nil move)
  call-interactively(gnus-summary-move-article)

Ulrich Pfeifer
--
@J = split //,"J!k Phau^eHeens%rarrot&\ncl t ";
for(0..24){print $J[$_*7%($#J+1)]}


             reply	other threads:[~1996-01-17 14:00 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
1996-01-17 14:00 Ulrich Pfeifer [this message]
1996-01-17 14:20 Ulrich Pfeifer

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=9601171400.AA02473@buster.informatik.uni-dortmund.de \
    --to=pfeifer@buster.informatik.uni-dortmund.de \
    --cc=pfeifer@charly.informatik.uni-dortmund.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).