For some reason, the user badge page, user/%/badges, ignores the global badge limit setting ("Number of badges to display").
Since this page is always accessible, this limit must be respected.

This patch fixes it.

CommentFileSizeAuthor
#3 user_badges-respect_limit-1204284.patch423 bytesaaronbauman
user-badge-respect-my-authoritah.patch561 bytesaaronbauman

Comments

aaronbauman’s picture

aaronbauman
he/him
commented
Status: Active » Needs review
shabana.navas’s picture

shabana.navas commented
Status: Needs review » Closed (fixed)
aaronbauman’s picture

aaronbauman
he/him
commented
Status: Closed (fixed) » Needs review
StatusFileSize
new user_badges-respect_limit-1204284.patch423 bytes

Not sure why you closed this, since you didn't leave any comment.
This issue -- a one-liner -- persists in 6.x-1.x-dev

Here's a re-rolled patch.

shabana.navas’s picture

shabana.navas commented

This problem has been fixed in subsequent versions of userbadges. Please upgrade to the latest version.

shabana.navas’s picture

shabana.navas commented
Status: Needs review » Closed (fixed)
aaronbauman’s picture

aaronbauman
he/him
commented
Status: Closed (fixed) » Needs review

This issue is against 6.x branch, in which this bug persists.
Here is a link to the exact line in the repository to which this patch applies.

If 6.x branch is no longer supported, it should be noted on the project page and removed from recommended versions.

shabana.navas’s picture

shabana.navas commented
Status: Closed (fixed) » Needs review

The patch will be committed shortly.

shabana.navas’s picture

shabana.navas commented
Status: Needs review » Closed (fixed)

Committed patch in https://drupal.org/node/1204284#comment-7561043.

Status: Needs review » Closed (fixed)

Correcting issue status, see #2125263: Issue status not taken from last comment during D7 migration.