Make `move article' work again (bug#33653)

* lisp/gnus/gnus-sum.el (gnus-summary-move-article): Back to while loop
m dolist that blocks nov and active from saving (bug#33653).

1 files changed, 3 insertions, 2 deletions

diff --git a/lisp/gnus/gnus-sum.el b/lisp/gnus/gnus-sum.el
index fd72e1d3abb..8959a2b3d0a 100644
--- a/lisp/gnus/gnus-sum.el
+++ b/lisp/gnus/gnus-sum.el
@@ -10038,7 +10038,7 @@ ACTION can be either `move' (the default), `crosspost' or `copy'."
                  (crosspost "Crosspost" "Crossposting")))
         (copy-buf (save-excursion
                     (nnheader-set-temp-buffer " *copy article*")))
-        art-group to-method new-xref to-groups
+        art-group to-method new-xref article to-groups
         articles-to-update-marks encoded)
     (unless (assq action names)
       (error "Unknown action %s" action))
@@ -10088,7 +10088,8 @@ ACTION can be either `move' (the default), `crosspost' or `copy'."
                   (or (car select-method)
                       (gnus-group-decoded-name to-newsgroup))
                   articles)
-    (dolist (article articles)
+    (while articles
+      (setq article (pop articles))
       ;; Set any marks that may have changed in the summary buffer.
       (when gnus-preserve-marks
         (gnus-summary-push-marks-to-backend article))