From c3045e6828fb05a41d079f1d268359368fdd863d Mon Sep 17 00:00:00 2001
From: Jarek Radosz <jradosz@gmail.com>
Date: Wed, 14 Jul 2021 06:42:31 +0200
Subject: [PATCH] FIX: Don't try to load badges if there none left (#13695)

Converted `actions` hash to `@action` and added:

```
if (!this.canLoadMore) {
  return;
}
```
---
 .../discourse/app/controllers/badges/show.js  | 64 ++++++++++---------
 1 file changed, 34 insertions(+), 30 deletions(-)

diff --git a/app/assets/javascripts/discourse/app/controllers/badges/show.js b/app/assets/javascripts/discourse/app/controllers/badges/show.js
index 4cab9625da..3c3d9ec71d 100644
--- a/app/assets/javascripts/discourse/app/controllers/badges/show.js
+++ b/app/assets/javascripts/discourse/app/controllers/badges/show.js
@@ -1,7 +1,7 @@
+import EmberObject, { action } from "@ember/object";
 import Controller, { inject as controller } from "@ember/controller";
 import discourseComputed, { observes } from "discourse-common/utils/decorators";
 import Badge from "discourse/models/badge";
-import EmberObject from "@ember/object";
 import I18n from "I18n";
 import UserBadge from "discourse/models/user-badge";
 
@@ -50,35 +50,6 @@ export default Controller.extend({
     return this.siteSettings.enable_badges && hasTitleBadges && hasBadge;
   },
 
-  actions: {
-    loadMore() {
-      if (this.loadingMore) {
-        return;
-      }
-      this.set("loadingMore", true);
-
-      const userBadges = this.userBadges;
-
-      UserBadge.findByBadgeId(this.get("model.id"), {
-        offset: userBadges.length,
-        username: this.username,
-      })
-        .then((result) => {
-          userBadges.pushObjects(result);
-          if (userBadges.length === 0) {
-            this.set("noMoreBadges", true);
-          }
-        })
-        .finally(() => {
-          this.set("loadingMore", false);
-        });
-    },
-
-    toggleSetUserTitle() {
-      return this.toggleProperty("hiddenSetTitle");
-    },
-  },
-
   @discourseComputed("noMoreBadges", "grantCount", "userBadges.length")
   canLoadMore(noMoreBadges, grantCount, userBadgeLength) {
     if (noMoreBadges) {
@@ -96,4 +67,37 @@ export default Controller.extend({
   _showFooter() {
     this.set("application.showFooter", !this.canLoadMore);
   },
+
+  @action
+  loadMore() {
+    if (!this.canLoadMore) {
+      return;
+    }
+
+    if (this.loadingMore) {
+      return;
+    }
+    this.set("loadingMore", true);
+
+    const userBadges = this.userBadges;
+
+    UserBadge.findByBadgeId(this.get("model.id"), {
+      offset: userBadges.length,
+      username: this.username,
+    })
+      .then((result) => {
+        userBadges.pushObjects(result);
+        if (userBadges.length === 0) {
+          this.set("noMoreBadges", true);
+        }
+      })
+      .finally(() => {
+        this.set("loadingMore", false);
+      });
+  },
+
+  @action
+  toggleSetUserTitle() {
+    return this.toggleProperty("hiddenSetTitle");
+  },
 });