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).

diff --git a/lisp/gnus/gnus-sum.el b/lisp/gnus/gnus-sum.el
index fd72e1d3abbc240c3cbca0387edee1c960ee99f5..8959a2b3d0a65c47e3455b2ab9d17642c9dea54b 100644 (file)
--- 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))