Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bulk moderation of messages is broken in Sympa 6.2.74 #1926

Open
dpc22 opened this issue Jan 22, 2025 · 0 comments
Open

Bulk moderation of messages is broken in Sympa 6.2.74 #1926

dpc22 opened this issue Jan 22, 2025 · 0 comments
Labels

Comments

@dpc22
Copy link
Contributor

dpc22 commented Jan 22, 2025

Version

6.2.74

Installation method

My own rpm, derived from the "official" RHEL 9 rpm for 6.2.74.

Expected behavior

List moderators should be able to distribute multiple moderated messages at a go by clicking on multiple checkboxes on the modindex screen, including the "Check All" checkbox at the top of the table.

Actual behavior

Clicking on any checkbox after the first row generates a popup window for the first message in the list only.

Steps to reproduce

Queue multiple messages for moderation on a test list, and attempt to select the second message from the modindex screen.

Additional information

As I just mentioned on the sympa-users list, I think that this is just down to a missing '>' in the new default modindex.tt2:

--- sympa-6.2.74/default/web_tt2/modindex.tt2	2024-12-16 10:46:08.000000000 +0000
+++ /etc/sympa/web_tt2/modindex.tt2	2025-01-22 14:10:05.594077652 +0000
@@ -69,7 +69,7 @@
                     <div class="small-6 medium-3 columns" role="cell">
                         <a href="[% 'ajax/viewmod' | url_rel([list,msg.key]) %]"
                            data-reveal-id="mainviewmod" data-reveal-ajax="true"
-                           data-tooltip aria-haspopup="true"
+                           data-tooltip aria-haspopup="true">
                         [% UNLESS msg.value.subject.length ~%]
                             <i>[%|loc%]No subject[%END%]</i>
                         [%~ ELSE ~%]

One other oddity that I see is that whenever messages are distributed, I see an additional confirmation screen:

"Distribute

Please select one or more topic(s) that corresponds to the messages you wish to distribute:"

which is empty as we don't have any topics configured on our Sympa server. I can't remember if this used to be the case with Sympa 6.2.72. There are "Confirm" and "Back" buttons which work.

@dpc22 dpc22 added the bug label Jan 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant