fix: watch for flags once per post

src/AdvancedFlagging.ts

@@ -232,7 +232,7 @@ function Setup(): void {
         setupConfiguration();
 
         // TODO make more specific & remove jQuery
-        // appends advanced flagging link to new/edited posts
+        // appends Advanced Flagging link to new/edited posts
         $(document).ajaxComplete(() => setupPostPage());
 
         isDone = true;

src/UserscriptTools/Post.ts

@@ -291,8 +291,8 @@ export default class Post {
         // Watch for manual flags if the user has chosen to do so
         const watchFlags = Store.config[Cached.Configuration.watchFlags];
 
-        // don't watch for flags if the user doesn't want to
-        // exclude listener from running in deleted posts
+        // Don't watch for flags if the user doesn't want to.
+        // Exclude listener from running in deleted posts.
         if (!watchFlags || this.deleted) return;
 
         addXHRListener(xhr => {
@@ -338,7 +338,7 @@ export default class Post {
                 await addProgress(event, flagType, this);
                 $(this.flagged).fadeIn();
             }, { once: true });
-        });
+        }, this.id);
     }
 
     public filterReporters(feedbacks: FlagTypeFeedbacks): Reporter[] {

src/shared.ts

@@ -113,8 +113,16 @@ export async function delay(milliseconds: number): Promise<void> {
 
 // Credits: https://github.com/SOBotics/Userscripts/blob/master/Natty/NattyReporter.user.js#L101
 const callbacks: ((request: XMLHttpRequest) => void)[] = [];
+const postIds: number[] = [];
+
+export function addXHRListener(
+    callback: (request: XMLHttpRequest) => void,
+    postId?: number
+): void {
+    // used so that the script watches for flags once per post
+    if (postId && postIds.includes(postId)) return;
+    else if (postId) postIds.push(postId);
 
-export function addXHRListener(callback: (request: XMLHttpRequest) => void): void {
     callbacks.push(callback);
 }